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.

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

Fixed small design bugs #397

Closed
wants to merge 5 commits into from
Closed

Fixed small design bugs #397

wants to merge 5 commits into from

Conversation

cybrneon
Copy link

Fixes to fonts, logo, layouts, optical sizes and details. + new image of browsers

Fixes to fonts, logo, layouts, optical sizes and details. + new image of browsers
@NOCanoa
Copy link
Member

NOCanoa commented Dec 31, 2024

somthing is broken somewhere lol
pic

@NOCanoa NOCanoa added the invalid This doesn't seem right label Dec 31, 2024
@cybrneon
Copy link
Author

Weird, it's not like that on my side. What could cause that?

Screenshot 2024-12-31 at 11 51 11

@TheRealMG
Copy link
Collaborator

TheRealMG commented Dec 31, 2024

Weird, it's not like that on my side. What could cause that?

Looks like it comes down to the dark:text-paper attribute on line 25 in NavBar.astro. Is that necessary? It works perfectly without it as far as I can tell.

@TheRealMG
Copy link
Collaborator

Could you also do a version without the orange outline, revert that to normal? Cheff doesn't like the orange accent there

@cybrneon
Copy link
Author

cybrneon commented Jan 1, 2025

Sure, I fixed the navbar.

@cybrneon
Copy link
Author

cybrneon commented Jan 1, 2025

Also, I just removed the white border around the navbar

image

@NOCanoa NOCanoa removed the invalid This doesn't seem right label Jan 2, 2025
@NOCanoa
Copy link
Member

NOCanoa commented Jan 2, 2025

@mr-cheff thots? ,
also conflict ._.

@NOCanoa NOCanoa added the mr. m ⌚ waithing for cheff opinion/confirmation label Jan 6, 2025
@cybrneon cybrneon closed this Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mr. m ⌚ waithing for cheff opinion/confirmation
Projects
Development

Successfully merging this pull request may close these issues.

3 participants