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

feat: email styling #547

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Conversation

LeandroCarvalho200409
Copy link

Description

Provide a detailed description of the purpose of the changes included in this pull request. Optionally, include background information, relevant screenshots, and any other context that helps explain the work.

The purpose of this PR is to change the style of all the E-mail templates, so that they look legit and have a user-friendly design.

Related Issues

If applicable, specify the main parts of the application that will be impacted by this pull request.

#544

Steps to reproduce or test

Describe the steps that you did to reproduce this.

@LeandroCarvalho200409
Copy link
Author

Here's a first prototype of the E-mail styling for the E-mail confirmation mail.

image

Copy link
Member

@joaodiaslobo joaodiaslobo left a comment

Choose a reason for hiding this comment

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

I like how it is looking! 🤩
Keep up the good work! 💪

</div>
<div class="h-fit w-fit bg-gray-100 rounded-md pt-8 mx-16">
<h1 class="flex font-extrabold text-xl justify-center">
<p class="text-orange-700 mr-1">Hi</p>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<p class="text-orange-700 mr-1">Hi</p>
<p class="text-orange-700 mr-1">Hello</p>

<div class="h-fit w-fit bg-gray-100 rounded-md pt-8 mx-16">
<h1 class="flex font-extrabold text-xl justify-center">
<p class="text-orange-700 mr-1">Hi</p>
<p class="text-black"><%= @user.email %>!</p>
Copy link
Member

Choose a reason for hiding this comment

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

Pretty sure this exists, but not sure!

Suggested change
<p class="text-black"><%= @user.email %>!</p>
<p class="text-black"><%= @user.name %>!</p>

<p class="text-black"><%= @user.email %>!</p>
</h1>

<p class="text-center mx-8">Welcome to the Atomic-Platform! You can confirm your Atomic account by clicking on the button bellow:</p>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<p class="text-center mx-8">Welcome to the Atomic-Platform! You can confirm your Atomic account by clicking on the button bellow:</p>
<p class="text-center mx-8">Welcome to Atomic! You can confirm your account by clicking on the button below:</p>

<p class="text-center mx-8">Welcome to the Atomic-Platform! You can confirm your Atomic account by clicking on the button bellow:</p>

<div class="flex justify-center my-5">
<a class="bg-transparent text-orange-700 hover:text-white font-semibold border border-orange-500 hover:bg-orange-500 hover:border-transparent rounded py-2 px-4" href="<%= @url %>">Verify your account</a>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<a class="bg-transparent text-orange-700 hover:text-white font-semibold border border-orange-500 hover:bg-orange-500 hover:border-transparent rounded py-2 px-4" href="<%= @url %>">Verify your account</a>
<a class="bg-transparent text-orange-700 hover:text-white font-semibold border border-orange-500 hover:bg-orange-500 hover:border-transparent rounded py-2 px-4" href={@url}>">Verify your account</a>

@joaodiaslobo joaodiaslobo changed the title Styling of all E-mails #544 feat: email styling Dec 6, 2024
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