Skip to content

chore(preprod): Improve no results content on comparison page#102438

Merged
rbro112 merged 1 commit into
masterfrom
ryan/improve_no_results_content_on_comparison_page
Oct 30, 2025
Merged

chore(preprod): Improve no results content on comparison page#102438
rbro112 merged 1 commit into
masterfrom
ryan/improve_no_results_content_on_comparison_page

Conversation

@rbro112

@rbro112 rbro112 commented Oct 30, 2025

Copy link
Copy Markdown
Member

Worked with Dan on the copy here:

  • Updates filter to be more concise
  • Adds richer empty state with button to turn off the hide small changes filter
  • Lock the Items changed: X value to be the original items, indicating that things did change in all cases

Resolves EME-485

Before
Screenshot 2025-10-30 at 1 56 55 PM

After:
Screenshot 2025-10-30 at 2 00 58 PM

@rbro112 rbro112 requested a review from a team as a code owner October 30, 2025 21:01

rbro112 commented Oct 30, 2025

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 30, 2025
@rbro112 rbro112 changed the title Improve no results content on comparison page chore(preprod): Improve no results content on comparison page Oct 30, 2025
@linear

linear Bot commented Oct 30, 2025

Copy link
Copy Markdown

<SimpleTable.Empty>
<Stack gap="lg" align="center" justify="center">
<Text size="lg" variant="muted" bold>
{t('No results found')}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: wdyt of "No results found matching your filters"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Dan pushed for "No results found" (originally had No results found < 500B) since it was concise, so I'd say we just leave as-is

@codecov

codecov Bot commented Oct 30, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #102438   +/-   ##
========================================
  Coverage   80.90%    80.90%           
========================================
  Files        8825      8825           
  Lines      389987    389986    -1     
  Branches    24799     24799           
========================================
  Hits       315523    315523           
+ Misses      74113     74112    -1     
  Partials      351       351           

@rbro112 rbro112 force-pushed the ryan/improve_no_results_content_on_comparison_page branch from 9b010c9 to 7912d5e Compare October 30, 2025 21:18
@rbro112 rbro112 merged commit 7553e1d into master Oct 30, 2025
48 checks passed
@rbro112 rbro112 deleted the ryan/improve_no_results_content_on_comparison_page branch October 30, 2025 21:32
isabellaenriquez pushed a commit that referenced this pull request Oct 31, 2025
Worked with Dan on the copy here:
- Updates filter to be more concise
- Adds richer empty state with button to turn off the hide small changes
filter
- Lock the Items changed: X value to be the original items, indicating
that things did change in all cases

Resolves EME-485

Before
<img width="1500" height="470" alt="Screenshot 2025-10-30 at 1 56 55 PM"
src="https://github.com/user-attachments/assets/7dc8b731-0383-436f-97c6-d4a35a960d58"
/>


After:
<img width="1486" height="433" alt="Screenshot 2025-10-30 at 2 00 58 PM"
src="https://github.com/user-attachments/assets/c3a645ee-722a-43f0-b952-8645856340b3"
/>
shashjar pushed a commit that referenced this pull request Nov 4, 2025
Worked with Dan on the copy here:
- Updates filter to be more concise
- Adds richer empty state with button to turn off the hide small changes
filter
- Lock the Items changed: X value to be the original items, indicating
that things did change in all cases

Resolves EME-485

Before
<img width="1500" height="470" alt="Screenshot 2025-10-30 at 1 56 55 PM"
src="https://github.com/user-attachments/assets/7dc8b731-0383-436f-97c6-d4a35a960d58"
/>


After:
<img width="1486" height="433" alt="Screenshot 2025-10-30 at 2 00 58 PM"
src="https://github.com/user-attachments/assets/c3a645ee-722a-43f0-b952-8645856340b3"
/>
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
Worked with Dan on the copy here:
- Updates filter to be more concise
- Adds richer empty state with button to turn off the hide small changes
filter
- Lock the Items changed: X value to be the original items, indicating
that things did change in all cases

Resolves EME-485

Before
<img width="1500" height="470" alt="Screenshot 2025-10-30 at 1 56 55 PM"
src="https://github.com/user-attachments/assets/7dc8b731-0383-436f-97c6-d4a35a960d58"
/>


After:
<img width="1486" height="433" alt="Screenshot 2025-10-30 at 2 00 58 PM"
src="https://github.com/user-attachments/assets/c3a645ee-722a-43f0-b952-8645856340b3"
/>
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants