Skip to content

Fix control styles on the gallery settings page - #1238

Open
utkarshcloudinary wants to merge 1 commit into
developfrom
fix/gallery-settings-layout-controls-styling
Open

Fix control styles on the gallery settings page#1238
utkarshcloudinary wants to merge 1 commit into
developfrom
fix/gallery-settings-layout-controls-styling

Conversation

@utkarshcloudinary

@utkarshcloudinary utkarshcloudinary commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

The Layout control and the other custom controls on the gallery settings page showed the default browser button style. The controls did not get their styles.

The styles are in gallery.scss. Only the block editor imported this file. The settings page app did not import it, and the enqueued styles did not include it.

Approach

  • Import gallery.scss in the settings page app (src/js/components/settings-gallery.js). The build makes css/gallery.css.
  • Enqueue css/gallery.css on the gallery settings page (php/media/class-gallery.php).

Commit 0b0b520 removed the gallery.scss import from the settings page app.

QA notes

  1. Connect the plugin to a Cloudinary account.
  2. Go to Cloudinary > Gallery settings.
  3. Open the Layout panel.
  4. Make sure the four layout options (Expanded 1/2/3 Column, Classic) show as a list. Each option has an icon and a label. The options do not show the default browser button style.
  5. Select a different layout option. Make sure the selected option gets the active color.
  6. Make sure the preview on the right updates.
image

The Layout control and the other custom controls on the gallery
settings page showed the default browser button style. The controls
did not get their styles.

The styles are in gallery.scss. Only the block editor imported this
file. The settings page app did not import it, and the enqueued styles
did not include it.

To correct this:
- Import gallery.scss in the settings page app.
- Enqueue the built css/gallery.css on the settings page.

Commit 0b0b520 removed the gallery.scss import from the settings page
app.
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