Opened 15 years ago
Closed 10 years ago
#5285 closed bug (fixed)
Aesthetic bug in Mail app
Reported by: | xeon3d | Owned by: | bga |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Applications/Mail | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | #5732 | |
Platform: | All |
Description (last modified by )
Attachments (2)
Change History (11)
by , 15 years ago
Attachment: | Haiku-2010-01-18-02-37-48.png added |
---|
by , 11 years ago
Attachment: | 0001-Fix-5285-Aesthetic-bug-in-Mail-app.patch added |
---|
Calculate base width using widest string
comment:2 by , 11 years ago
patch: | 0 → 1 |
---|
comment:3 by , 11 years ago
Let me know how I can improve this patch. For example, should I use k prefix for a BString? Originally I did it using const char* but I tried to follow the guidelines to use B classes whenever possible/convinient.
The best solution would be to rewrite the layout of the program using BLayouts but that's going to be a bigger task, this patch takes care of the bug :)
Thanks, Guillermo
comment:4 by , 10 years ago
Blocking: | 5732 added |
---|
comment:5 by , 10 years ago
Blocking: | 5732 removed |
---|---|
Description: | modified (diff) |
Keywords: | visual language resize removed |
Platform: | x86 → All |
Priority: | low → normal |
Summary: | Aesthetic bug in Mail app. → Aesthetic bug in Mail app |
comment:6 by , 10 years ago
Patch looks good, but indeed converting to BLayout would be a better solution.
comment:7 by , 10 years ago
I'm glad it looks good, maybe it can be applied while someone rewrites it using layout, that would prevent more issues about this and also make the e-mail client usable while it's done.
Cheers, Guillermo
comment:8 by , 10 years ago
Blocking: | 5732 added |
---|
Screenshot of the affected area