Skip to content

fix: standardized 8px horizontal padding for editor actions toolbar #38257

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.

Sign up for GitHub

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

Open
willkhinz wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: standardized 8px horizontal padding for editor actions toolbar #38257

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 25 additions & 0 deletions FIX_PROPOSAL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
To fix the issue of asymmetric horizontal padding in the `.editor-actions` class, you can modify the `padding` property to have equal left and right values. Here is the exact code fix:

```css
.editor-actions {
cursor: default;
flex: initial;
padding: 0 8px 0 8px; /* Changed padding-left to 8px to match padding-right */
height: var(--editor-group-tab-height);
display: flex;
align-items: center;
}
```

Alternatively, you can use a symmetric shorthand padding syntax:

```css
.editor-actions {
cursor: default;
flex: initial;
padding: 0 8px; /* Sets padding-top and padding-bottom to 0, padding-left and padding-right to 8px */
height: var(--editor-group-tab-height);
display: flex;
align-items: center;
}
```
Comment on lines +1 to +25
Copy link
Copy Markdown

@coderabbitai coderabbitai bot Mar 26, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

PR objective is not implemented in code (proposal-only change).

Line 1 through Line 25 add a proposal document, but no actual .editor-actions stylesheet update is included in this PR. The bug fix remains unapplied in runtime/UI code.

🧰 Tools
🪛 LanguageTool

[style] ~1-~1: Consider using a different verb for a more formal wording.
Context: To fix the issue of asymmetric horizontal padd...

(FIX_RESOLVE)

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@FIX_PROPOSAL.md` around lines 1 - 25, The PR only adds a proposal document
but doesn't change the runtime CSS: update the actual stylesheet entry for the
.editor-actions rule (wherever editor/tab styles are defined) to use symmetric
horizontal padding (e.g., change padding: 0 8px 0 0; or padding: 0 0 0 8px; to
padding: 0 8px or padding: 0 8px 0 8px) so left and right match; commit the
CSS/SCSS change (replace the existing .editor-actions padding line) and run the
frontend build/check to verify the UI fix is applied.