Skip to content

Style Devise forms, #50#51

Open
gabrielbonner wants to merge 2 commits intomasterfrom
style-devise-forms
Open

Style Devise forms, #50#51
gabrielbonner wants to merge 2 commits intomasterfrom
style-devise-forms

Conversation

@gabrielbonner
Copy link
Copy Markdown
Collaborator

@gabrielbonner gabrielbonner commented Oct 9, 2017

Styled the devise forms to look like event cards.

From this:
screen shot 2017-10-14 at 11 22 01 pm

To this:
screen shot 2017-10-14 at 11 23 38 pm

@appleJax
Copy link
Copy Markdown
Collaborator

Looks good to me @gabrielbonner! Merge at will! :shipit:

Copy link
Copy Markdown
Collaborator

@appleJax appleJax left a comment

Choose a reason for hiding this comment

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

Found 1 typo actually

<%= form_for(resource,
as: resource_name,
url: password_path(resource_name),
html: { method: :post, :class = "form" }) do |f| %>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This line should say class: "form"

@gabrielbonner
Copy link
Copy Markdown
Collaborator Author

@appleJax I fixed that typo, good eye.

Styled the User edit form and put a link to it in the navbar
screen shot 2017-11-08 at 11 41 24 pm

@gabrielbonner
Copy link
Copy Markdown
Collaborator Author

@appleJax I've addressed the changes in your request, I can dismiss your request but I think the proper protocol is for you to review and approve. :)

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