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

Fix responsive sizing of datepicker #29

Merged

Conversation

cchaos
Copy link

@cchaos cchaos commented Jan 25, 2019

No description provided.

@nreese nreese merged commit 066a4fe into nreese:maps_time_picker Jan 25, 2019
nreese added a commit that referenced this pull request Jan 31, 2019
…er in Top Nav (elastic#29235)

* Embed timepicker in query bar (elastic#29130)

* replace kbnTimepicker directive with EuiSuperDatePicker

* remove kbnTimepicker directive

* remove bootstrap datepicker

* embed timepicker in query bar

* flesh out date picker in query bar for maps app

* wire up refresh config

* fix bug with way update function called by watcher

* get maps application functional tests working with new timepicker

* remove some changes outside of scoped work

* clean up typescript lint problems

* fix query_bar I18n lint error

* update query_bar jest test

* grab some parts missing from one cherry-pick

* pass dateRange to updateQueryAndDispatch when app state changes

* use timefilter disable methods to hide timepicker from top naav

* get selected refresh unit

* use EuiSuperUpdate button

* Fix responsive sizing of datepicker (#29)

* set isDisabled on EuiSuperUpdateButton

* review feedback

* remove ts-ignore comment from defaultProps, fix query_bar snapshot

* make new props optional

* fighting with ts linter

* do not include dateRangeFrom and dateRangeTo in isDirty when shoDateRange is not true

* pull initial query from UI settings
@cchaos cchaos deleted the nreese/maps_time_picker/design branch March 7, 2019 17:16
nreese pushed a commit that referenced this pull request Apr 24, 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.

2 participants