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

Improving the style of tabs #347

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Improving the style of tabs #347

wants to merge 7 commits into from

Conversation

RazinShaikh
Copy link
Collaborator

Resolve #323

@RazinShaikh RazinShaikh marked this pull request as ready for review July 31, 2024 21:34
@RazinShaikh
Copy link
Collaborator Author

I am satisfied with how it looks now. I would appreciate if people could check how it looks on their system. I have not tested on mac and linux, so it would be especially good to get feedback for those.

@RazinShaikh
Copy link
Collaborator Author

Tagging @boldar99 @y-richie-y for mac

@boldar99
Copy link
Collaborator

boldar99 commented Aug 1, 2024

For me, this is how it looks:
image

@RazinShaikh
Copy link
Collaborator Author

RazinShaikh commented Aug 1, 2024

What's that horizontal line passing through the tabs?

Also the tabs are centered on Mac for some reason. This makes the lack of a left tab border look a bit strange.

@RazinShaikh
Copy link
Collaborator Author

RazinShaikh commented Aug 1, 2024

Can you also check if the effects of hovering on the tab and the close button look decent? And can you also try tabs with the system's dark mode?

@boldar99
Copy link
Collaborator

boldar99 commented Aug 1, 2024

Hovering on X:
image
image
And dark mode:
image

@RazinShaikh
Copy link
Collaborator Author

Okay so we need more changes to make it work for Mac. Perhaps we can do this in person sometime?

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.

Improve the style of tabs
2 participants