Skip to content

Centre the disk image's icons on the new background's arrow - #2316

Merged
Johann-PLW merged 1 commit into
abrignoni:mainfrom
Johann-PLW:fix/dmg-background-arrow
Oct 1, 2026
Merged

Johann-PLW merged 1 commit into
abrignoni:mainfrom
Johann-PLW:fix/dmg-background-arrow

Conversation

@Johann-PLW

@Johann-PLW Johann-PLW commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

The new disk image background draws the drag arrow 44 points further right: 88 pixels on the @2x image, with its centre now at x=479. The app and Applications icons move with it, from (260, 290) and (610, 290) to (304, 290) and (654, 290).

  • packaging/dmg_background@2x.png is the new background at 1920x1080.
  • packaging/dmg_background.png is that same image reduced to 960x540. tiffutil -cathidpicheck accepts the pair.
  • packaging/dmg_settings.py has the new icon positions.
  • A new test, test_the_icons_sit_either_side_of_the_arrow, finds the arrow on the 1x background and requires the two icons to sit either side of it. If a future background moves the arrow, this test fails instead of the problem only showing up on a Mac. It fails with the old positions.
  • .claude/rules/ileapp-build-and-release.md describes the move.

🤖 Generated with Claude Code

The new background (2026-10-01) draws the drag arrow 44 points further
right, 88 pixels on the @2x image, centred at x=479. The app and the
Applications link move with it, from (260, 290) and (610, 290) to
(304, 290) and (654, 290), the layout VLEAPP already uses.

A test now finds the arrow on dmg_background.png and requires the two
icons to straddle it, so a background that moves the arrow again fails
in the test suite rather than on a Mac.

The 1x background is the @2x reduced to 960x540; tiffutil
-cathidpicheck accepts the pair.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Johann-PLW
Johann-PLW merged commit 0791b3a into abrignoni:main Oct 1, 2026
15 checks passed
@Johann-PLW
Johann-PLW deleted the fix/dmg-background-arrow branch October 1, 2026 08:21
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