Skip to content

Conversation

@ric0de
Copy link
Contributor

@ric0de ric0de commented Dec 17, 2025

Some notes:

  • IMPORTANT The "Paste list" modal doesn't let you import voters without an email, so in order to test for voters with missing emails, I hardcoded a list of invitees that overrides whatever list you currently had selected. To enable this, simply set const TEST_MODE to true.

  • We might want to add a 'Back to invite' button to "Preview invitation" modal when accessing it from the "Invite selected" modal

  • Currently, the first column on the invitees list (their names) has a fixed length of 95px, so any long names get cut off with ellipses. This column can be adjusted to be longer, and the subsequent columns will shift accordingly with no issue, but an idea is to maybe have every voter's last name just be their initial so we can leave the first column short for mobile. Currently any name longer than "John Smith" gets cut off.

  • When there's only 1 invitee left, its respective "Remove from list" button becomes un-selectable to prevent an empty list

  • Minor change, but I fixed the font for all the other modals' dialogTitleJSX since they had broken during the migration to Template A

@DaleMcGrew
Copy link
Member

Thank you @ric0de! 👍

@DaleMcGrew DaleMcGrew merged commit edc959f into wevote:develop Dec 17, 2025
4 checks passed
@ric0de ric0de deleted the WV-2048InviteSelectedModal branch December 18, 2025 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants