From a0e88bb24f1d18c9c0ad2d4bf18b6243093a5cb5 Mon Sep 17 00:00:00 2001 From: Gazook89 Date: Wed, 6 Nov 2024 21:55:30 -0600 Subject: [PATCH] Add comment about future Popover API use --- client/components/Anchored.jsx | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/client/components/Anchored.jsx b/client/components/Anchored.jsx index 4c7a225e4..87af5a6e1 100644 --- a/client/components/Anchored.jsx +++ b/client/components/Anchored.jsx @@ -7,6 +7,11 @@ import './Anchored.less'; // **The Anchor Positioning API is not available in Firefox yet** // So in Firefox the positioning isn't perfect but is likely sufficient, and FF team seems to be working on the API quickly. +// When Anchor Positioning is added to Firefox, this can also be rewritten using the Popover API-- add the `popover` attribute +// to the container div, which will render the container in the *top level* and give it better interactions like +// click outside to dismiss. **Do not** add without Anchor, though, because positioning is very limited with the `popover` +// attribute. + const Anchored = ({ children })=>{ const [visible, setVisible] = useState(false);