-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] main from facebook:main #28
Conversation
…ick (#7213) Co-authored-by: Bob Ippolito <[email protected]>
…7237) Co-authored-by: Bob Ippolito <[email protected]>
🚨 gitStream Monthly Automation Limit Reached 🚨 Your organization has exceeded the number of pull requests allowed for automation with gitStream. To continue automating your PR workflows and unlock additional features, please contact LinearB. |
Reviewer's Guide by SourceryThis pull request focuses on improving table cell selection behavior and standardizing the attribute setting for the typeahead menu. It introduces a new command for handling table clicks and ensures consistent behavior across browsers when selecting table cells on triple click. Additionally, it encapsulates the logic for setting attributes on the typeahead menu container, ensuring consistency and removing the 'id' attribute when the menu is unmounted. Sequence diagram for table cell selection on triple clicksequenceDiagram
participant User
participant TableElement
participant Editor
User->>TableElement: Triple click on a cell
TableElement->>Editor: CLICK_COMMAND event
Editor->>Editor: $tableClickCommand(event)
Editor->>Editor: $getNearestNodeFromDOMNode(event.target)
Editor->>Editor: $findMatchingParent(startNode)
Editor->>Editor: blockNode.select(0)
Editor-->>TableElement: Updates selection
Sequence diagram for LexicalMenu mounting and unmountingsequenceDiagram
participant ParentComponent
participant LexicalMenu
participant ContainerDiv
ParentComponent->>LexicalMenu: Mounts
LexicalMenu->>ContainerDiv: Creates or reuses ContainerDiv
LexicalMenu->>ContainerDiv: setContainerDivAttributes(ContainerDiv, className)
LexicalMenu->>ParentComponent: Appends ContainerDiv to parent
ParentComponent->>LexicalMenu: Unmounts
LexicalMenu->>ContainerDiv: Removes ContainerDiv from parent
LexicalMenu->>ContainerDiv: removeAttribute('id')
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )
Summary by Sourcery
This pull request improves table cell selection and enhances the accessibility of the typeahead menu. It ensures consistent cell selection behavior across browsers when triple-clicking and sets the appropriate ARIA attributes for the typeahead menu.
Bug Fixes:
Enhancements:
Tests: