Conversation
|
Pinging @elastic/kibana-gis (Team:Geo) |
from #96791 (comment) @markharwood agreed. I have replaced the next and previous icons with arrowRight and arrowLeft EUI icons. Thanks for the feedback. |
from #96791 (comment) @mdefazio Thanks for feedback. The show/hide icon is now colored with "primary" when the timeslider is open. I will need some help adding animations for opening. |
|
Some feedback @nreese @miukimiu:
|
|
One more idea as I use the Timeslider. When I initiate the control, it's a snapshot of the global time picker's time range. Let's take "last 10 minutes for example". A couple ideas. If the global time-picker has auto-update ON; the time-slider should refresh the snapshot as well. @nreese @miukimiu |
|
@elasticmachine merge upstream |
thomasneirynck
left a comment
There was a problem hiding this comment.
mostly nits. Still playing with it, it's really fun.
x-pack/plugins/maps/public/connected_components/timeslider/timeslider.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/maps/public/connected_components/toolbar_overlay/toolbar_overlay.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/maps/public/connected_components/timeslider/timeslider.tsx
Show resolved
Hide resolved
x-pack/plugins/maps/public/connected_components/timeslider/timeslider.tsx
Show resolved
Hide resolved
x-pack/plugins/maps/public/connected_components/timeslider/timeslider.tsx
Show resolved
Hide resolved
This fix was merged in EUI: elastic/eui#4781.
I opened a PR in EUI with the new icons: elastic/eui#4810. The idea is to use the icons
The design was not expecting the time to be so detailed (with milisencods). But as we discussed we can move the controls to the bottom row. |
|
@elasticmachine merge upstream |
thomasneirynck
left a comment
There was a problem hiding this comment.
thx, this is such a good feature.
|
Just dropping a note on some of the related bugs and requests that were reported/requested in EUI: |
|
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / general / X-Pack Accessibility Tests.x-pack/test/accessibility/apps/spaces·ts.Kibana spaces page meets a11y validations a11y test for space selection pageStandard OutStack TraceMetrics [docs]Module Count
Async chunks
Page load bundle
Unknown metric groupsReferences to deprecated APIs
History
To update your PR or re-run it, just comment with: |
* [Maps timeslider] * just arrowLeft and arrowRight icons * tslint * color icon when timeslider is open, auto select first section on open * increase width to prevent timeslider from changing sizes during interaction * fix filters disappearing when timeslice advances * use shorter date format for ticks * review feedback * do not show timeslider button when map is embedded Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
* [Maps timeslider] * just arrowLeft and arrowRight icons * tslint * color icon when timeslider is open, auto select first section on open * increase width to prevent timeslider from changing sizes during interaction * fix filters disappearing when timeslice advances * use shorter date format for ticks * review feedback * do not show timeslider button when map is embedded Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Nathan Reese <reese.nathan@gmail.com>




Note: Replaces #96791. #96791 included client-side masking functionality which is proving to have more edge cases then anticipated. This PR removes client-side masking to get to a timeslider MVP that is shippable. Client-side masking can be added in future iterations.
Implements #27714
This PR adds a timeslider to maps. Manipulating the time slider will set a new piece of redux state, timeslice, that will be used by layers to only show data for that timeslice.