-
-
Notifications
You must be signed in to change notification settings - Fork 123
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
Design improvements in pages #277
Conversation
@souravbadami Please review this PR. |
website/templates/email.html
Outdated
<div class="col-lg-12"> | ||
<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> |
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.
#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
background: #eee; | ||
border-radius: 50%; | ||
font-size: 30px; | ||
box-shadow: 0 1px 1px #dd4252; | ||
} | ||
|
||
|
||
.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 |
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: