-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add logo to login and password pages #322
Conversation
Signed-off-by: Valentin Bräutigam <valentin.braeutigam@gmail.com>
Signed-off-by: Valentin Bräutigam <valentin.braeutigam@gmail.com>
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.
Nice! This addition makes the page a little prettier but it would be even prettier in my opinion if the logos were outside the cards I find that it overloads the cards.
<div class="logos"> | ||
<img class="didentity_login" src="../../assets/images/DIdentity.png" alt="DIdentity" /> | ||
|
||
<img | ||
class="adorsys_login" | ||
src="../../assets/images/adorsys.png" | ||
alt="adorsys" | ||
/> | ||
</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.
I think it would be better to separate the logo from the cards for the login and the change password
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.
The problem is that then there would be a white box around each of the logos which looks quite awful. I tried it but it looks better inside the cards.
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.
OK I understand. But would if we place a separator between the logo and the title of the card it would be a little better?
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.
I do not think so
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.
Okay!
Signed-off-by: Valentin Bräutigam <valentin.braeutigam@gmail.com>
<div class="logos"> | ||
<img class="didentity_login" src="../../assets/images/DIdentity.png" alt="DIdentity" /> | ||
|
||
<img | ||
class="adorsys_login" | ||
src="../../assets/images/adorsys.png" | ||
alt="adorsys" | ||
/> | ||
</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.
Okay!
Added the didentity and adorsys logo to the login page and the change password page