-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Web maps want #504
Update Web maps want #504
Conversation
Add privacy and accessibility tags. Update description to be less lame. cc @nchan0154 to validate these changes, because she's co-author.
|
Name | Link |
---|---|
🔨 Latest commit | 855705a |
Always a 👍 for putting accessibility at the forefront! |
Thanks for these improvements. Co-authored-by: Robert Linder <robert.vuj.linder@outlook.com>
This would be great to get merged, in light of the survey. Thanks! |
@aarongustafson please merge. If browsers are watching this space for action items, we would like to keep it fresh. Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of this update looks good. Would folks be open to some grammar improvements? E.g. the sentence "Web maps are often seen as used only for store locator pages."
@tantek thanks, of course! I don't know if what I did is what you had in mind, but if you can improve it, please have a shot at it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better! LGTM. Thanks for this update.
Add privacy and accessibility tags. Update description to be less lame. cc @nchan0154 to validate these changes, because she's co-author.