Skip to content

feat: add safe hover polygon handling #625

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.

Sign up for GitHub

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

Open
li-jia-nan wants to merge 12 commits into
base: master
Choose a base branch
from
Open

feat: add safe hover polygon handling #625

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
67829e5
chore: update deps
li-jia-nan Jun 7, 2026
4a69bd2
chore: update deps
li-jia-nan Jun 7, 2026
ba5359b
update
li-jia-nan Jun 7, 2026
6ba40ac
update
li-jia-nan Jun 8, 2026
098bb3c
Potential fix for pull request finding 'CodeQL / Useless comparison t…
li-jia-nan Jun 8, 2026
e356ae1
update
li-jia-nan Jun 8, 2026
63e2743
Potential fix for pull request finding 'CodeQL / Useless comparison t…
li-jia-nan Jun 8, 2026
a05aaaf
update
li-jia-nan Jun 8, 2026
b251836
update
li-jia-nan Jun 8, 2026
fac2f2d
update
li-jia-nan Jun 8, 2026
8e48450
update
li-jia-nan Jun 8, 2026
bc5448b
Merge remote-tracking branch 'origin/master' into feat-safe-hover-pol…
zombieJ Jul 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/react-component-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,4 @@ jobs:
test:
uses: react-component/rc-test/.github/workflows/test-utoo.yml@main
secrets:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
81 changes: 40 additions & 41 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@

<p align="center">English | <a href="./README.zh-CN.md">简体中文</a></p>


## Highlights

- Built for React and maintained by the rc-component team.
Expand Down Expand Up @@ -64,47 +63,47 @@ Then open `http://localhost:8000`.

### Trigger

