-
Notifications
You must be signed in to change notification settings - Fork 107
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
New download button, svg icons and banners and a go to top button #218
base: master
Are you sure you want to change the base?
Conversation
Great work!!! Thanks for contributing! :D I love that you've made the banner an SVG! Are there any other fonts you could use for the banner? Something like Inter might be nicer. As I had mentioned in #164, the banner looks off-centre because the little dragon in the egg isn't there. Could you move the qBittorrent logos to the right a bit? I'm not super keen on the hover effect of the download button. I think simply changing the opacity would be nicer as well as increasing the shadow. I'd change the neutral shadow to 30%, the hover shadow to 60%, and the active shadow to 90%. |
Thanks @Poopooracoocoo ,I re-centered the banner , changed the font to Inter Medium (looks great now) and followed your advice for the button effects, what do you think?
I followed your advice and removed dark mode from this PR via a commit since I lack the git knowledge (wasn't a lot of code), I will finish this first then try to get dark mode added later. |
The banner doesn't seem to be using Inter Medium and looks like its using a fallback. Everything is pretty great apart from that issue! Thank you! Something to consider is removing the TLbanner. You could do that in a separate PR. |
Should be right , looks exactly how I made it or am I missing something? and yeah I agree, but first this :) |
Checked on another device and the banner looks great. Just a weird once-off issue! Nevermind me! |
Thank you, I'll read up on it when I have the time, I'm using MINGW64 git bash already just got to put some time into learning more |
Greetings,
My old PR was outdated so decided to start fresh with a new branch, I did things better this time with a css only download button.
Here's a live preview: https://bluewave2.github.io/
Edit: Dark mode removed from this PR
Screenshot is older, check live website for latest look
Dark mode preview (removed from this PR):