-
-
Notifications
You must be signed in to change notification settings - Fork 313
Design improvements in pages #277
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
Conversation
|
@souravbadami Please review this PR. |
website/templates/email.html
Outdated
| <p>{{emails}}</p> | ||
| </div> | ||
| </div> | ||
| {% endblock %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a line ending here.
| </div> | ||
| </div> | ||
| </div> | ||
| </div> No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a line ending here.
website/static/css/main.css
Outdated
| #login p { | ||
| font-size: 13px; | ||
| padding-top: 20px; | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a line ending here.
website/static/css/main.css
Outdated
|
|
||
|
|
||
| .no-shadow{ | ||
| box-shadow: 0 0 0 #000; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace tab with 4 spaces.
website/static/css/main.css
Outdated
| @@ -1,30 +1,68 @@ | |||
| .fa{ | |||
| color: #555; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace tab with 4 spaces. Also, look for indentation with 4 spaces in this entire file. I can see several places that has to be fixed.
Remove inline css Remove useless JS snippets Improve design Refactor code for better readabilty Fix indentation Add a line at the end
|
@souravbadami Updated the commit |
souravbadami
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Remove inline css Remove useless JS snippets Improve design Refactor code for better readabilty Fix indentation Add a line at the end
The commit handles the following: