Opened 12 months ago

Closed 12 months ago

Last modified 4 months ago

#15269 closed bug (fixed)

Wrong BMenuBar width

Reported by: X512 Owned by: leavengood
Priority: normal Milestone: R1/beta2
Component: Kits/Interface Kit Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

This is hrev53381. BMenuBar auto layout width is 1 bigger than it should be. I causes clipping beveled right side and corrupting design.

Attachments (4)

screenshot1 (1).png (7.4 KB ) - added by X512 12 months ago.
Before fix
screenshot2 (1).png (7.4 KB ) - added by X512 12 months ago.
After fix
0001-Fix-BMenuBar-width.patch (943 bytes ) - added by X512 12 months ago.
Patch
0001-fix-BMenuBar-width-in-Tracker.patch (826 bytes ) - added by X512 12 months ago.
patch for Tracker

Download all attachments as: .zip

Change History (9)

by X512, 12 months ago

Attachment: screenshot1 (1).png added

Before fix

by X512, 12 months ago

Attachment: screenshot2 (1).png added

After fix

by X512, 12 months ago

Patch

by X512, 12 months ago

patch for Tracker

comment:1 by waddlesplash, 12 months ago

Owner: changed from nobody to leavengood
Status: newassigned

I'll assign this to Leavengood, he started picking up BMenu issues lately.

Please check that BFilePanels and the "Open with" window still look OK after the Tracker change.

Can you submit these changes to Gerrit?

comment:2 by leavengood, 12 months ago

Wow that is subtle, I don't know how you noticed it. It would help if you can get these to Gerrit otherwise I can apply them locally, do some testing, then push them up.

in reply to:  1 comment:3 by X512, 12 months ago

Replying to waddlesplash:

Please check that BFilePanels and the "Open with" window still look OK after the Tracker change.

BFilePanel look correct. BFilePanel use default layouting (https://xref.plausible.coop/source/xref/haiku/src/kits/tracker/FilePanelPriv.cpp#679).

Can you submit these changes to Gerrit?

I do not have private GitHub account for now.

comment:4 by leavengood, 12 months ago

Resolution: fixed
Status: assignedclosed

Thanks, I used git am to apply your patches, tested and pushed them up as hrev53397.

comment:5 by nielx, 4 months ago

Milestone: UnscheduledR1/beta2

Assign tickets with status=closed and resolution=fixed within the R1/beta2 development window to the R1/beta2 Milestone

Note: See TracTickets for help on using tickets.