-
Notifications
You must be signed in to change notification settings - Fork 310
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
[WV-659] & [WV-660] VoterPositionEntryAndDisplay && VoterPositionEditNameAndPhotoModal #4228
[WV-659] & [WV-660] VoterPositionEntryAndDisplay && VoterPositionEditNameAndPhotoModal #4228
Conversation
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 are accessibility issues in these changes.
<div className={classes.container}> | ||
<span className={classes.label}>Opinion visible to:</span> | ||
<FormControl className={classes.formControl}> | ||
<Select |
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.
Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.
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.
👏 You fixed the issue(s)! Great work.
07b536c
to
855c6f5
Compare
… click "What's your opinion?" & VoterPositionEntryAndDisplay: Open modal to edit name & add photo [WV-659] & [WV-660] Implement VoterPositionEntryAndDisplay with Related Modals - Created a new functional component `VoterPositionEntryAndDisplay` to manage user opinions and enable modal interactions. - Implemented two related modals: 1. "What's your opinion?" modal for adding or editing opinions (`WebApp/src/js/components/PositionItem/VoterPositionEntryAndDisplay.jsx`). 2. Edit Name & Add Photo modal for updating user details (`WebApp/src/js/components/PositionItem/VoterPositionEditNameAndPhotoModal.jsx`). - Managed state for visibility, opinion text, and user details dynamically using `VoterStore`. - Styled components to align with the design system, utilizing `DesignTokenColors` for consistent theming. - Both modals were included in the same commit since one opens the other, ensuring seamless integration. Notes: - Future updates to each modal can be made independently in different branches i jest decided to show it and push commit together for this two modals.
…PostPublicDropdown.jsx Apply fixes and improve accessibility in ActivityPostPublicDropdown.jsx
…ur opinion?" Fix all styles, Add mobile styles including for Iphone SE, Remove font-familly, Move down styles in file, Remove icon svg use MUI instead
fix dropdown height on iphone SE and fix bug with radiogroup
- Updated modal structure to mirror SettingsProfile.jsx design - Integrated components: SettingsProfilePicture, SettingsWidgetAccountType, SettingsWidgetFirstLastName, SettingsWidgetOrganizationDescription, and SettingsWidgetOrganizationWebsite - Added styled-components for cleaner layout and responsiveness - Fixed eslint issues across the file
[WV-659] & [WV-660] add 2 modals [TEAM REVIEW] [WV-659] & [WV-660] add 2 modals [TEAM REVIEW] [WV-659] & [WV-660] add 2 modals [TEAM REVIEW] [WV-659] & [WV-660] add 2 modals [TEAM REVIEW] [WV-659] & [WV-660] add 2 modals [TEAM REVIEW] [WV-659] & [WV-660] add 2 modals [TEAM REVIEW] [WV-659] & [WV-660] add 2 modals [TEAM REVIEW]
79bb2fc
to
b6972a1
Compare
Thank you Kateryna! 👍 |
WV-659 & WV-660 VoterPositionEntryAndDisplay && VoterPositionEditNameAndPhotoModal
VoterPositionEntryAndDisplay
to manage user opinions and enable modal interactions.VoterPositionEditNameAndPhotoModal
to user photo and name.WebApp/src/js/components/PositionItem/VoterPositionEntryAndDisplay.jsx
).WebApp/src/js/components/PositionItem/VoterPositionEditNameAndPhotoModal.jsx
).DesignTokenColors
for consistent theming.Notes:
Both modals were included in the same commit since one opens the other.