-
Notifications
You must be signed in to change notification settings - Fork 1.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
chore: Add optional shouldFocusOnHover prop to aria combobox #7051
Open
p1pah
wants to merge
359
commits into
adobe:main
Choose a base branch
from
p1pah:p1pah-shouldfocusonhover
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains 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
* testing theme change and webkit-highlight-color change * Fix iOS tap highlight * moving webkit highlight to theme * forgot to remove extra tsignore
* css rename to styles * fix comment
* fix SearchFieldProps children type * remove isEntering and isExiting from TooltipProps * omit more props from Tooltip * remove open props from Tooltip Co-authored-by: Reid Barber <[email protected]>
* setup storybook docs * add intro page * Remove Picker autodocs * add syntax highlighting to code snippet * add missing autodocs * Remove intro docs for now * remove extra args from Dialog * clean up source formatting Co-authored-by: Reid Barber <[email protected]> Co-authored-by: Rob Snow <[email protected]>
Co-authored-by: Devon Govett <[email protected]>
* starting a rollup add to test the built files * rainbow and macros installed and working Co-authored-by: Kyle Taborski <[email protected]>
* Add vite example * Add svgs and fix config
Co-authored-by: Kyle Taborski <[email protected]>
Co-authored-by: Kyle Taborski <[email protected]>
…e for invalid validation (adobe#115) * revert Field change when invalid * Update Modal and Popover so corner with scrollbar is square
* update css prop in InlineAlert * add webpack 5 example * remove Provider * use render from react-dom * add gitignore * cleanup package.json * fix React import * add icon to button Co-authored-by: Reid Barber <[email protected]>
* add menu and actionmenu * fix stories for actionbutton and checkboxgroup * fix actionbutton resizing story * Remove styles prop from code output * remove handlers from code output * add // code: null to config * fix StaticColorDecorator * cleanup * improve dialog example * add global padding to stories * remove css file * remove role from DialogProps type * improve ActionButton story * use single quotes for jsdoc @default * cleanup Dialog stories and code output Co-authored-by: Reid Barber <[email protected]>
* add storybook logo * Use italics for tagline Co-authored-by: Reid Barber <[email protected]>
* fix show code button z-index * fix to use isolation: isolate instead Co-authored-by: Reid Barber <[email protected]>
Co-authored-by: Yihui Liao <[email protected]>
Co-authored-by: Kyle Taborski <[email protected]>
* Fix chromatic publish to storybook this prevents the Intro page from being sent to Chromatic, looks like it causes issues for them * Update change so it is isolated to chromatic run
* Returning the RSP Accordion Chromatic-FC story to it previous behavior * seperating s1 and s2 accordion stories and bringing s1 stories to previous parity --------- Co-authored-by: Yihui Liao <[email protected]>
* rough progress in refactoring api and testing timers * update RAC select tests to use the select util * fix edge cases and add flows for different interaction patterns * adding sections and removing valueElement, wrapping up final conversions for Picker tests only modified some of the Picker tests to leverage the utils where I deemed appropriate. Mainly where selection/opening of the Picker wasnt the main focus of the test or if the test itself wasnt checking stuff about the item nodes themselves. * add initial table util scaffolding mostly from old PR, adapted to fit the new api * testing against the RAC tests/React 16/17 had to add an increased test timeout for React 16 tests * more testing of long press, add findCell/findRow * test all the different interaction types the keyboard sort change util function isnt working properly, investigating. The focus isnt being restored to the right place * use click for now for some keyboard operations focus management and certain elements arent working properly, see comments * fix keyboard interaction for sort column util * figure out the proper timeout for realTimers * lint * fix docs and build point * forgot comma in rebase * Update packages/@react-spectrum/table/test/Table.test.js * go with factory instead of having the testers created in constructor this avoids possible issues with the same tester being used across multiple tests and getting in a weird state in parallel runs * fix select tests * replace jest expects with generic assertion this makes it so the test utils arent specific to jest * initial menu tester util * Fixing strict and using MenuTester in tests * fix failing tests * fix react 17 test * add combobox util * test combobox util * get rid of setText and add some additional utilities setText is a bit trouble some because we arent sure if the user wants to simulate a realistic typing flow and/or needs to perform a delete operation to clear the field so will leave that operation to them. * inital gridlist tester * refactor gridlist util * refactor combobox and menu * refactor select for consistency * refactor table for consistency * add selected row getter for looking at tests in quarry * have user provide callback for advancing timers this means we can handle real timers vs fake timers without needing to rely on jest or handling detection ourselves * update createTester so we get proper return types when creating a tester * fix lint and react 19 test * fix tsstrict and tests * review comments * make useLongPress accept mouse or touch * refactor so setting root element is done in contructor * refactor combobox tests so they dont destructure * refactor the rest of the utils to avoid destruct * support passing in interaction type directly when calling interaction * update api as per feedback and remove unnecessary warning * cleanup and lower test timeout to minimum to work in react 16 * fix strict and revert jest timing build failed with 15000 timeout which passed locally... --------- Co-authored-by: Robert Snow <[email protected]>
* Fix pending animation docs safari * Extract code for progress circle into a collapsed component --------- Co-authored-by: Devon Govett <[email protected]>
* Fix Dialog and TagGroup using the wrong ImageContext * Fix text slot in LinkButton * lint * Fix when a card collection has no previews --------- Co-authored-by: Rob Snow <[email protected]>
* Fix extract starters * add branch filters back --------- Co-authored-by: Reid Barber <[email protected]>
* Add Spectrum 2 docs to storybook * progress on table styles * fix highlight checkbox rendering and add rounded corners to body * fix checkbox styles, update to columns, font family * refactor cell focus ring and add row focus ring row focus ring needs row render function first * wrapping up disabled row styles * empty/loading state brainstorming * Replace empty state with loading spinner * add basic sorting * attempt to add showDivider this will need changes to RAC table as well.... * Fix select all checkbox click * add placeholder for DnD and use new icon for sortable column * update HCM colors * fix DnD story, removed need for sortDescriptor on context, fix drag button focus visible also tested moving the sort descriptor in front of the column text andwithout reserving room * clean up * switch to box shadow on row for grey row underline and selection focus ring * update colors/sizes from table specific tokens * updating lightdark so it works with opacity values * add row visible focus style and fix empty columnheader focus style for drag/checkbox columns * initial setup for resizing support * progress on resizer handle styling and menu icons * fix more resize bugs and some clean up * nubbin and more DnD styling * fix drag indicator so it respects density * got verdaccio working, updating row background style and confirmed resizers are skipped * updating loading more to use rowLoader * use isTableEmpty renderProps from local RAC * Add virtualizer and begin fixing styles * add scale, update to column widths, some updates to reflect new token values * add async loading stories and updates to use background image for table body outline * fix broken empty state story and more debugging with empty state centering * move outline to whole table and fix borders and padding for columns * fix progress circle centering, rows showing up behind column, and scuffed way to curve the last row border * Support showing the resizer when table header is hovered, fix to table outline, get rid of curved row * support sticky column cells * fix sticky positioning for the body loader and row loader * investigations for sticky column/row selection border, fix for sort icon placement * making sticky columns cells properly cover cells scrolled beneith them * fix vertical centering * lint * add full height column resize indicator * hack for show dividers * add text alignment * refactor to pass divider and align to cell directly this avoids us having to expose column props on the cell render props which are tricky to type properly (would need to be a generic). Also fixes some overlapping of the focus rings and the divder and resize indicators * saving some investigations * get rid of blue selection outlines and add column header background * add table focus indicator that is sticky * HCM and nubbin appearance when triggered via menu * fixes to icon sizing, illustrations, props in story, font default, and RTL * adding focus style for empty style and translations * fix typeahead and load more progress circle positioning * only expose sort options when avaialble and align resizing column with colum align * some clean up and notes from investigations * remove DnD for now * add more complex story * fix lint * fixing infinite load issue I was applying the scroll ref onto the wrong element causing useLoadMore to trigger indefinitely because the scroll height and the client height of the provided ref were always equal * increase body focus width, all resizer appears when in resizing mode, fix for premature resizing exit * forgot to remove extra console.log * fix strict error * address review comments and update header background * make links actually clickable and fix focus ring on link * fix scroll padding so rows and cells scroll fully into view * fix dark mode/HCM colors, column min width and tap highlight * review comments * fix width/height when determined by flex * fix lint * fix checkbox appearing above scrollbar and ipad tap highlight issues * review comments * making loader row taller * fix overflow wrap decreasing row height and add menu min width for small viewports * clean up context to try to help performance * update scrollPadding to expected value chrome behavior is strange because of chrome bug, see https://issues.chromium.org/issues/365913982 which I filed * fix slow performance when many rows are provided and isQuiet/densitiy is flipped * add discrete border styles to theme * Revert "fix slow performance when many rows are provided and isQuiet/densitiy is flipped" This reverts commit feb0ada. * fixing lint and tests * update resizer icon * update HCM for resizer and add browser bug info * disable onAction on story by default and add label for single select empty cell * yarn lock update * fix lint * address initial review comments * make popovers inert when they are transitioning out so table resizing doesnt get canceled if the user clicks the resize option and hovers another item while the popover is transitioning out it can cause resizing to be canceled. This prevents that case from happening * add useScale and use it in table instead * ignore warnings in test for invalid attribute and remove items length check warning is a compat one for React 19, items check is uneeded due to user error * forgot the forward ref and add descriptions * Revert "make popovers inert when they are transitioning out so table resizing doesnt get canceled" This reverts commit 9cfb896. * Revert "ignore warnings in test for invalid attribute" This reverts commit 7006a76. # Conflicts: # packages/react-aria-components/test/ComboBox.test.js # packages/react-aria-components/test/Select.test.js * Use pointerEvents: none in popover during exit animation instead of inert * Don't expose ResizableTableContainerContext publicly It has a bunch of internal stuff on it. We don't need the exact height for the resizer, we just need a large enough value that it goes past the end of the table. 100vh should work. * Use VisuallyHidden from react-aria monopackage * Revert border style longhands * lint --------- Co-authored-by: danilu <[email protected]> Co-authored-by: Daniel Lu <[email protected]>
* fix undefined id in Card * remove useId
* fix selection with falsy keys * fix falsy check if Selection class * add unit tests * revert additional change
* pass through disabledKeys to ActionGroup * fix linting issues --------- Co-authored-by: Grant Russell <[email protected]>
* add s2 components to Parcel app * add components to Webpack example * add to verdaccio build * build verdaccio on branch * try adding yarn config * fix yarn * add build script to parcel all * typo in path * enable comment on PR * comment on PR * try removing typekits * update webpack render to createRoot * try removing TagGroup * remove TagGroup from example apps * add branch filters back * update example app S2 versions to be "latest" * add TagGroup back * remove branch filters * Revert "remove branch filters" This reverts commit b7b54fe. * Fix style publishing * turn on verdaccio * Revert "turn on verdaccio" This reverts commit 3be5f9c. --------- Co-authored-by: GitHub <[email protected]>
* Fix ComboBox and Picker getting stuck in down state * Fix table types and export from package
* Remove extraneous hover props from s2 * Fix ts * use the correct prop name
* S2 Table optimizations * fix merge
* Add some missing docs descriptions * Rename strange interface in props * Fix sizes of pending button spinner * Add docs for --tree-item-level custom property in Tree * fix ts * Require children for disclosures * Make it so Accordion isn't labellable * fix button typescript * narrow test utils exports and future proof * Remove isPending from ActionButton --------- Co-authored-by: Robert Snow <[email protected]> Co-authored-by: GitHub <[email protected]>
* disable snapshot for regular stories, remove s2 from title * add chromatic stories from missing components * fix lint * add more chromatic stories for missing components and remove s2 prefix * remove disable snapshot, update chromatic-fc * Some docs improvements --------- Co-authored-by: Devon Govett <[email protected]>
* S2 chromatic table * fix cardview chromatic stories
* Export pressScale and focusRing utilities from S2 * Also export StyleString type
* Rename SegmentedControl props to match selection APIs * fix chromatic stories * ts strict * Add AriaLabelingProps * Pass value and defaultValue to RadioGroup * fix merge conflicts
* add disclosure tests * lint/cleanup * Pass isExpanded to useDisclosure as well --------- Co-authored-by: Robert Snow <[email protected]> Co-authored-by: Devon Govett <[email protected]>
Any chance this could come in with next release :D? |
* fix macro import in codemod output * remove recast patch
* Updated @react-spectrum/picker translations to match latest English 'Select...' * Latest @react-spectrum/s2 translations, partially complete * remove .localizationManfiest as not needed for github * Updated translations from PR review
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No issue to close, but solves this discussion! #7045
✅ Pull Request Checklist:
📝 Test Instructions:
🧢 Your Project: