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

feat: rework sider design #360

Merged
merged 1 commit into from
Mar 10, 2024
Merged

Conversation

quentinguidee
Copy link
Contributor

@quentinguidee quentinguidee commented Mar 2, 2024

Some design refinements for the drawer

Before After
image image
Before After
image image

@Yooooomi
Copy link
Owner

Yooooomi commented Mar 2, 2024

Dude you got no chill haha. It seems cool. Is light mode working as intended? Is responsive still ok?

@quentinguidee
Copy link
Contributor Author

Here are screenshots for dark mode:

image image

And responsive seems to work fine!

@maaaathis
Copy link

It looks really good 👍
The only thing I've never fully understood is the state of the current page. The red text differs greatly from the hover state. But idk.

@Yooooomi
Copy link
Owner

Yooooomi commented Mar 2, 2024

I guess we can just have the hover state persist when on the page

@maaaathis
Copy link

I guess we can just have the hover state persist when on the page

A bit darker or lighter, but yes.

@Yooooomi
Copy link
Owner

Yooooomi commented Mar 2, 2024

Lighter on dark mode, darker on light mode

@quentinguidee
Copy link
Contributor Author

This way?

Enregistrement.d.ecran.le.2024-03-02.a.13.26.20.mov

Signed-off-by: Quentin Guidée <git@arra.red>
@Yooooomi
Copy link
Owner

Yooooomi commented Mar 2, 2024

Maybe full white written in black when focused

@maaaathis
Copy link

Maybe full white written in black when focused

Thats the same too big contrast as on the search before.

@Yooooomi Yooooomi merged commit e6ad0bc into Yooooomi:master Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants