Skip to content

fix bogus 404 errors for sidebar toolbar icons#15378

Open
timar wants to merge 1 commit intomainfrom
private/timar/404
Open

fix bogus 404 errors for sidebar toolbar icons#15378
timar wants to merge 1 commit intomainfrom
private/timar/404

Conversation

@timar
Copy link
Copy Markdown
Member

@timar timar commented Apr 3, 2026

Strip the sc_ (small-command) prefix in getIconNameOfCommand() so that sidebar .uno: toolbar buttons resolve to the correct lc_ SVG icons that COOL ships. Without this, sidebar controllers that register commands like sc_changecasetoupper produce icon lookups that don't match any file.

Also:

  • skip legacy core sr* resource IDs (like sr20006) in icon lookup
  • add icon aliases: insertfooter, textbodyparastyle,
    showmultiplepages, showtwopages, validatedialogsa11y,
    validatesidebara11y
  • add changecasetoupper icon (light + dark) from colibre theme
  • add dark mode icon for ai_sidebar

@github-project-automation github-project-automation bot moved this to To Review in Collabora Online Apr 3, 2026
@timar timar force-pushed the private/timar/404 branch 3 times, most recently from 0e671bd to 35b1e30 Compare April 4, 2026 10:11
@timar timar requested a review from vmiklos April 4, 2026 14:12
@timar timar force-pushed the private/timar/404 branch from 35b1e30 to c69d9aa Compare April 6, 2026 08:02
Strip the sc_ (small-command) prefix in getIconNameOfCommand()
so that sidebar .uno: toolbar buttons resolve to the correct
lc_ SVG icons that COOL ships. Without this, sidebar controllers
that register commands like sc_changecasetoupper produce icon
lookups that don't match any file.

Also:
- skip legacy core sr* resource IDs (like sr20006) in icon lookup
- add icon aliases: insertfooter, textbodyparastyle,
  showmultiplepages, showtwopages, validatedialogsa11y,
  validatesidebara11y
- add changecasetoupper icon (light + dark) from colibre theme
- add dark mode icon for ai_sidebar

Signed-off-by: Andras Timar <andras.timar@collabora.com>
Change-Id: I6a91b161af0269330265822440c3f0a87fbb104b
@timar timar force-pushed the private/timar/404 branch from c69d9aa to 04b06bc Compare April 6, 2026 08:05
@timar timar requested a review from eszkadev April 7, 2026 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To Review

Development

Successfully merging this pull request may close these issues.

1 participant