Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Infra UI] Replace jump menu with EUI datepicker #20422

Conversation

weltenwort
Copy link
Member

@weltenwort weltenwort commented Jul 3, 2018

This replaces the "Jump to time" menu with the EUI datepicker. It also tweaks the design of the streaming toggle. The date and time formats of the datepicker are depend on the current browser locale via the corresponding Moment.js format string tokens.

It also stores the current scroll state of the log view in the url such that it persists across reloads and can be shared between users.

Normal
image

Open
image

Streaming
image

@weltenwort weltenwort added WIP Work in progress :Ingest UI labels Jul 3, 2018
@weltenwort weltenwort self-assigned this Jul 3, 2018
@weltenwort weltenwort changed the title [WIP] Replace jump menu with EUI datepicker [WIP] [Infra UI] Replace jump menu with EUI datepicker Jul 3, 2018
@simianhacker simianhacker mentioned this pull request Jul 3, 2018
4 tasks
@weltenwort weltenwort added the Feature:Metrics UI Metrics UI feature label Jul 4, 2018
@weltenwort weltenwort force-pushed the infra-ui-enhancement-eui-datepicker branch from f2da831 to bb20bd7 Compare July 6, 2018 14:46
@elasticmachine
Copy link
Contributor

💔 Build Failed

@weltenwort weltenwort force-pushed the infra-ui-enhancement-eui-datepicker branch from a6dd6f4 to c25b752 Compare July 9, 2018 16:55
@elasticmachine
Copy link
Contributor

💔 Build Failed

@weltenwort weltenwort force-pushed the infra-ui-enhancement-eui-datepicker branch from c25b752 to 8a6386d Compare July 12, 2018 09:50
@elasticmachine
Copy link
Contributor

💔 Build Failed

@weltenwort weltenwort force-pushed the infra-ui-enhancement-eui-datepicker branch from 8a6386d to 2f82f2a Compare July 12, 2018 14:02
This replaces the "Jump to time" menu with the EUI datepicker. It also
tweaks the design of the streaming toggle. The date and time formats of
the datepicker are depend on the current browser locale via the
corresponding Moment.js format string tokens.
This wraps the `ScrollableLogTextStreamView` in a HoC, which in turn
uses the `withStateFromLocation` HoC to retrieve the scroll state from
the URL. It also updates the location accordingly when the visible log
interval changes.

The connection between the location and the app state is implemented as
a HoC, because the location can be seen as a part of the UI.
Additionally, this keeps the implementation agnostic of the state
management used in the app, be it redux or apollo-link-state.

The serialization is performed using the `mapStateToRisonAppLocation`
and `mapRisonAppLocationToState` functions, that use the `_a` key of the
query string for the sake of consistency with many parts of Kibana.
@weltenwort weltenwort force-pushed the infra-ui-enhancement-eui-datepicker branch from 2f82f2a to 4c588e2 Compare July 12, 2018 14:18
@weltenwort weltenwort changed the title [WIP] [Infra UI] Replace jump menu with EUI datepicker [Infra UI] Replace jump menu with EUI datepicker Jul 12, 2018
@weltenwort weltenwort added review and removed WIP Work in progress labels Jul 12, 2018
@weltenwort weltenwort requested a review from simianhacker July 12, 2018 15:00
@elasticmachine
Copy link
Contributor

💔 Build Failed

Copy link
Member

@simianhacker simianhacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (tested)

@weltenwort weltenwort merged commit 4bfbb4a into elastic:feature-infra-ui Jul 13, 2018
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.

3 participants