Skip to content

Commit c3aeaea

Browse files
LSalamiLSalami
committed
fix(gui): thicker reload stroke, dedicated message-bubble icon (amule-org#735 review)
got3nks, testing amule-org#735: - reload: arrow bodies read too thin at 16px. Regenerated via Recraft with an explicit thicker/bolder-stroke prompt. - Messages panel header: reusing "amule:toolbar_messages" (the main toolbar's detailed gradient mascot bust) at 16x16 doesn't read as "messages" once shrunk that far from its 32x32 native size. Added a dedicated "amule:message" chat-bubble glyph instead, sized for legibility at 16x16 specifically, and pointed the Messages panel header at it instead of the toolbar art. Folder/folder_shared and the tab-close X were already approved as-is. Verified via a full amule build (macOS) and a visual check of both fixes: the shared-files reload button and the Messages panel header.
1 parent 6503597 commit c3aeaea

6 files changed

Lines changed: 523 additions & 214 deletions

File tree

‎src/icons/icon_data.c‎

Lines changed: 508 additions & 208 deletions
Large diffs are not rendered by default.

‎src/icons/message.png‎

467 Bytes
Loading

‎src/icons/message.svg‎

Lines changed: 7 additions & 0 deletions
Loading

‎src/icons/reload.png‎

-27 Bytes
Loading

‎src/icons/reload.svg‎

Lines changed: 2 additions & 2 deletions
Loading

‎src/muuli_wdr.cpp‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3446,11 +3446,13 @@ wxSizer *messagePageMessages( wxWindow *parent, bool call_fit, bool set_sizer )
34463446

34473447
wxBoxSizer *item1 = new wxBoxSizer( wxHORIZONTAL );
34483448

3449-
// Explicit 16x16: this header sits next to Friends' equally-sized icon
3450-
// (messagePageFriends above), while the bare art id's natural size is
3451-
// the 32x32 toolbar rendition of the same "amule:toolbar_messages" SVG.
3449+
// "amule:message" is a dedicated small chat-bubble glyph, not the main
3450+
// toolbar's "amule:toolbar_messages" mascot illustration (#735 review:
3451+
// that one is a detailed gradient bust that stops reading as
3452+
// "messages" once shrunk from its native 32x32 down to this header's
3453+
// 16x16).
34523454
wxStaticBitmap *item2 = new wxStaticBitmap( parent, -1,
3453-
wxArtProvider::GetBitmapBundle( "amule:toolbar_messages", wxART_OTHER, wxSize(16, 16) ),
3455+
wxArtProvider::GetBitmapBundle( "amule:message" ),
34543456
wxDefaultPosition, wxDefaultSize );
34553457
item1->Add( item2, 0, wxALIGN_CENTER, 5 );
34563458

0 commit comments

Comments
 (0)