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

Add UI for filtering on class level #417

Open
Famlam opened this issue Nov 5, 2022 · 0 comments
Open

Add UI for filtering on class level #417

Famlam opened this issue Nov 5, 2022 · 0 comments
Labels

Comments

@Famlam
Copy link
Contributor

Famlam commented Nov 5, 2022

Spin-off from the discussion in osm-fr/osmose-backend#1595

On the front-end, a user cannot easily disable warnings related to a single class. Checkboxes only exist on an item level. (And I guess 99% of the users will only use the main UI)
Hence, if you disable e.g. Inconsistent tagging of construction (item 4070, class 40701) you immediately disable Finished construction and so on.

Proposal:
allow filtering the active analysers by making the checkboxes expandable to i.e. class level. So instead of e.g.:

[v] addresses
[v] combinations
[v] deprecated

... have user-expandable lists, like:

[v] ▷addresses
[v] ▼combinations
  [v] missing tag
  [v] suspicious tag combination
  [v]  {{0.key}} on a relation without {{1.key}}={{1.value}}
  [v]  incomplete usage of {{0.key}} on a way without {{1.key}}
  [v]  Use transition only as value of placement
[v] ▷deprecated

And preferably the 'Hide from map' button will only disable the specific class, not the entire item. Currently it shows Do not want to see [class title] markers? [Hide from map], but clicking it disables the entire item

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants