Skip to content
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

Popup does not change display speed when delay prop is set #3604

Closed
GrimzEcho opened this issue May 10, 2019 · 2 comments
Closed

Popup does not change display speed when delay prop is set #3604

GrimzEcho opened this issue May 10, 2019 · 2 comments
Labels

Comments

@GrimzEcho
Copy link

GrimzEcho commented May 10, 2019

Bug Report

The Popup component does not adjust its show or hide speed when the delay property is set.
The mouseEnterDelay and mouseLeaveDelay properties work, but they are undocumented and were only found after searching for "popup delay" in the project's issues.

Steps

  1. Create a Popup like the following
const PopupExample = () => (
  <Popup
    content="Add users to your feed"
    trigger={<Button icon="add" />}
    delay={{ show: 2000, hide: 4000 }}
  />
);

Expected Result

The popup appears two seconds after mouseover and hides two seconds after the mouse leave

Actual Result

The popup appears immediately and hides immediately

Version

0.87

Testcase

https://codesandbox.io/s/8zo77xv0j2

Notes

My assumption is that Semantic-UI-React components should have all of the same options as normal Semantic-UI components, but perhaps this is incorrect. This is based on the link to the SUI docs from the SUI-React docs, and on the "Complete SUI component definition support" phrasing in the SUI-React's readme file

@welcome
Copy link

welcome bot commented May 10, 2019

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you've completed all the fields in the issue template so we can best help.

We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

@layershifter
Copy link
Member

We support only mouseEnterDelay and mouseLeaveDelay. They belong to Portal component and are mentioned there: https://react.semantic-ui.com/addons/portal/

The problem with missing docs for some props is tracked under #1169.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants