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

Visually display when operations are not-applicable #145

Open
alukach opened this issue Jun 15, 2015 · 3 comments
Open

Visually display when operations are not-applicable #145

alukach opened this issue Jun 15, 2015 · 3 comments

Comments

@alukach
Copy link
Member

alukach commented Jun 15, 2015

In the spirit of "data-first" GIS, menu items should visually display whether they are applicable to the current selection.

This would likely be done by firing a signal on any change to the L.Dropchop.app.layerlist.selection.list object. All operations would subscribe to this signal, and alter their dom element to have a lighter text color if not applicable. I think a user should still be able to click on the operation and attempt to use it, as the error message returned from failed validation provides feedback to the user on the reason why an operation is not applicable.

@mapsam
Copy link
Member

mapsam commented Jun 17, 2015

I always assumed the operation wouldn't be clickable, but we would show a tooltip when the user hovered over a disabled button, but this is also interesting - and seems like a good way to get the user to register that their data do not match the operation.

@alukach
Copy link
Member Author

alukach commented Jun 17, 2015

A tooltip showing why an operation is disabled would also work. I just want to make sure that there's some feedback available as to why it's disabled.

@mapsam mapsam modified the milestone: Alpha Sep 8, 2015
@mapsam
Copy link
Member

mapsam commented Sep 12, 2015

Make operations available when data selection matches requirements for the operations:

dropchop-data-first

@mapsam mapsam mentioned this issue Sep 12, 2015
37 tasks
@mapsam mapsam removed this from the Alpha milestone Oct 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants