Skip to content
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

Explain to users there is a limit to amount of notes displayed #3744

Open
danieldegroot2 opened this issue Oct 8, 2022 · 2 comments
Open
Labels
notes Related to the notes feature ui User Interface

Comments

@danieldegroot2
Copy link
Contributor

Explain to users they may only see a limited amount of notes displayed when they enable the notes layer while not zoomed in closer. This may be done next to the checkbox for the notes layer in the sidebar (the notes button in the menu will usually be pressed when the user is already zoomed in far enough to see all the notes). It may be a static message.

I can understand this is common sense, but if you don't know this, you'll zoom into an area without notes and suddenly see a bunch of notes appear out of nowhere, which may be confusing to the user. Or, you may not see notes of interest to you and miss them because of it. (Also, the data layer already asks the user to zoom in, but the reason is slightly different afaik.)

@AntonKhorev
Copy link
Contributor

The amount of notes displayed is also limited by javascript bugs. Once you receive a note without comments, the client-side code trips over it and stops displaying notes in the area. This bug is easily fixable.

@lectrician1 lectrician1 added ui User Interface notes Related to the notes feature labels Nov 13, 2022
@mmd-osm
Copy link
Contributor

mmd-osm commented Dec 5, 2022

I think we should revisit this one only after fixing #3617 . Most likely that’s the only issue here really.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notes Related to the notes feature ui User Interface
Projects
None yet
Development

No branches or pull requests

4 participants