Skip to content

Closes #2203. GetFeatureInfo active by default#2459

Merged
offtherailz merged 5 commits intogeosolutions-it:masterfrom
kappu72:i#2203
Dec 7, 2017
Merged

Closes #2203. GetFeatureInfo active by default#2459
offtherailz merged 5 commits intogeosolutions-it:masterfrom
kappu72:i#2203

Conversation

@kappu72
Copy link
Copy Markdown
Contributor

@kappu72 kappu72 commented Dec 6, 2017

Description

The GFI tool has become active by default. The user just needs to click on map to get the feature info. The GFI is stopped if Annotation or Grid plugins are editing, draw or measure support are active.

Issues

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
The GFI is enabled from the btn in toolbar. It's disabled by some other tolls but not reenabled.

What is the new behavior?
The GFI is enabled by default, and disabled/enabled under some circumstance.

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes
  • No

Other information:
It is possible to control the GFI, by setting the state variables: enabled and disableAlwaysOn. #2462
Remove "alwaysVisible": true from override tool configuration in identify config (localconfig.json)

@ghost ghost assigned kappu72 Dec 6, 2017
@ghost ghost added In Test labels Dec 6, 2017
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 81.101% when pulling 888c007 on kappu72:i#2203 into 18e36c9 on geosolutions-it:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.09%) to 81.168% when pulling 1487118 on kappu72:i#2203 into 18e36c9 on geosolutions-it:master.

@offtherailz offtherailz merged commit 22afc99 into geosolutions-it:master Dec 7, 2017
@ghost ghost added the Accepted label Dec 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Always-on GetFeatureInfo

4 participants