| Prop | Description | Type | Default |
| -------------------------- | ------------------------------------------------------------------------- | ----------------------------------------- | ------------------ |
| action | Actions that control popup visibility. | `ActionType` \| `ActionType[]` | `hover` |
| afterOpenChange | Called after popup visibility changes. | `(visible) => void` | - |
| afterPopupVisibleChange | Deprecated. Use `afterOpenChange` instead. | `(visible) => void` | - |
| alignPoint | Align popup to mouse position for click, hover, and context menu actions. | `boolean` | `false` |
| arrow | Render popup arrow. | `boolean` \| `ArrowType` | - |
| autoDestroy | Destroy popup DOM when it is hidden. | `boolean` | `false` |
| blurDelay | Delay before hiding on blur, in seconds. | `number` | - |
| builtinPlacements | Named placement presets. | `BuildInPlacements` | `{}` |
| defaultPopupVisible | Initial uncontrolled visibility. | `boolean` | `false` |
| Prop | Description | Type | Default |
| -------------------------- | ------------------------------------------------------------------------------- | ----------------------------------------- | ------------------ |
| action | Actions that control popup visibility. | `ActionType` \| `ActionType[]` | `hover` |
| afterOpenChange | Called after popup visibility changes. | `(visible) => void` | - |
| afterPopupVisibleChange | Deprecated. Use `afterOpenChange` instead. | `(visible) => void` | - |
| alignPoint | Align popup to mouse position for click, hover, and context menu actions. | `boolean` | `false` |
| arrow | Render popup arrow. | `boolean` \| `ArrowType` | - |
| autoDestroy | Destroy popup DOM when it is hidden. | `boolean` | `false` |
| blurDelay | Delay before hiding on blur, in seconds. | `number` | - |
| builtinPlacements | Named placement presets. | `BuildInPlacements` | `{}` |
| defaultPopupVisible | Initial uncontrolled visibility. | `boolean` | `false` |
| disabled | Temporarily suppress popup visibility without resetting the current open state. | `boolean` | `false` |
| focusDelay | Delay before showing on focus, in seconds. | `number` | - |
| forceRender | Render popup before it is first shown. | `boolean` | `false` |
| fresh | Keep popup content updated while closed. | `boolean` | - |
| getPopupClassNameFromAlign | Return a popup class name from the current alignment. | `(align) => string` | - |
| getPopupContainer | Return the element that should contain the popup. | `(node) => HTMLElement` | - |
| hideAction | Actions that hide popup visibility. | `ActionType[]` | - |
| mask | Render a mask behind the popup. | `boolean` | `false` |
| maskClosable | Close when clicking the mask. | `boolean` | `true` |
| maskMotion | Motion config for the mask. | `CSSMotionProps` | - |
| mouseEnterDelay | Delay before showing on mouse enter, in seconds. | `number` | - |
| mouseLeaveDelay | Delay before hiding on mouse leave, in seconds. | `number` | `0.1` |
| popup | Popup content. | `ReactNode` \| `() => ReactNode` | - |
| popupAlign | Alignment config compatible with dom-align. | `AlignType` | - |
| popupClassName | Class name added to popup. | `string` | - |
| popupMotion | Motion config for the popup. | `CSSMotionProps` | - |
| popupPlacement | Placement key from `builtinPlacements`. | `string` | - |
| popupStyle | Inline popup style. | `React.CSSProperties` | - |
| popupVisible | Controlled popup visibility. | `boolean` | - |
| prefixCls | Popup class name prefix. | `string` | `rc-trigger-popup` |
| showAction | Actions that show popup visibility. | `ActionType[]` | - |
| stretch | Stretch popup width and/or height from target. | `string` | - |
| unique | Share popup container through `UniqueProvider`. | `boolean` | - |
| uniqueContainerClassName | Class name passed to `UniqueProvider` container. | `string` | - |
| uniqueContainerStyle | Style passed to `UniqueProvider` container. | `React.CSSProperties` | - |
| zIndex | Popup z-index. | `number` | - |
| onOpenChange | Called when visibility changes. | `(visible) => void` | - |
| onPopupAlign | Called when the popup is aligned. | `(element, align) => void` | - |
| onPopupClick | Called when popup is clicked. | `React.MouseEventHandler<HTMLDivElement>` | - |
| onPopupVisibleChange | Deprecated. Use `onOpenChange` instead. | `(visible) => void` | - |
| focusDelay | Delay before showing on focus, in seconds. | `number` | - |
| forceRender | Render popup before it is first shown. | `boolean` | `false` |
| fresh | Keep popup content updated while closed. | `boolean` | - |
| getPopupClassNameFromAlign | Return a popup class name from the current alignment. | `(align) => string` | - |
| getPopupContainer | Return the element that should contain the popup. | `(node) => HTMLElement` | - |
| hideAction | Actions that hide popup visibility. | `ActionType[]` | - |
| mask | Render a mask behind the popup. | `boolean` | `false` |
| maskClosable | Close when clicking the mask. | `boolean` | `true` |
| maskMotion | Motion config for the mask. | `CSSMotionProps` | - |
| mouseEnterDelay | Delay before showing on mouse enter, in seconds. | `number` | - |
| mouseLeaveDelay | Delay before hiding on mouse leave, in seconds. | `number` | `0.1` |
| popup | Popup content. | `ReactNode` \| `() => ReactNode` | - |
| popupAlign | Alignment config compatible with dom-align. | `AlignType` | - |
| popupClassName | Class name added to popup. | `string` | - |
| popupMotion | Motion config for the popup. | `CSSMotionProps` | - |
| popupPlacement | Placement key from `builtinPlacements`. | `string` | - |
| popupStyle | Inline popup style. | `React.CSSProperties` | - |
| popupVisible | Controlled popup visibility. | `boolean` | - |
| prefixCls | Popup class name prefix. | `string` | `rc-trigger-popup` |
| showAction | Actions that show popup visibility. | `ActionType[]` | - |
| stretch | Stretch popup width and/or height from target. | `string` | - |
| unique | Share popup container through `UniqueProvider`. | `boolean` | - |
| uniqueContainerClassName | Class name passed to `UniqueProvider` container. | `string` | - |
| uniqueContainerStyle | Style passed to `UniqueProvider` container. | `React.CSSProperties` | - |
| zIndex | Popup z-index. | `number` | - |
| onOpenChange | Called when visibility changes. | `(visible) => void` | - |
| onPopupAlign | Called when the popup is aligned. | `(element, align) => void` | - |
| onPopupClick | Called when popup is clicked. | `React.MouseEventHandler<HTMLDivElement>` | - |
| onPopupVisibleChange | Deprecated. Use `onOpenChange` instead. | `(visible) => void` | - |

## Development

Expand Down
Loading
Loading