Opened 11 years ago

Closed 10 years ago

#2827 closed bug (fixed)

[Pe] Save As dialog box has over lapping button (easy)

Reported by: scottmc Owned by: axeld
Priority: normal Milestone: R1
Component: - General Version: R1/pre-alpha1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

The Save As dialog box for Pe has the type drop down overlapping the cancel button. I suspect this points to something wrong in Haiku?

Attachments (2)

pe-save-as.png (36.1 KB) - added by scottmc 11 years ago.
2827.diff (4.1 KB) - added by maxime.simon 10 years ago.

Download all attachments as: .zip

Change History (9)

Changed 11 years ago by scottmc

Attachment: pe-save-as.png added

comment:1 Changed 11 years ago by scottmc

This still happens with PE-2.4.1-RC3 on hrev28589.

comment:2 Changed 10 years ago by scottmc

Summary: [Pe] Save As dialog box has over lapping button[Pe] Save As dialog box has over lapping button (easy)

Changed 10 years ago by maxime.simon

Attachment: 2827.diff added

comment:3 Changed 10 years ago by maxime.simon

The patch reorganizes the SavePanel to look like other SavePanels which have a "FileType" menu.
(Indeed I have copied a lot on the source code of Icon-o-matic. :)

comment:4 Changed 10 years ago by zooey

Resolution: fixed
Status: newclosed

Applied Maxime's patch in hrev555 of Pe's repo - thanks!

Maxime: as given, your patch did not work at all under BeOS R5 and it took me some time of debugging to find out that it was simply using an uninitialized font-height structure. Haiku just happens to have some appropriate value on the stack then, while R5 does not ;-) I took the liberty to fix that by initializing the struct via be_plain_font->GetHeight().

comment:5 Changed 10 years ago by maxime.simon

Really sorry about that, I did just have Haiku as BeOS-like on which to test.

comment:6 Changed 10 years ago by scottmc

Resolution: fixed
Status: closedreopened

Reopening... the patch fixes the issue, but we still need to get the updated Pe into the image. Once someone posts a new Pe binary from hrev555 or newer, and adds it to the Optional Packages, then this ticket can be closed.

comment:7 Changed 10 years ago by stippi

Resolution: fixed
Status: reopenedclosed

I built Pe on Haiku, called it 2.4.2 and updated the optional packages file in hrev30153.

Note: See TracTickets for help on using tickets.