diff --git a/website/src/components/UserFeedback/UserFeedback.stories.jsx b/website/src/components/UserFeedback/UserFeedback.stories.jsx index 32f5c40a44..b61dcf5038 100644 --- a/website/src/components/UserFeedback/UserFeedback.stories.jsx +++ b/website/src/components/UserFeedback/UserFeedback.stories.jsx @@ -8,8 +8,8 @@ export default { const Template = (args) => ; -export const FullComponent = Template.bind({}); -FullComponent.args = { +export const Step1 = Template.bind({}); +Step1.args = { open: true, }; diff --git a/website/src/components/UserFeedback/index.jsx b/website/src/components/UserFeedback/index.jsx index 3c0fbef31c..cd53804837 100644 --- a/website/src/components/UserFeedback/index.jsx +++ b/website/src/components/UserFeedback/index.jsx @@ -278,42 +278,46 @@ export const FeedbackWrapper = ({ seedData, open }) => { ); }; - return feedbackIsOpen ? ( -
-
- -
- {state.currentStep === 1 ? ( - - ) : state.currentStep === 2 ? ( - - ) : ( - - )} -
- ) : ( - setFeedbackIsOpen(true)} /> - ); -}; - -const OpenFeedbackButton = ({ openFeedback }) => { return ( - +
+

feedback is {feedbackIsOpen ? 'open' : 'closed'}

+ + +
+
+ +
+ {state.currentStep === 1 ? ( + + ) : state.currentStep === 2 ? ( + + ) : ( + + )} +
+
); }; diff --git a/website/src/components/UserFeedback/styles.module.css b/website/src/components/UserFeedback/styles.module.css index 05588a72f5..565343f3ff 100644 --- a/website/src/components/UserFeedback/styles.module.css +++ b/website/src/components/UserFeedback/styles.module.css @@ -1,3 +1,9 @@ +.user-feedback-container { + --outline-style: 2px solid var(--ifm-color-primary); + --row-gap: 1rem; + --element-horizontal-gap: 1rem; +} + .user-feedback { width: 100%; position: absolute; @@ -8,9 +14,6 @@ box-shadow: var(--ifm-global-shadow-lw); padding: var(--ifm-spacing-vertical) var(--ifm-spacing-horizontal); text-align: center; - --outline-style: 2px solid var(--ifm-color-primary); - --row-gap: 1rem; - --element-horizontal-gap: 1rem; } .user-feedback > * + * { @@ -33,11 +36,16 @@ margin-top: var(--row-gap); } -.user-feedback * { + +.hidden { + display: none; +} + +.user-feedback-container * { outline-offset: 4px; } -.user-feedback *:focus-visible { +.user-feedback-container *:focus-visible { outline: var(--outline-style); } @@ -139,11 +147,11 @@ button.close-button { padding-inline: calc(var(--ifm-spacing-horizontal) * 4); } -.user-feedback button[type='submit']:hover { +.primary-button:hover, .user-feedback button[type='submit']:hover { background-color: var(--ifm-color-primary-lighter); } -.user-feedback button[type='submit']:active { +.primary-button:hover, .user-feedback button[type='submit']:active { background-color: var(--ifm-color-primary-dark); } @@ -183,7 +191,6 @@ button.close-button { } .open-feedback-button { - opacity: 0.55; padding-top: var(--ifm-spacing-vertical); padding-bottom: calc(var(--ifm-spacing-vertical) * 2); border-radius: var(--ifm-global-radius) var(--ifm-global-radius) 0 0;