-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix workspace switcher font weight #9471
Fix workspace switcher font weight #9471
Conversation
Fixes twentyhq#9456 Hi, this should fix the problem
Welcome!
Hello there, congrats on your first PR! We're excited to have you contributing to this project. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
Modified font weight styling in Chrome extension buttons from 600 to 500 to maintain consistent design across the application.
- Updated font weight to 500 in
packages/twenty-chrome-extension/src/contentScript/createButton.ts
button styles - Aligned font weight to 500 in
packages/twenty-chrome-extension/src/contentScript/insertSettingsButton.ts
for visual consistency
💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!
2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's not code without testing it locally, thanks! ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PoloSpark not the right package, please test it locally first :)
The changes you made are in twenty-chrome-extension
, whereas the issue is in twenty-front
.
@PoloSpark should we close this or do you plan to make the change in the right place? Thanks! |
Closing the PR as there is no activity and the changes are not made in the right place |
Fixes #9456
Hi, this should fix the problem