Skip to content

Give the move-subject action an icon that is not the drag cursor - #1364

Merged
JeroenDeDauw merged 1 commit into
masterfrom
feature/move-subject-icon
Sep 7, 2026
Merged

Give the move-subject action an icon that is not the drag cursor#1364
JeroenDeDauw merged 1 commit into
masterfrom
feature/move-subject-icon

Conversation

@JeroenDeDauw

Copy link
Copy Markdown
Member

Follows-up to #1356

The Move action on the Data tab used cdxIconMove, the four-way arrow that is also the
browser's drag cursor, one button away from the row's drag handle in a list that is dragged to
reorder. A glyph that says "drag me" sat on the button that relocates a Subject to another page.

It now uses cdxIconArticleRedirect, a page with an arrow leading away from it: the
container-plus-arrow shape mail and file tools use for "move to", with a page as the container.
The dialog's confirm button and the overflow-menu entry carry the same icon, and the
ResourceLoader icon list follows.

Considered, omitted: a custom page-with-arrow glyph without the redirect association, and moving
the action into a labelled overflow menu on desktop.

AI-authored — Claude Code, Fable 5.1 (max); one-line ask from @JeroenDeDauw after an in-session icon discussion, no revisions; diff not yet human-reviewed; SubjectsManager vitest specs and eslint run locally, icon checked in the dev wiki, CI pending.

Follows-up to #1356

The Move action on the Data tab used `cdxIconMove`, the four-way arrow that is also the
browser's drag cursor, one button away from the row's drag handle in a list that is dragged to
reorder. A glyph that says "drag me" sat on the button that relocates a Subject to another page.

It now uses `cdxIconArticleRedirect`, a page with an arrow leading away from it: the
container-plus-arrow shape mail and file tools use for "move to", with a page as the container.
The dialog's confirm button and the overflow-menu entry carry the same icon, and the
ResourceLoader icon list follows.

Considered, omitted: a custom page-with-arrow glyph without the redirect association, and moving
the action into a labelled overflow menu on desktop.
@JeroenDeDauw
JeroenDeDauw marked this pull request as ready for review September 7, 2026 15:55
@JeroenDeDauw

Copy link
Copy Markdown
Member Author
image

@JeroenDeDauw
JeroenDeDauw merged commit 5aa23a7 into master Sep 7, 2026
20 checks passed
@JeroenDeDauw
JeroenDeDauw deleted the feature/move-subject-icon branch September 7, 2026 15:59
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.

1 participant