PCF-391 The keyboard handling for the revision dropdown should be done differently#926
Open
esanuandra wants to merge 4 commits into
Open
PCF-391 The keyboard handling for the revision dropdown should be done differently#926esanuandra wants to merge 4 commits into
esanuandra wants to merge 4 commits into
Conversation
✅ Deploy Preview for mozilla-perfcompare ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
3 times, most recently
from
90de22d to
d5beba4
Compare
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
d5beba4 to
d19522f
Compare
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
6 times, most recently
from
fdde6ef to
54ddcf0
Compare
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
5 times, most recently
from
b6922d9 to
031a978
Compare
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
5 times, most recently
from
13e39ea to
7c2d3b9
Compare
Contributor
Author
|
I need some help with the Snackbar tests, fetchRevisionByID and fetchRevisionsBySearch. |
esanuandra
added
Ready For Review
and removed
🚧 WIP 🚧
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
7c2d3b9 to
208f150
Compare
gmierz
reviewed
Nov 27, 2025
gmierz
reviewed
Nov 27, 2025
gmierz
reviewed
Nov 28, 2025
gmierz
reviewed
Nov 28, 2025
gmierz
reviewed
Nov 28, 2025
gmierz
reviewed
Nov 28, 2025
kala-moz
requested changes
Dec 1, 2025
Contributor
kala-moz
left a comment
kala-moz
left a comment
There was a problem hiding this comment.
Added my comments and suggestions to the PR.
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
208f150 to
0458490
Compare
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
7785cde to
c80baad
Compare
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
c80baad to
c2bde8b
Compare
kala-moz
requested changes
Apr 9, 2026
Contributor
kala-moz
left a comment
kala-moz
left a comment
There was a problem hiding this comment.
@esanuandra I left some suggestions on your tests to help you get them green.
Contributor
|
@esanuandra Looks like those three tests passed! Just need to update the snapshots. |
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
249241b to
fc7ef71
Compare
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
2 times, most recently
from
fcff5f2 to
406cedf
Compare
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
dd8fff3 to
dd9662d
Compare
Contributor
|
@esanuandra rebase again and I'll look through the results of that. |
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
dd9662d to
cd2a99a
Compare
Contributor
|
@esanuandra Have you already tried |
Contributor
|
Or let me know what you've already done to try and fix the snapshots and we can work from there. |
fix tests fix multiple select revision checkbox Fix Snackbar, fetchRevisionBySearch and fetchRevisionbyID test Update SearchInputAndResult to theme.palette Rename AutocompleteInput and AutocompleteOption Remove SearchResultsList and SearchResultsListItem Fix input field for new remains populated after a revision is selected
esanuandra
force-pushed
the
keyboard-handling-for-revision-dropdown
branch
from
448b8df to
ab80166
Compare
Contributor
Author
|
hi @kala-moz I fixed the snapshots🥳 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #926 +/- ##
==========================================
- Coverage 96.39% 96.19% -0.21%
==========================================
Files 113 113
Lines 3330 3311 -19
Branches 761 760 -1
==========================================
- Hits 3210 3185 -25
- Misses 119 125 +6
Partials 1 1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Jira ticket - The keyboard handling for the revision dropdown should be done differently
This PR should make the navigation using the keyboard possible for searching revisions in Search component. Part of the implementation was made with the help of Cursor.