Open
Bug 789744
Opened 12 years ago
Updated 2 years ago
dialog prompting for message body format is cut off at bottom
Categories
(Thunderbird :: Message Compose Window, defect)
Tracking
(Not tracked)
NEW
People
(Reporter: jik, Assigned: aceman)
References
(Depends on 1 open bug)
Details
Attachments
(2 files)
With Mozilla/5.0 (X11; Linux x86_64; rv:18.0) Gecko/18.0 Thunderbird/18.0a1, built 20120830130605, on Linux:
1. Open a compose window from an email account.
2. Put some HTML text (e.g., bold, italic) in the body of the message.
3. Change the sender to a news account identity.
4. Add both a To and a Newsgroups recipient to the header of the message.
5. Try to send.
You will get the attached dialog. Note how the bottom is cut off.
Reporter | ||
Updated•12 years ago
|
Status: NEW → UNCONFIRMED
Ever confirmed: false
Comment 1•12 years ago
|
||
I see that you specified Linux in platform affected.
FWIW, the full dialog window appears correctly in Win7
Strangely enough I now tested it in TB18 on Win XP and I get different wording in the dialog than you show in the screenshot.
Can you update your Daily and see if it has changed?
Reporter | ||
Comment 3•12 years ago
|
||
There are actually two different windows with slightly different wording. One of them is cut off and one of them is not. If you follow *exactly* the steps I posted above, on Linux, you will get the prompt window that this bug is about. I have just reconfirmed this after updating my trunk build.
Reporter | ||
Comment 5•12 years ago
|
||
(In reply to :aceman from comment #4)
> What is the format of a Newsgroups recipient ?
Name of the newsgroup.
I think I have followed the steps but I can't get the other (truncated dialog).
Reporter | ||
Comment 7•12 years ago
|
||
Aha! I have a customization in my userChrome.js which makes the steps slightly different for me.
You should be able to reproduce by changing the foreground color of some text in step 2, instead of just bold or italic.
Reporter | ||
Comment 8•12 years ago
|
||
And by the way, when I undo my userChrome.js customizations and follow the original steps above, I get a different dialog like you were getting, and the Cancel and Send buttons are visible in that dialog, but they aren't _fully_ visible, i.e., even that dialog is slightly shorter than it should be.
OK, I have seen the other dialog now, in Win XP and it is OK. I'll see in Linux in a while.
Comment 10•12 years ago
|
||
(In reply to :aceman from comment #9)
> OK, I have seen the other dialog now, in Win XP and it is OK. I'll see in
> Linux in a while.
Flags: needinfo?(acelists)
Assignee | ||
Comment 11•12 years ago
|
||
Yeah I can see it on linux.
The dialog is this: chrome://messenger/content/messengercompose/askSendFormat.xul
I suspect this is because some of the texts in the dialog are inserted after the dialog is up via .textContent and that does not cause reflow.
The problem may depend on exact font used for the texts and whether the "anyway?" word fits on the preceding line. For me the word "plain" wrapped to the second line.
I've tried window.sizeToContent(); but it does not help either.
Assignee: nobody → acelists
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(acelists)
Hardware: x86_64 → All
Comment 12•12 years ago
|
||
Huh? The basic alert dialog does something similar except it manually inserts its text nodes, so I don't see why this dialog should have a problem. Is there a regression range?
Assignee | ||
Comment 13•12 years ago
|
||
OK, on Windows both of the paragraphs also wrap (the 'plain text' and 'anyway' words are on second line). Also the buttons are centered in the bottom of the dialog. On linux they are on bottom right.
Assignee | ||
Comment 14•12 years ago
|
||
No regression range yet.
There were big changes to the file in bug 507682 but that one does not seem to touch the logic much. I do not see any other relevant changes in hg log of this file.
Assignee | ||
Comment 15•12 years ago
|
||
Neil, there is something crappy going on here. The Dom inspector reports the <dialog> height to be 600x250px. But that is not true, the window manager sees it as 600x218px. That is when the buttons are hidden behind the bottom edge of the dialog. If you cancel the dialog and open it again, it comes up fine. Both heights are now really 250px.
Assignee | ||
Comment 16•10 years ago
|
||
So we need reflow to happen after the text is filled in. There is probably no .innerText in Gecko, so maybe we can use innerHTML. If that doesn't help, there may be a problem in XUL toolkit.
Assignee | ||
Comment 17•10 years ago
|
||
Neither .innerHTML not sizeToContent() helps me.
<deck> does help a bit because it preallocates the space for the longest of the strings.
But the buttons are still cut of slightly. The dialog is now 250px high, but DOM inspector says it is 266px (also .clientHeight says so). So there is something wrong in the reflow.
Any further proposals?
Flags: needinfo?(neil)
Comment 18•10 years ago
|
||
Not sure, I can't seem to reproduce this on Windows.
Once you have the dialog in the "incorrect" state where DOM Inspector disagrees on its height, what happens when you evaluate target.defaultView.sizeToContent() in DOM Inspector's JS Object pane?
Flags: needinfo?(neil)
Comment 19•8 years ago
|
||
I also see this on SeaMonkey.
Should we just add a height to the dialog box or could the explanations all be moved to the dtd and put into the xul, then just reveal the relevant one?
Flags: needinfo?(acelists)
Assignee | ||
Comment 20•8 years ago
|
||
I don't think we can add a fixed height as the translations may be differently long.
If you mean using <deck>, I tried that in comment 17 and it didn't work. Or did you mean something else?
There seems to be some bug in rendering that the window does not have the size the other components think it has.
Flags: needinfo?(acelists)
Reporter | ||
Comment 21•5 years ago
|
||
lol just saw this issue again, went to see if someone had already reported it, and discovered that I had seven years ago.
Is the cut-off dialog shown in the attachment the same issue as the one I reported seven years ago, or should it be treated as a new, different issue?
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•