Skip to content

Extend Apply dimension to connect with Chart Axis current Time. #12408#12413

Merged
offtherailz merged 11 commits into
geosolutions-it:masterfrom
rowheat02:applyDimensionForAxis
May 21, 2026
Merged

Extend Apply dimension to connect with Chart Axis current Time. #12408#12413
offtherailz merged 11 commits into
geosolutions-it:masterfrom
rowheat02:applyDimensionForAxis

Conversation

@rowheat02
Copy link
Copy Markdown
Contributor

Description

This PR extends the apply Dimension to be able to connect with the axis of the chart(if context has no timeline)

Screen.Recording.2026-05-20.at.16.40.15.mov

Now the user can directly connect to the axis's current Time. If a timeline exists, timeline currentTime is used. If not, then the current value saved in the axis from filterWidget will be used.

Other improvements / fixes from report1 and here also included

Please check if the PR fulfills these requirements

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

  • Bugfix
  • Enhancement

Issue

What is the current behavior?

#12408
Fixes Reported here #12268 (comment)
Improvement reported here: #12267 (comment)

What is the new behavior?

  • Apply dimension on axis

  • Fixes for Slider-related bugs/glitches and fixes for reported issues/improvements mentioned above in "What is the current behavior?"

  • User can configure the on-axis to enable a tooltip only on tickValues match.

    Screen.Recording.2026-05-20.at.17.19.56.mov

Breaking change

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

  • Yes, and I documented them in migration notes
  • No

Other useful information

@rowheat02 rowheat02 added this to the 2026.01.02 milestone May 20, 2026
@rowheat02 rowheat02 requested a review from offtherailz May 20, 2026 11:46
@cla-bot cla-bot Bot added the CLA Ready label May 20, 2026
@rowheat02 rowheat02 marked this pull request as ready for review May 20, 2026 12:45
@rowheat02 rowheat02 linked an issue May 20, 2026 that may be closed by this pull request
1 task
@offtherailz offtherailz enabled auto-merge (squash) May 21, 2026 12:02
@offtherailz offtherailz merged commit fb6d739 into geosolutions-it:master May 21, 2026
11 of 12 checks passed
@ElenaGallo
Copy link
Copy Markdown
Contributor

@rowheat02 From the Chart Widget, the Show the current time on the axis is not visible.

test_time

Sample map here: https://dev-mapstore.geosolutionsgroup.com/mapstore/#/context/notimeline/66588

@offtherailz
Copy link
Copy Markdown
Member

Request failed due to following response errors:

  • Pull request Pull request is a draft

@rowheat02
Copy link
Copy Markdown
Contributor Author

@ElenaGallo Show current time label is changed to "Highlight selected time". Logically the same to Show current Time.
Changed to make sense to connect from the widget also.

offtherailz pushed a commit that referenced this pull request May 22, 2026
#12413) (#12423)

* Slider cache issue fixes

* Apply dimension for axis

* fix: show tick label instead of value

* configure hover only for custom Ticks

* update elevation chart after when it's value from params changes

* fix: avoid negative max height

* fix: translations

* fix: labels

* fix: local env crash due to invalid cleanup

* fix: apply clean up fordeleted filter interaction
huntsmarinho pushed a commit to huntsmarinho/MapStore2 that referenced this pull request May 31, 2026
…lutions-it#12408 (geosolutions-it#12413)

* Slider cache issue fixes

* Apply dimension for axis

* fix: show tick label instead of value

* configure hover only for custom Ticks

* update elevation chart after when it's value from params changes

* fix: avoid negative max height

* fix: translations

* fix: labels

* fix: local env crash due to invalid cleanup

* fix: apply clean up fordeleted filter interaction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

3 participants