-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Improve migrate page and add card CSS #13751
Conversation
silverwind
commented
Nov 29, 2020
- Use original gitea logo on migrate page
- Add card styles and map colors to css vars
- Tweak migrate page, adding hover effect to cards
a2df779
to
cc6fe5f
Compare
@silverwind I think the wihte in the middle of gitea logo should be transparent, that's why I mainly did not use the original logo. |
Actually, the non-white center is what got me to replacing the logo. I think white parts of logos should stay white in dark themes and it's also consistent with the page logo/favicon. |
I'm realy not a fain of white-center ... I wont block the refactor - but in my opinion, would not change the logo EDIT: write if you are done with refactoring this area :) |
Happy with refactors already :) |
It's not consistent with other logos, both git and github has a transparent but gitea log doesn't. |
It makes our use of the logo consistent. Our logo in the upper left has the teabag filled in. |
I guess after a certain degree of "internal complexitity" of a logo, it's warrented to fill that part white as it's not meant to be seen as transparent. Maybe it's just a personal preference of some that those parts are white but I've been getting used to it. |
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.
Makes sense to me -- the white is a teabag string and card which are actually white and supposed to be on the outside of the cup. Other logos are not really based so directly off of real world objects.
e994144
to
b7b043a
Compare
Rebased. @6543 maybe you want to reconsider? |
yes LGTM for the JS & CSS but I'm still unhappy with the logo |
a06810f
to
a8e0e2c
Compare
@silverwind conflicts ocure :/ |
- Use original gitea logo on migrate page - Add card styles and map colors to css vars - Tweak migrate page, adding hover effect to cards
a8e0e2c
to
82fa3b6
Compare
Rebased. |