Email verifycation template is not styled

readthedocs / ext-theme

Read the Docs drop in replacement site templates

2 stars 2 forks source link

Email verifycation template is not styled #435

Closed humitos closed 3 days ago

humitos commented 1 month ago

It seems we are using a default allauth template. This one in particular: https://github.com/pennersr/django-allauth/blob/301324e3ed2bf7c95987f24625ee8ebd7480ce85/allauth/templates/account/verification_sent.html

I think we want to override them to:

Add an email icon (similar to our "Verify email" template shown when the user clicks the link from the email): https://github.com/readthedocs/ext-theme/blob/790e59de6c0e2a82a9df026a7553470c384bc49f/readthedocsext/theme/templates/account/email_confirm.html#L10-L13
Don't use title case

Screenshot_2024-07-25_17-50-58

humitos commented 1 month ago

Our looks like:

Screenshot_2024-07-25_17-52-13

agjohnson commented 3 weeks ago

You know, I have noticed a lot of regressions on allauth templates that look like this too. I wonder if we lost something in an upgrade. I know this is just a missing template that needs added, but I've seen this fairly broken styling on a number of allauth views lately.

I opened up #462 to investigate. Maybe I'm wrong though.

agjohnson commented 4 days ago

Just a note, on the examples above, the first template is missing, the second template is the one that users see after clicking the email verification email they are sent, so is a separate template.