Skip to content

[Papercut] Change in sortBy dropdown component option name wording#206464

Merged
paulinashakirova merged 6 commits intoelastic:mainfrom
paulinashakirova:listing-page-sortBy-component
Mar 5, 2025
Merged

[Papercut] Change in sortBy dropdown component option name wording#206464
paulinashakirova merged 6 commits intoelastic:mainfrom
paulinashakirova:listing-page-sortBy-component

Conversation

@paulinashakirova
Copy link
Contributor

@paulinashakirova paulinashakirova commented Jan 13, 2025

Summary

It's a tiny papercut to improve the UX.
(Issue)

Screenshot 2025-01-13 at 16 21 58 Screenshot 2025-02-21 at 15 07 12
@paulinashakirova paulinashakirova added release_note:fix Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// backport:prev-minor papercut Small "burr" in the product that we should fix. labels Jan 13, 2025
@paulinashakirova paulinashakirova self-assigned this Jan 13, 2025
@paulinashakirova paulinashakirova requested a review from a team as a code owner January 13, 2025 15:23
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

'contentManagement.tableList.listing.tableSortSelect.updatedAtAscLabel',
{
defaultMessage: 'Least recently updated',
defaultMessage: 'Old to New',
Copy link
Contributor

Choose a reason for hiding this comment

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

I saw the original ask and the wording:

Can we standardize this closer to the default datagrid sort, "Old-New, New-Old" or something like this.

And I thought this needs more discussion because "Old to New" is not technically very correct label.

@timductive, Wanted to confirm if we should proceed with this change

@paulinashakirova paulinashakirova marked this pull request as draft January 13, 2025 22:06
@paulinashakirova paulinashakirova marked this pull request as ready for review February 21, 2025 16:27
@paulinashakirova paulinashakirova marked this pull request as draft February 21, 2025 18:34
@paulinashakirova paulinashakirova marked this pull request as ready for review March 5, 2025 09:44
@Dosant Dosant added backport:skip This PR does not require backporting and removed backport:prev-minor labels Mar 5, 2025
Copy link
Contributor

@Dosant Dosant left a comment

Choose a reason for hiding this comment

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

Thanks, lgtm!
I changed backport label to skip, since we no longer want to backport to 9.0 as it is bugs only now

@elasticmachine
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
dashboard 532.5KB 532.5KB -28.0B
eventAnnotationListing 207.7KB 207.7KB -28.0B
filesManagement 106.3KB 106.3KB -28.0B
graph 394.5KB 394.5KB -28.0B
maps 3.0MB 3.0MB -28.0B
visualizations 342.2KB 342.2KB -28.0B
total -168.0B

History

cc @paulinashakirova

@paulinashakirova paulinashakirova merged commit 5135acf into elastic:main Mar 5, 2025
12 checks passed
@paulinashakirova paulinashakirova deleted the listing-page-sortBy-component branch March 5, 2025 12:18
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Mar 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting papercut Small "burr" in the product that we should fix. release_note:fix Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// v9.1.0

4 participants