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(signup page): update design #217

Merged
merged 4 commits into from
Oct 27, 2023
Merged

Conversation

andreybakanovsky
Copy link
Contributor

No description provided.

@andreybakanovsky
Copy link
Contributor Author

image

= link_to 'Login now', new_user_session_path, class: 'has-text-weight-bold has-text-budgeting-kid-color'
.for-desktop.column.has-background-image.is-flex.is-justify-content-center.is-align-items-center
figure.image.is-coins
= image_tag('coins.svg')
Copy link
Contributor

Choose a reason for hiding this comment

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

add empty line, please

Copy link
Member

Choose a reason for hiding this comment

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

remove?

Base automatically changed from dev/update-disign-login-page to master October 27, 2023 17:32
figure.image.is-24x24
= image_tag('google.svg', class:'is-rounded', alt: 'G')
|   Google
/= link_to "#", method: :post, class: 'button has-fixed-width is-outlined is-budgeting-kid-hoverable' do
Copy link
Member

Choose a reason for hiding this comment

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

do we need this?

@andreybakanovsky andreybakanovsky force-pushed the dev/update-disign-signup-page branch from fb7abc7 to 9a31e33 Compare October 27, 2023 18:41
@ka8725 ka8725 marked this pull request as ready for review October 27, 2023 20:54
@ka8725 ka8725 merged commit 2b952b1 into master Oct 27, 2023
2 checks passed
@ka8725 ka8725 deleted the dev/update-disign-signup-page branch October 27, 2023 20:54
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.

3 participants