Skip to content

ci: add apt-get update/upgrade - #307

Merged
sc0w merged 1 commit into
amule-project:masterfrom
sc0w:aptupgrade
Dec 5, 2021
Merged

ci: add apt-get update/upgrade#307
sc0w merged 1 commit into
amule-project:masterfrom
sc0w:aptupgrade

Conversation

@sc0w

@sc0w sc0w commented Nov 22, 2021

Copy link
Copy Markdown
Member

Fixes ci error

@sc0w
sc0w merged commit e26d06a into amule-project:master Dec 5, 2021
@sc0w
sc0w deleted the aptupgrade branch December 5, 2021 12:24
mrjimenez pushed a commit to mrjimenez/amule that referenced this pull request Jul 5, 2026
…#307)

Follow-up to amule-project#295: the old `#ifdef __WXOSX__` right/bottom pads (14/9)
compensated for `AssignImageList`'s NSTableView render path — zero
native icon/text gap plus vertical-centre-of-the-full-bitmap, which
made a bare 16x16 icon sit flush against the label and drift toward
the row baseline. `SetSmallImages` uses a different wxOSX path that
already inserts native icon/text spacing, so the pads now render as
visible extra whitespace between icon and label. Drop them (and the
now-dead `img.Size()` pass) so wxOSX matches Windows/Linux layout.

Verified locally: rebuilt on macOS 15 ARM64 (wxOSX Cocoa 3.3.2), page
icons now sit at the natural offset the wx-native path picks. No
change on wxGTK or wxMSW paths (both pads were already 0 there).
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