Closed Bug 1778958 Opened 3 years ago Closed 3 years ago

Folder properties dialog too small for translations/localizations like Finnish, Russian etc.

Categories

(Thunderbird :: Theme, defect, P4)

Thunderbird 102

Tracking

(thunderbird_esr102+ fixed, thunderbird103 fixed)

RESOLVED FIXED
104 Branch
Tracking Status
thunderbird_esr102 + fixed
thunderbird103 --- fixed

People

(Reporter: tikkanen, Assigned: henry-x)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(2 files, 2 obsolete files)

Attached file Screenshot of the problem (obsolete) (deleted) —

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:103.0) Gecko/20100101 Firefox/103.0

Steps to reproduce:

  1. Right click any folder
  2. Select "Ominaisuudet" ("Properties")

Actual results:

Dialog that opens is too small for content.

I guess that problem is that title fields are longer than in original (English) version.

Expected results:

Dialog should be resized for the content.

Attached file Too_Small_Dialog.png (obsolete) (deleted) —
Attachment #9284897 - Attachment is obsolete: true

Ouch! Thanks Tuomo.

Richard, if it's not your territory, you can ask Henry I guess for auto-sizing the dialog.

Component: Untriaged → Theme
Flags: needinfo?(richard.marti)
Summary: folder property dialog too small for Finnish translation → Folder properties dialog too small for Finnish translation
Blocks: tb102found
Severity: -- → S4
Priority: -- → P4

Probably from bug 1703164. A width of 570px was set on the window https://hg.mozilla.org/comm-central/rev/78d2559050ab#l2.29

I think removing the width and height from the toplevel <html> and putting a max-width: 570px on the xul:description elements might work better.

Regressed by: 1703164

(In reply to Henry Wilkes (they/them) [:henry] from comment #3)

Probably from bug 1703164. A width of 570px was set on the window https://hg.mozilla.org/comm-central/rev/78d2559050ab#l2.29

I think removing the width and height from the toplevel <html> and putting a max-width: 570px on the xul:description elements might work better.

Tried this and it doesn't work, at least on Windows there only a minimal square is shown.

Henry, could you look if you find a solution? I played also with window.sizeToContent(); but this failed too for me.

Flags: needinfo?(richard.marti)
The content of attachment 9284899 [details] has been deleted for the following reason: Requested by author
Assignee: nobody → henry
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
The content of attachment 9284897 [details] has been deleted for the following reason: Deleted at request of author

Reattaching the original screenshot as it got deleted accidentally.

Target Milestone: --- → 104 Branch

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/40e08e3440e4
Don't size the folderProps dialog. r=Paenglab,mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Summary: Folder properties dialog too small for Finnish translation → Folder properties dialog too small for translations/localizations like Finnish, Russian etc.

Let's uplift this soon to avoid getting duplicates for more languages...

Flags: needinfo?(henry)

Comment on attachment 9284956 [details]
Bug 1778958 - Don't size the folderProps dialog. r=Paenglab

[Approval Request Comment]
Regression caused by (bug #): bug 1703164
User impact if declined: Folder property sizing issues
Testing completed (on c-c, etc.): c-c
Risk to taking this patch (and alternatives if risky): it is very safe

Flags: needinfo?(henry)
Attachment #9284956 - Flags: approval-comm-esr102?
Attachment #9284956 - Flags: approval-comm-beta?

Comment on attachment 9284956 [details]
Bug 1778958 - Don't size the folderProps dialog. r=Paenglab

[Triage Comment]
Approved for beta

Attachment #9284956 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9284956 [details]
Bug 1778958 - Don't size the folderProps dialog. r=Paenglab

[Triage Comment]
Approved for esr102

Attachment #9284956 - Flags: approval-comm-esr102? → approval-comm-esr102+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: