Skip to content

Fix #3279. Added settings form for timeline #3328

Merged
offtherailz merged 2 commits intogeosolutions-it:c127_geonode_integrationfrom
offtherailz:fix_#3279
Nov 12, 2018
Merged

Fix #3279. Added settings form for timeline #3328
offtherailz merged 2 commits intogeosolutions-it:c127_geonode_integrationfrom
offtherailz:fix_#3279

Conversation

@offtherailz
Copy link
Copy Markdown
Member

@offtherailz offtherailz commented Nov 12, 2018

Description

This PR introduces settings form for animation settings of timeline/playback plugin. This is only the UI (connected with the state), but the functionalities, of course, have yet to be implemented.

Also added i18n for the 2 plugins, that was missing.
image

Issues

Please check if the PR fulfills these requirements

What kind of change does this PR introduce?

  • Feature

What is the current behavior?
Playback settings window was a draft

What is the new behavior?
Playback settings window contains all the settings that need to be implemented.

Does this PR introduce a breaking change?

  • No

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.03%) to 80.852% when pulling 5e6deea on offtherailz:fix_#3279 into 704ae99 on geosolutions-it:c127_geonode_integration.

@giohappy
Copy link
Copy Markdown
Contributor

Generally we will have a guiding layer that will dictate the animation steps. I suppose the "Following" option is for that, but I would make it more automated. IMHO the "Following" option is not needed: if a guide layer is selected the amination automatically follows it.
I only keep the "Fixed step" option, which will be off by default if a guide layer is selected, true otherwise. For this option an help tooltip can explain what's the effect of forcing it on or off.

@giohappy
Copy link
Copy Markdown
Contributor

so @offtherailz, to resume the "Fixed step" behaviours:

  • on -> select guide layer -> off
  • off -> deselct guide layer -> on
  • on -> off -> select first available guide layer
  • off -> on -> deselect guide layer

The "Following" remains because it's about following marker while animating...

@offtherailz
Copy link
Copy Markdown
Member Author

All correct, please move your comment in the issue for the effective implementation, here:
#3277

@offtherailz offtherailz merged commit 8f91ebc into geosolutions-it:c127_geonode_integration Nov 12, 2018
@offtherailz offtherailz added this to the 2018.02.02 milestone Nov 12, 2018
@tdipisa tdipisa modified the milestones: 2018.02.02, 2018.03.00 Nov 16, 2018
@offtherailz offtherailz deleted the fix_#3279 branch February 18, 2020 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants