Skip to content

Commit 1fd5861

Browse files
dknaussDan Knaussclaude
authored
fix(editor): match toolbar Exit icon to admin-bar exit (dashicons-exit) (#55)
The edit-mode toolbar's Exit button used dashicons-arrow-left-alt while the admin-bar toggle for the same action uses dashicons-exit. Switch the toolbar to dashicons-exit so one action shares one glyph across both surfaces, and so Exit reads distinctly from the toolbar's own up/down move arrows. Also corrects the stale code comment that claimed a no-alt (×) glyph for Exit; it had already drifted to the back-arrow. Co-authored-by: Dan Knauss <you@real.email> Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
1 parent dcd208a commit 1fd5861

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

assets/maestro.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -553,17 +553,20 @@
553553

554554
var right = el( 'div', 'maestro-toolbar-right' );
555555

556-
// Icon-only like the panel controls. Distinct glyphs: image-rotate (restore
556+
// Icon-only like the panel controls. Distinct glyphs: backup/restore (restore
557557
// everything) for Reset All vs the panel's undo (single-item) Reset Item;
558-
// no-alt (×) for Exit. aria-label + title carry the name; Reset All is still
559-
// guarded by the confirm dialog in doResetAll().
558+
// the exit-door glyph for Exit — matching the admin-bar toggle
559+
// (class-admin-bar.php) so the same action shares one icon across surfaces,
560+
// and so Exit reads distinctly from the toolbar's ▲/▼ move arrows. aria-label
561+
// + title carry the name; Reset All is still guarded by the confirm dialog in
562+
// doResetAll().
560563
var resetAll = el( 'button', 'button maestro-reset-all' );
561564
resetAll.type = 'button';
562565
iconButton( resetAll, 'dashicons-backup', I.resetAll );
563566
resetAll.addEventListener( 'click', doResetAll );
564567

565568
var exit = el( 'a', 'button maestro-exit' );
566-
iconButton( exit, 'dashicons-arrow-left-alt', I.exit );
569+
iconButton( exit, 'dashicons-exit', I.exit );
567570
exit.href = D.exitUrl;
568571
exit.addEventListener( 'click', onExit );
569572

0 commit comments

Comments
 (0)