-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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] Review UI text of population, advanced, categ, rare, and geo wizards #149407
Conversation
…-ref HEAD~1..HEAD --fix'
Pinging @elastic/ml-ui (:ml) |
@jgowdyelastic Could you please help me to remove the title "Enable per-partition categorization" above the corresponding switch in the Advanced wizard when the time permits? I tried the usual methods but they didn't work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
...n/jobs/new_job/pages/components/pick_fields_step/components/population_field/description.tsx
Outdated
Show resolved
Hide resolved
Not touched by the current PR, but here are some additional comments about text seen in these screenshots:
|
.../pick_fields_step/components/categorization_partition_field/categorization_per_partition.tsx
Show resolved
Hide resolved
.../pages/components/pick_fields_step/components/categorization_partition_field/description.tsx
Outdated
Show resolved
Hide resolved
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Latest edits LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Text LGTM!
…ards (elastic#149407) ## Summary This PR adjusts the anomaly detection UI text to be shorter, cleaner, and more in line with the EUI guidelines. The field names above the field selectors are removed.
…ards (elastic#149407) ## Summary This PR adjusts the anomaly detection UI text to be shorter, cleaner, and more in line with the EUI guidelines. The field names above the field selectors are removed.
Summary
Related issue: https://github.com/elastic/ml-team/issues/845
This PR adjusts the anomaly detection UI text to be shorter, cleaner, and more in line with the EUI guidelines. The field names above the field selectors are removed.
Population
Before
After
Advanced
Before
After
Categorization
Before
After
Rare
Before
After
Geo
After