Skip to content

Conversation

@albozek
Copy link
Contributor

@albozek albozek commented Aug 20, 2025

@albozek albozek requested review from a team and Copilot August 20, 2025 13:04
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the icon size classes in CKEditor embed components from ibexa-icon--small to ibexa-icon--small-medium as part of a coordinated effort to standardize icon sizing across the platform.

  • Updated icon size class in CKEditor embed content components
  • Changed from ibexa-icon--small to ibexa-icon--small-medium for options menu buttons

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/bundle/Resources/public/js/CKEditor/embed/content/embed-editing.js Updated icon size class for embedded content menu trigger button
src/bundle/Resources/public/js/CKEditor/embed/content-inline/embed-inline-editing.js Updated icon size class for inline embedded content menu trigger button

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@albozek albozek force-pushed the IBX-10440-base-size-icon branch from eb1c1f9 to 7fa7f9f Compare September 2, 2025 11:15
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 2, 2025

@pawlakadrian pawlakadrian self-requested a review September 4, 2025 07:15
Copy link

@pawlakadrian pawlakadrian left a comment

Choose a reason for hiding this comment

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

QA Approved on DXP Commerce 5.0.x-dev

@dew326 dew326 merged commit 52c08d3 into main Sep 4, 2025
17 checks passed
@dew326 dew326 deleted the IBX-10440-base-size-icon branch September 4, 2025 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants