Skip to content

Fix #3608 add TOC tooltip customization#3609

Merged
tdipisa merged 3 commits intogeosolutions-it:masterfrom
MV88:3608_toc_tooltip
Mar 11, 2019
Merged

Fix #3608 add TOC tooltip customization#3609
tdipisa merged 3 commits intogeosolutions-it:masterfrom
MV88:3608_toc_tooltip

Conversation

@MV88
Copy link
Copy Markdown
Contributor

@MV88 MV88 commented Mar 11, 2019

Description

This pr add the possibility to configure and customize

Note for testers

You cannot test this unless the localConfig.json has the "TOC" section with "tooltipOptions" configured

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)
Actually the tooltip of the TOC nodes shows only the title.

What is the new behavior?
you can configure to use other node properties for composing the tooltip
by defaults the various fragments are joined with " - "

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

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

@MV88 MV88 requested a review from offtherailz March 11, 2019 11:57
@ghost ghost assigned MV88 Mar 11, 2019
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 11, 2019

Coverage Status

Coverage decreased (-0.02%) to 80.967% when pulling 7402472 on MV88:3608_toc_tooltip into 891f3db on geosolutions-it:master.

Co-Authored-By: MV88 <matteo.velludini.geosolutions@gmail.com>
@MV88 MV88 changed the title Fix #3608 add TOC tooltip custmoization Fix #3608 add TOC tooltip customization Mar 11, 2019
Co-Authored-By: MV88 <matteo.velludini.geosolutions@gmail.com>
@tdipisa tdipisa merged commit cd62de6 into geosolutions-it:master Mar 11, 2019
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.

TOC mouse over tooltip customization

5 participants