153.13 KB, image/png
158.02 KB, image/gif
86.23 KB, image/png
2.37 KB, patch
|Details | Diff | Splinter Review|
6.36 KB, application/octet-stream
72.75 KB, image/png
77.44 KB, image/png
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:220.127.116.11) Gecko/20100401 Ubuntu/9.10 (karmic) Firefox/3.5.9 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:18.104.22.168) Gecko/20100317 Lightning/1.0b1 Thunderbird/3.0.4 If I have an RSS feed with long URLs, the buttons Delete etc are pushed off the right hand edge of the screen. See screenshot for example. Reproducible: Always Steps to Reproduce: 1. Add a feed with a long URL such as The H - Grand unified feed
can you paste a feed url showing up the issue ?
Well yes, there's one in the screenshot, and I mention it by name in the bug report. Here's the URL then. http://rss.feedsportal.com/c/32569/f/491734/index.rss
this WFM on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:22.214.171.124pre) Gecko/20100424 Lanikai/3.1b2pre
Another feed this happens on is http://www.nintendo.co.uk/NOE/en_GB/xml/news_rss2.xml (I'm running Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:126.96.36.199) Gecko/20100826 Thunderbird/3.0.7)
Created attachment 490939 [details] Screenshot of missing buttons and vertical scroll bar I also found UI elements missing - when the FROM line of a message was very long (in my case it is a normal email and not an RSS message). See my screen shot. I think the bug requires that the message body be HTML formatted and not plain text. This happens on all such messages, but the UI displays correctly if I export the message to an EML file and then view the EML with TB. The missing UI elements are located beyond the right side of the window. (expanding the window shows the missing UI elements). Missing are message buttons: Forward Archive Junk and Delete, and also missing is the vertical scroll bar. Also there is no horizontal scroll bar (but I think that is another bug).
I should note that I am using Windows 7 32-bit and TB 3.1.2. Also, if I forward the message to myself, then the FROM line changes, and then the UI displays correctly on the forwarded email.
this is not happening in Tb11, reopen if reproducible in current releases.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WORKSFORME
Created attachment 617266 [details] Screenshot of missing UI problem in v11. I think the problem is much improved - but not gone in TB 11.0.1 In the attached screen shot... with the window narrower than the HTML message width, some of the UI (eg the Delete button) and the scroll bars in the message body are off screen. For this to happen, a warning must be present - like "This message may be a scam" with its "Ignore Warning" button, or the "Show Remote Content" button. Making the window wider shows the missing elements (but this only works if the screen is wide enough to hold the wider window). [Using TB 11.0.1 and Windows 7.]
Created attachment 617317 [details] [diff] [review] patch. you are correct. identifying an edge case and general excellence in bug reporting warrant a fix.
Assignee: nobody → alta88
Status: RESOLVED → REOPENED
Ever confirmed: true
Attachment #617317 - Flags: review?(magnus)
Resolution: WORKSFORME → ---
Attachment #617317 - Flags: review?(magnus) → review?(mbanner)
Comment on attachment 617317 [details] [diff] [review] patch. I think Blake or Mike would be better to look at this.
Attachment #617317 - Flags: review?(mbanner) → review?(bwinton)
Summary: RSS feed with large URLs causes buttons of UI to disappear off right end of screen. → Message headers with large URLs causes buttons of UI to disappear off right end of screen.
Comment on attachment 617317 [details] [diff] [review] patch. I can't replicate this behaviour, so I don't think I can review this patch. Can one of you attach a .eml file that shows the problem, so that I cna continue with it? (For what it's worth, the code changes seem okay, so once I can verify that this actually fixes the problem, it should get a quick r+. Oh, another way you could verify it would be to write a test that, say, checks for the horizontal position of an element, and fails if it would be off the screen. Then I wouldn't need the .eml, and we could be sure we don't break it accidentally in the future.) Thanks, Blake.
The eml file can be used to more or less reproduce the bug when it's under vertical view.
Comment on attachment 617317 [details] [diff] [review] patch. Awesome, thanks! I'll review the patch this afternoon with this .eml file.
Comment on attachment 617317 [details] [diff] [review] patch. Sadly, I'm still not seeing it. (Here's what I do see https://dl.dropbox.com/u/2301433/Screenshots/NoCrop.png if that helps us figure out what I'm doing wrong.) Thanks, Blake.
that's what it's supposed to look like with the patch. without it, the Show Remote Content button is pushed off the screen. at least on win7.
Yeah, but I didn't have the patch applied when I took the screenshot… That's what I saw with just the plain trunk code.
Your computer seems to truncate 'always load content from [email address here]' (see attachment 490939 [details]). I just checked the latest Daily build for Mac OS X, and that email address doesn't seem to be truncated. I think the behavior on your computer would be the desired behavior.
Created attachment 629113 [details] Screen shot - window wide - showing all of UI OK. This screen shot is with the window wide enough to display all of the UI correctly for the test eml file. I will upload another shot of the same window made narrow enough to show the error of the missing UI elements.
Created attachment 629115 [details] Screen shot - window narrow - showing missing UI. Below is a way to reproduce the bug. Just opening an eml file in TB won't do it for me. TB produces no warning message like "To protect your privacy, Thunderbird has blocked remote content in this message" - and the FROM line is not long enough to easily see the problem. So here is one way to reproduce the bug.... 1) go to www.spamgourmet.com and create an account - which is trivial to do. Just give it a user name and your real email address, and respond to their confirmation email. [I guarantee they will not spam you in any way. Its a cool free service that has eaten many thousands of spam emails for me. (See their site for info on how it works).] 2) Download the example "eml file to reproduce the bug" (from tech163) (shortcut: https://bugzilla.mozilla.org/attachment.cgi?id=628776 ) to a temp location on your disk. 3) Open the example eml file in TB (ie double click it in Windows Explorer). 4) Forward the example to your spamgourmet account (which will forward it back to you with a longer FROM line). For best results, use a long spamgourmet 'keyword' to make the FROM line even longer, eg, forward it to something like: LongTestKeyWord.20.XXXX@spamgourmet.com (where XXXX is your Spamgourmet user name). 5) Now read the email that Spamgourmet sends back to you. To see the problem: you must make the TB window small enough, so ... 6) make the window short enough to create a vertical scroll bar in the message, 7) then narrow the window until the scroll bar disappears (bug!). Also some buttons in the header will get clipped off. See my new screen shots. One is with the window wide enough to display all of the UI, the other screen shot shows the window narrower so the scroll bar etc is missing. I am using TB 12.0.1 in Widows 7. FYI, I looked around to try to learn how I could test this patch, but I am not a TB developer and I found no simple directions for how to try out a patch. Is there a reasonably simple way for me to test it?
Comment on attachment 617317 [details] [diff] [review] patch. Okay, I tried it on my Mac with this morning's code, and I managed to reproduce the bug! Woo! So your patch works, and as I'm pretty sure I said previously, the code seems good, so r=me, and thank you for your patience! >+++ b/mail/base/content/mailWindowOverlay.xul >@@ -2058,16 +2058,17 @@ > <label id="allowRemoteContentForAuthorDesc" class="msgNotificaton-smallText text-link" >+ crop="end" > onclick="allowRemoteContentForSender();"/> The only thing I'ld change here (and I'm not going to insist on it, it's more of a preference) is that you put the onclick on the same line as the crop. Thanks again, Blake.
Attachment #617317 - Flags: review+
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago → 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 15.0
You need to log in before you can comment on or make changes to this bug.