1291000 - Local directory text box too long, causing half buttons in Account manager

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, defect)

Description

[Ton [:Tonnes]]

Attachment: AS RSS Local dir.png

Starting in 48a, the text box for Local directory has become a little longer when viewing Feed settings, causing the dialog width to be too small if its size is not wide enough. See screenshot, created with new profile.

This may however be intentional and only show up because the dialog width is just too small (by default / unresized) and not remembered - see bug 1290991.

While at it, I think "Local directory" could or even should be changed to use "Local folder" for obvious reasons.

Comment 1

[:aceman]

Isn't this caused by the width of "tag prefix" field, from bug 1263878 ?

(In reply to Ton from comment #0)
> While at it, I think "Local directory" could or even should be changed to
> use "Local folder" for obvious reasons.

Bug 721332 .

Comment 2

[Richard Marti (:Paenglab)]

(In reply to :aceman from comment #1)
> Isn't this caused by the width of "tag prefix" field, from bug 1263878 ?

You mean bug 1263873

Comment 3

[:aceman]

Yes :) Can you see the problem on Windows on clean profile? I can't on Linux, as I have a wider font.
Then I would make a patch to test.

Comment 4

[Richard Marti (:Paenglab)]

Yes, I see it too on Win 10.

Comment 5

[:aceman]

Attachment: patch

So the account manager has width defined in 'ch', and the feed subscription dialog in 'em'. Maybe it wasn't the best to have a large element in both of them sized in px. Let's try '35ch'. Or we can have a different width in each dialog (but we didn't want that in the mentioned bug where we introduced the width also to AM).

Comment 6

[Richard Marti (:Paenglab)]

Comment on attachment 8776748 [details] [diff] [review]
patch

Looks good. Could you look if you could add a overflow: auto; to let appear scrollbars when the AM is too small like we have on other panels?

Comment 7

[:aceman]

Attachment: patch v2

Good catch, thanks.

Comment 8

[Philip Chee]

Comment on attachment 8776987 [details] [diff] [review]
patch v2

a=me for suite changes

Comment 9

[:aceman]

https://hg.mozilla.org/comm-central/rev/5d476d6dfe087743a2b6abf1593ff3aa6deb1035