-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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: Adjust chevron alignment to the right edge #7438
Fix: Adjust chevron alignment to the right edge #7438
Conversation
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
This pull request adjusts the alignment of chevron icons in the webhook and API keys tables, addressing the issue described in #7403.
- Modified
SettingsDevelopersWebhookTableRow.tsx
to use a more flexible grid layout for better chevron alignment - Updated
SettingsApiKeysFieldItemTableRow.tsx
andSettingsApiKeysTable.tsx
to set a fixed 24px width for the chevron column - Adjusted padding in
SettingsDevelopersWebhookTableRow.tsx
to position the chevron 4px from the right edge - Removed unused React import in
SettingsDevelopersWebhookTableRow.tsx
for code cleanliness
3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
...twenty-front/src/modules/settings/developers/components/SettingsApiKeysFieldItemTableRow.tsx
Outdated
Show resolved
Hide resolved
packages/twenty-front/src/modules/settings/developers/components/SettingsApiKeysTable.tsx
Outdated
Show resolved
Hide resolved
...wenty-front/src/modules/settings/developers/components/SettingsDevelopersWebhookTableRow.tsx
Outdated
Show resolved
Hide resolved
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.
left some comments :) 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.
LGTM!
Thankyou @Vardhaman619 ;)
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.
LGTM thanks
Awarding Vardhaman619: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/Vardhaman619 |
This pull request addresses the alignment issue of the chevron icon, ensuring that it is positioned correctly on the right edge. Fixes [twentyhq#7403](twentyhq#7403) ![after fix](https://github.com/user-attachments/assets/84e6cd14-1d10-4331-8f25-da5423b15dd3) --------- Co-authored-by: Charles Bochet <[email protected]> Co-authored-by: ehconitin <[email protected]>
This pull request addresses the alignment issue of the chevron icon, ensuring that it is positioned correctly on the right edge.
Fixes #7403