Skip to content

Conversation

@newhoa
Copy link
Collaborator

@newhoa newhoa commented Jun 8, 2025

This a very trivial change but it centralizes the source of the "real" (non-symlink) question mark icon to system-help.

Fixes issue where question mark symlinks sometimes linked to /actions/*/help-contents.svg and sometimes linked to /status/*/dialog-question.svg.

Now categories/*/system-help.svg is the question mark icon from which all other question mark icons symlink.


A little more explanation and possible background...

The question mark icon at one time was located at
action/help-contents.svg, and all other question
mark icons were symlinked from there. At one point
the action icons above 48px were removed, and the
symlinks became fragmented.

The status folder dialog-question had symlinks to
actions up to 48px, then had real icons for 64px and 128px.

Then /categories/*/system-help.svg linked sometimes to
actions and sometimes to status.

I figured since system-help existed in all sizes, and categories
is sort of a system-wide concept, it would make sense to use
that one to be the source from which the others linked to.

Fixes issue where question mark symlinks sometimes
linked to `/actions/*/help-contents.svg` and sometimes
linked to `/status/*/dialog-question.svg`.

Now `categories/*/system-help.svg` is the question mark
icon from which all other question mark icons symlink.
@danirabbit danirabbit added this to OS 9 Jul 1, 2025
@danirabbit danirabbit moved this to Needs Review in OS 9 Jul 1, 2025
Copy link
Member

@danirabbit danirabbit left a comment

Choose a reason for hiding this comment

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

This makes sense to me! Good cleanup 🚀

@danirabbit danirabbit enabled auto-merge (squash) July 1, 2025 22:38
@danirabbit danirabbit merged commit 15e7bee into elementary:main Jul 1, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Done in OS 9 Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants