Skip to content
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

Donate help page form assembly form styling #11316

Merged

Conversation

danielfmiranda
Copy link
Collaborator

@danielfmiranda danielfmiranda commented Oct 25, 2023

Description

Link to sample test page: https://donate-foundation-s-11173-dona-mrgari.mofostaging.net/en/help/
Related PRs/issues: #11173

Screenshots

Screenshot 2023-10-26 at 14-11-20 Donate Help

@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 25, 2023 19:28 Inactive
@danielfmiranda
Copy link
Collaborator Author

Hi @kristinashu and design team!

I now have a working, styled rendition of the donate help page form that can be found here:
https://donate-foundation-s-11173-dona-mrgari.mofostaging.net/en/help/

I was wondering if I can please get some help by having someone on the design team giving the form a look over, and letting me know if there are any more updates to the styling that I should make, or if everything looks OK?

Thank you very much!

@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 25, 2023 20:42 Inactive
@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 25, 2023 21:22 Inactive
@danielfmiranda danielfmiranda changed the base branch from main to feature/donate-help-page October 26, 2023 03:52
@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 26, 2023 03:53 Inactive
@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 26, 2023 04:49 Inactive
@kristinashu
Copy link

Hi @danielfmiranda, sorry I missed this yesterday! I'm heading out on PTO this afternoon so I won't be able to review it but @nancyt1 could you please do it?

At first glace it looks great! Thank you for flagging the re-captcha and the long drop down titles.

… word wrapping on select element for chrome)
@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 26, 2023 21:06 Inactive
@danielfmiranda danielfmiranda changed the title WIP: donate help fa form styling Donate help page form assembly form styling Oct 26, 2023
@danielfmiranda
Copy link
Collaborator Author

no problem @kristinashu!

After our discussion during the creative-crit meeting, I have left recaptcha as is, and was able to get the dropdown menus to wrap on mobile for both Safari and Chrome browsers.

However, I am still working on implementing that for Firefox.

Also, I reached out to stakeholders for review and they gave the OK on the current design.

@nancyt1 do you think you could please take a look at the review page whenever you find a free moment and let me know if everything is looking OK?

Thanks!

Copy link
Collaborator

@nancyt1 nancyt1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't have an android device to test on but I think everything is working as expected!

@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 27, 2023 02:16 Inactive
@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 27, 2023 05:20 Inactive
@danielfmiranda danielfmiranda changed the base branch from feature/donate-help-page to 11173-donate-help-fa-form October 27, 2023 05:21
@danielfmiranda danielfmiranda changed the base branch from 11173-donate-help-fa-form to feature/donate-help-page October 27, 2023 05:21
@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 27, 2023 05:25 Inactive
@danielfmiranda danielfmiranda requested review from mmmavis and removed request for mmmavis October 27, 2023 05:30
@kristinashu
Copy link

Thank you @danielfmiranda, it's looking good! Thank you @nancyt1 for reviewing!

@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 27, 2023 18:43 Inactive
@danielfmiranda
Copy link
Collaborator Author

Thanks @kristinashu @nancyt1 for the reviews and approvals!

Wanted to call one thing out regarding the dropdown wrapping issue that we had discussed earlier.

While I was able to get the select element to wrap its contents on Chrome and Safari, It appears that it is not possible on FireFox.

As a workaround, I updated the <select> element's <option> font-size at the XS breakpoint to 12px.

That way, even if the dropdown <select> element is truncating the selected option with an ellipsis, the user is able to fully see every option when selecting.

Screenshots

word-wrap working on chrome and safari:

image

<select> truncating long choices with ellipsis on FireFox:

image

Choice font size before (note that they are also being truncated):

Screenshot 2023-10-27 at 7 24 47 PM

Choice font size after:

Screenshot 2023-10-27 at 7 23 43 PM

Copy link
Collaborator

@mmmavis mmmavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! The FA form looks pretty well on the page. I left one UX concern around having fixed height textarea. Please see my inline comment. Thanks, @danielfmiranda !

}

textarea {
height: 10em !important;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<textarea> by deafult can be resized by users. Setting a fixed height for <textarea> would be confusing to users because they still see the resize control but dragging it around doesn't change the sizing due to our CSS override.

To improve the behaviours, we have three options here:

  1. Give textarea a fixed height & add resize: none; to hide the resizing UI control.
  2. Change this line to use min-height instead.
  3. Don't add any height/size CSS overrides to textarea.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation @mmmavis!

The height of the text box was updated per request of the stakeholder, however, I agree that users should be able to resize it if needed.

For that reason, I went with option 2, which is updating the height to min-height.

@danielfmiranda danielfmiranda temporarily deployed to foundation-s-11173-dona-mrgari October 31, 2023 23:39 Inactive
Copy link
Collaborator

@mmmavis mmmavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lovely! R+ ✨

@danielfmiranda danielfmiranda merged commit 50cdacb into feature/donate-help-page Nov 1, 2023
6 checks passed
@danielfmiranda danielfmiranda deleted the 11173-donate-help-fa-form-styling branch November 1, 2023 19:25
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.

4 participants