fix: adjust media toolbar styles and comment out attachment filters i… - #16
fix: adjust media toolbar styles and comment out attachment filters i…#16shakurShirajul wants to merge 3 commits into
Conversation
shakurShirajul
commented
Aug 5, 2026
Fixed this css issues.
There was a problem hiding this comment.
Pull request overview
Adjusts WordPress media modal/admin UI styling in the shared CSS reset to address layout issues in the media toolbar area.
Changes:
- Commented out a media modal attachment filter margin rule.
- Added new overrides for
.media-toolbarand.media-toolbar-secondary.
Suppressed comments (1)
frontend/resources/css/wp-css-reset.css:171
- There are now two "Media styles end" markers, and the first one appears before additional media-related rules. This is misleading when navigating the file; keep a single end marker at the actual end of the section or rename the earlier comment to describe the subsection.
/* Media styles end */
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…s into fix/media-modal-toolbar-styles
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
frontend/resources/css/wp-css-reset.css:170
- The "Media styles end" comment is placed before additional media styles (.media-toolbar-secondary), and it is duplicated again at the actual end of the section. This makes the section boundary misleading for future edits.
.media-toolbar {
margin-left: 0 !important;
height: 60px !important;
padding-left: 0 !important;
}
/* Media styles end */
.media-toolbar-secondary {
width: 0px !important;
}
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
frontend/resources/css/wp-css-reset.css:172
- The "Media styles end" section marker is duplicated and currently placed before the
.media-toolbar-secondaryrule, which makes the section boundary misleading. Also,width: 0pxcan be simplified to0.
Consider keeping a single end marker after the last media-related rule and using 0 for the width value.
.media-toolbar {
margin-left: 0 !important;
height: 60px !important;
padding-left: 0 !important;
}
/* Media styles end */
.media-toolbar-secondary {
width: 0px !important;
}
/* Media styles end */