-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feedback for Confirmation Email Template #92
Comments
For the email header:The Zoning Application Portal text in should be around For the information card:We might want to try to have the text shifted more towards the center or make the background box smaller since there is a decent amount of negative space on the right. Though we should consider how it looks as well if there are multiple lines of text when a user subscribes to multiple CDs versus one. |
@jessicashanshanhuang Tried to adjust the text to be more center aligned. Following is how it looks on desktop and mobile respectively. QuestionDoes this seem enough? Adjusting the background box or the text more to cover the negative space makes the mobile views more wonky, see below examples. |
@pratishta hmm, I do see how it does look wonky on mobile. The current changes seems good for centering without distorting the other views! Also I noticed the hex color for the text seem to have changed in your version, was that an intentional change? |
@jessicashanshanhuang No, these screenshots are from the Sendgrid dynamic template preview. A screenshot from my Gmail inbox looks like the following |
@jessicashanshanhuang Regarding changing the font size, we currently calculate the font size of 'Zoning Application Portal' in the header based on the viewport height and width. Changing it to Let me know if you'd still like to go ahead with the font changes. |
Hey @pratishta 👋 The body text looks good, I was wondering if we could adjust the header font size to |
@jessicashanshanhuang By header, do you mean the 'Department of City Planning' part? |
@TylerMatteo QA feedback for when a user sees an email to confirm their subscriptions.
The text was updated successfully, but these errors were encountered: