Skip to content

fix: adjust media toolbar styles and comment out attachment filters i… - #16

Open
shakurShirajul wants to merge 3 commits into
mainfrom
fix/media-modal-toolbar-styles
Open

fix: adjust media toolbar styles and comment out attachment filters i…#16
shakurShirajul wants to merge 3 commits into
mainfrom
fix/media-modal-toolbar-styles

Conversation

@shakurShirajul

Copy link
Copy Markdown
image Fixed this css issues.

Copilot AI lite review requested due to automatic review settings August 5, 2026 10:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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-toolbar and .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.

Comment thread frontend/resources/css/wp-css-reset.css Outdated
Copilot AI review requested due to automatic review settings August 13, 2026 06:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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;
}

Copilot AI review requested due to automatic review settings August 13, 2026 06:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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-secondary rule, which makes the section boundary misleading. Also, width: 0px can be simplified to 0.

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 */

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants