Skip to content
Snippets Groups Projects
Unverified Commit 2d8d5798 authored by Venkata Krishna S's avatar Venkata Krishna S Committed by GitHub
Browse files

Adds delay before note is displayed on hover. Solves issue #658. (#668)


* Adds delay

* change delay to half second.

* Added async

* Changed Delay

* lint fix

* Changed hover actions

* onMouseLeave

* Change logic

---------

Co-authored-by: default avatarJed Fox <git@jedfox.com>
parent 69a54a16
No related branches found
No related tags found
No related merge requests found
......@@ -79,6 +79,22 @@ export default function NotesButton({
tooltip.close();
}
const [delayHandler, setDelayHandler] = useState(null);
const handleMouseEnter = event => {
setDelayHandler(
setTimeout(() => {
// whatever your data is
setHover(true);
}, 300),
);
};
const handleMouseLeave = () => {
clearTimeout(delayHandler);
setHover(false);
};
// This account for both the tooltip hover, and editing tooltip
const tooltipOpen = tooltip.isOpen || (hasNotes && hover);
......@@ -97,8 +113,12 @@ export default function NotesButton({
'& button, & .hover-visible': { display: 'flex', opacity: 1 },
},
]}
onMouseEnter={() => setHover(true)}
onMouseLeave={() => setHover(false)}
onMouseEnter={async () => {
handleMouseEnter();
}}
onMouseLeave={() => {
handleMouseLeave();
}}
>
<Button
bare
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment