Skip to content
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.

Sign up for GitHub

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

Jump to bottom

updated design elements #109

Merged
merged 7 commits into from
Dec 22, 2023
Merged

updated design elements #109

merged 7 commits into from
Dec 22, 2023

Conversation

Kilian
Copy link
Contributor

@Kilian Kilian commented Dec 8, 2023 •
edited
Loading

Design updates

todo

  • dont say welcome, but explain in title plus add columns
  • background for categories/browsers bar and unique checkbox for increased segmentation
  • hover/focus effect for tags/browsers, main menu and all links
  • fix clear search button in firefox/make style fit other places
  • add browser icons to ui like canidevtools
  • 320px width horizontal scrollbar
  • accidentally apply prettier to the entire style.css

Screenshots:

homepage:
image

search icon:
image

Tip page:
image

Checkbox:
image

@Kilian Kilian marked this pull request as draft December 8, 2023 09:53
@Kilian
Copy link
Contributor Author

Kilian commented Dec 8, 2023

320px wide no longer has a horizontal scrollbar:
image

captainbrosset
captainbrosset reviewed Dec 8, 2023
View reviewed changes
src/assets/style.css Outdated Show resolved Hide resolved
@Kilian
Copy link
Contributor Author

Kilian commented Dec 8, 2023

and icons:
image

@Kilian Kilian marked this pull request as ready for review December 8, 2023 10:32
@captainbrosset
Copy link
Owner

Thanks so much for the PR @Kilian. Sorry for the delay here. I'll try to get to this in the coming week.

@Kilian
Copy link
Contributor Author

Kilian commented Dec 11, 2023

no rush!

@captainbrosset
Copy link
Owner

Ok I just went through this and really like most of it! Thank you Kilian. So I'll merge the PR and then just push another commit on top of it to change a few minor things.

@captainbrosset captainbrosset merged commit 3ff92fd into captainbrosset:main Dec 22, 2023
@captainbrosset captainbrosset mentioned this pull request Dec 22, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@captainbrosset captainbrosset captainbrosset left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Kilian @captainbrosset