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 )
When changing the language to Portuguese, the Mail window isn't resized properly. Please check the attached screenshots, since it's kinda late here to properly describe it.
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. 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