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

Measure and Feature info tools should be mutually exclusive #258

Closed
offtherailz opened this issue Oct 26, 2015 · 2 comments
Closed

Measure and Feature info tools should be mutually exclusive #258

offtherailz opened this issue Oct 26, 2015 · 2 comments

Comments

@offtherailz
Copy link
Member

offtherailz commented Oct 26, 2015

Measure tool and info tool can be activated at the same time. They should be in a toogle group or something else.

To reduce the number of buttons in the toolbar I suggest to put the tools that can be active on the map in the setting panel, and when one of them is active, it should be placed on top.

Example: on startup the info button is on top of the bar, as now is. When the user opens the settings panel , maybe in an area dedicated to the tools, and he selects the measure tool, the measure tool appears active on the top of the bar. the info button is now available again in the settings panel.

@simboss simboss changed the title Measure and Feture info tools should be mutually exclusive Measure and Feature info tools should be mutually exclusive Oct 28, 2015
@simboss simboss added this to the W372016 milestone Jul 29, 2016
@Gnafu Gnafu self-assigned this Sep 27, 2016
@Gnafu
Copy link
Contributor

Gnafu commented Sep 27, 2016

Can't we just disable the measure tool when the user clicks on the Info tool and vice-versa?

@mbarto
Copy link
Contributor

mbarto commented Sep 27, 2016

I would prefer the tools to be independent and make this kind of decisions at an higher level

@Gnafu Gnafu removed their assignment Sep 27, 2016
@mbarto mbarto modified the milestones: W422016, W372016 Oct 3, 2016
@Gnafu Gnafu modified the milestones: W422016, W372016 Oct 7, 2016
@Gnafu Gnafu added ready and removed in progress labels Oct 17, 2016
@simboss simboss modified the milestones: R52016, R42016 Nov 23, 2016
@simboss simboss removed the ready label Nov 23, 2016
Gnafu added a commit to Gnafu/MapStore2 that referenced this issue Jan 26, 2017
@ghost ghost assigned offtherailz Feb 9, 2017
@ghost ghost removed the pending review label Feb 9, 2017
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

4 participants