Fixes issue where modal heading could overlap button.#286
Conversation
|
Would be cool to also fix up the line-height on mobile. It's sooooo roomy! lol |
|
This is too much copy for the header part of the modal, the "make sure to full everything out." part should be removed. @mikefantini @aaronschachter Is there any type of character limit in header for the address collection modal? |
|
oh crap I don't think Aaron B understood what I meant... I'll talk to him that should just be "Want Free Thumb Socks" |
|
@DFurnes the part I was talking about re: overlapping was field headers like "Who are you sharing Thumb Socks With and Why" (in addition to the user feedback i.e. The Fantini-inator. People call you that right? gets cut off on mobile) |
|
@mikefantini Yep, working on that in DoSomethingArchive/legacy-website#2167. |
|
cool... ok well I just changed the header copy for that modal |
|
Campaigns page: Modals:
|
Fixes issue where modal heading could overlap button.
4 participants
Fixes DoSomethingArchive/legacy-website#2217.
For review: @DoSomething/front-end