Opened 14 years ago

Last modified 9 years ago

#5285 closed bug

Aesthetic bug in Mail app — at Version 5

Reported by: xeon3d Owned by: bga
Priority: normal Milestone: R1
Component: Applications/Mail Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description (last modified by diver)

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.

Screenshot of the affected area

Change History (7)

by xeon3d, 14 years ago

Attachment: Haiku-2010-01-18-02-37-48.png​ added

Screenshot of the affected area

comment:1 by xeon3d, 14 years ago

Forgot to add that it might affect other languages as well.

by gbonvehi, 10 years ago

Attachment: 0001-Fix-5285-Aesthetic-bug-in-Mail-app.patch​ added

Calculate base width using widest string

comment:2 by gbonvehi, 10 years ago

patch: 0 → 1

comment:3 by gbonvehi, 10 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

Last edited 10 years ago by gbonvehi (previous) (diff)

comment:4 by diver, 9 years ago

Blocking: 5732 added

comment:5 by diver, 9 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
Note: See TracTickets for help on using tickets.