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: Bolder Typeface + Configurable Letter Spacing for Minimap's Section Header Labels ✨ #209990

Merged
merged 8 commits into from
May 27, 2024

Conversation

pouyakary
Copy link
Contributor

@pouyakary pouyakary commented Apr 9, 2024

This PR fixes #209989 and brings a bolder minimal Section Header Label with letter spacing for the section header labels configurable by user:

Help Wanted Portion — I have noticed that changing the letter spacing's value in the settings doesn't translate to a rerender right away, which I haven't figured why. Can you please give me guidance on how to address it?

@pouyakary pouyakary changed the title Closes #209989 Closes #209989 - Bolder Section Header Labels + Configurable Letter Spacing for Section Header Labels. Apr 9, 2024
@pouyakary pouyakary changed the title Closes #209989 - Bolder Section Header Labels + Configurable Letter Spacing for Section Header Labels. Bolder Section Header Labels + Configurable Letter Spacing for Section Header Labels. Apr 9, 2024
@pouyakary pouyakary changed the title Bolder Section Header Labels + Configurable Letter Spacing for Section Header Labels. Feat: Bolder Typeface + Configurable Letter Spacing for Minimap's Section Header Labels. Apr 10, 2024
@pouyakary pouyakary marked this pull request as draft April 10, 2024 13:33
@pouyakary
Copy link
Contributor Author

I have noticed that overflow is not addressed correctly:

Screenshot 1403-01-22 at 5 02 13 PM

I have changed the PR to a draft so that I can fix it.

@pouyakary
Copy link
Contributor Author

Overflow is fixed:

@pouyakary pouyakary marked this pull request as ready for review April 10, 2024 15:44
@pouyakary pouyakary force-pushed the pouya/better-minimap-labels branch from d44d63d to ac91d42 Compare April 10, 2024 15:52
@pouyakary pouyakary closed this Apr 11, 2024
@pouyakary pouyakary reopened this Apr 11, 2024
@pouyakary
Copy link
Contributor Author

pouyakary commented Apr 11, 2024

Sorry I closed it by mistake trying to post a comment. One thing about the bold typeface is that if you notice it, in some screenshots it is bold, and in some (like my machine) it was not. So I guess putting the weight ensures we're all on the same thing. However, I guess in the future it could be very nice to have a custom font for the minimal headers in the settings. Maybe each person sees better with a different font.

@pouyakary pouyakary changed the title Feat: Bolder Typeface + Configurable Letter Spacing for Minimap's Section Header Labels. Feat: Bolder Typeface + Configurable Letter Spacing for Minimap's Section Header Labels ✨ Apr 13, 2024
@pouyakary
Copy link
Contributor Author

@alexdima can you please check this out?

Copy link
Member

@alexdima alexdima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@alexdima alexdima enabled auto-merge (squash) May 27, 2024 18:37
@alexdima alexdima added this to the May 2024 milestone May 27, 2024
@alexdima alexdima merged commit 841a916 into microsoft:main May 27, 2024
6 checks passed
@pouyakary pouyakary deleted the pouya/better-minimap-labels branch June 5, 2024 15:02
@pouyakary
Copy link
Contributor Author

@alexdima thank you a lots! ♥️

@microsoft microsoft locked and limited conversation to collaborators Jul 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Minimap Section Header Labels Are Really Hard To Read
3 participants