-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
✅ Deploy Preview for ethereum-org-fork ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
yarn add focus-trap-react, add associated hooks, implement in RootLayout
cf9f1d4
to
6c16af5
Compare
@@ -20,6 +20,7 @@ | |||
"@emotion/styled": "^11.11.0", | |||
"@storybook/react": "^7.3.2", | |||
"clipboard": "^2.0.11", | |||
"focus-trap-react": "^10.2.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is this being used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import FocusTrap from "focus-trap-react" |
<FocusTrap |
or in canonical:
https://github.com/ethereum/ethereum-org-website/blob/9163e6b7a3063c4ccec282c63b39685b60abe667/src/components/FeedbackWidget.tsx#L189
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job @wackerow, looks great.
I've pushed a commit to fix the not parsed translation and some conflicts with dev.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
focus-trap-react
packageuseKeyPress
hookuseOnClickOutside
hookFeedbackWidget
componentFeedbackWidget
insideRootLayout
Cherry-picks bc400d6 from #152, implementing
useSurvey
hookRelated Issue
https://www.notion.so/efdn/Add-Feedback-widget-c99e6d8e04b0425ca6769c7ddf63ba7d?pvs=4