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

[ML] Fixes positions of calendar arrow buttons in start datafeed modal #92625

Merged

Conversation

peteharverson
Copy link
Contributor

Summary

Fixes the positioning of the date picker back and forward arrow buttons in the Anomaly detection start datafeed modal. Previously depending on the number of weeks in the calendar month, the modal height would change, which meant the position of the month forward / backward arrow buttons would change.

Before:
start_datafeed_modal_before

After:
start_datafeed_modal_after

Checklist

Fixes #81989

@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

Copy link
Contributor

@alvarezmelissa87 alvarezmelissa87 left a comment

Choose a reason for hiding this comment

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

LGTM ⚡

@peteharverson peteharverson self-assigned this Feb 24, 2021
@peteharverson
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 6.4MB 6.4MB +80.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @peteharverson

Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

LGTM

@peteharverson peteharverson merged commit 54b4d9a into elastic:master Feb 25, 2021
@peteharverson peteharverson deleted the ml-start-datafeed-modal-fix branch February 25, 2021 09:23
peteharverson added a commit to peteharverson/kibana that referenced this pull request Feb 25, 2021
elastic#92625)

* [ML] Fixes positions of calendar arrow buttons in start datafeed modal

* [ML] Move height styling to parent container.

Co-authored-by: Kibana Machine <[email protected]>
peteharverson added a commit to peteharverson/kibana that referenced this pull request Feb 25, 2021
elastic#92625)

* [ML] Fixes positions of calendar arrow buttons in start datafeed modal

* [ML] Move height styling to parent container.

Co-authored-by: Kibana Machine <[email protected]>
peteharverson added a commit that referenced this pull request Feb 25, 2021
#92625) (#92772)

* [ML] Fixes positions of calendar arrow buttons in start datafeed modal

* [ML] Move height styling to parent container.

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
peteharverson added a commit that referenced this pull request Feb 25, 2021
#92625) (#92771)

* [ML] Fixes positions of calendar arrow buttons in start datafeed modal

* [ML] Move height styling to parent container.

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Feb 25, 2021
* master: (38 commits)
  Fixes Cypress flake by adding pipe, click, and should (elastic#92762)
  [Discover] Fix filtering selected sidebar fields (elastic#91828)
  [ML] Fixes positions of calendar arrow buttons in start datafeed modal (elastic#92625)
  [dev/build_ts_refs] check that commit in outDirs matches mergeBase (elastic#92513)
  add dep on `@kbn/config` so it is built first
  [Expressions] [Lens] Add id and copyMetaFrom arg to mapColumn fn + add configurable onError argument to math fn (elastic#90481)
  [Lens] Fix Workspace hidden when using Safari (elastic#92616)
  [Lens] Fixes vertical alignment validation messages (elastic#91878)
  forbid x-elastic-product-origin header in elasticsearch configuration (elastic#92359)
  [Security Solution][Detections] Set default indicator path to reduce friction with new filebeat modules (elastic#92081)
  [ILM][Accessibility] Added A11y test for ILM new policy form. (elastic#92570)
  [Security Solution][Exceptions] - Fixes exceptions builder UI where invalid values can cause overwrites of other values (elastic#90634)
  Automatically generated Api documentation (elastic#86232)
  Increase index pattern select limit to 1000 (elastic#92093)
  [core.logging] Add RewriteAppender for filtering LogMeta. (elastic#91492)
  [Security Solution][Detection Rules] Update prebuilt rule threats to match schema (elastic#92281)
  [Security Solutions][Detection Engine] Fixes bug with not being able to duplicate indicator matches (elastic#92565)
  [Dashboard] Export appropriate references from byValue panels (elastic#91567)
  [Upgrade Assistant] Align code between branches (elastic#91862)
  [Security Solution][Case] Fix alerts push (elastic#91638)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ML] start datafeed UI calendar size toggles depending on month
5 participants