Last Comment Bug 309606 - Too long string in file not found xul error page isn't wrapped
: Too long string in file not found xul error page isn't wrapped
Status: VERIFIED FIXED
: fixed1.8
Product: Core
Classification: Components
Component: Document Navigation (show other bugs)
: Trunk
: All All
: -- trivial (vote)
: mozilla1.9alpha1
Assigned To: Henrik Skupin (:whimboo) [away 09/30 - 10/06]
: Adam Lock
:
Mentors:
: 309407 (view as bug list)
Depends on: 280190
Blocks:
  Show dependency treegraph
 
Reported: 2005-09-22 04:08 PDT by DooShan
Modified: 2006-03-12 18:56 PST (History)
6 users (show)
asa: blocking1.8b5+
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch for SeaMonkey (1.24 KB, patch)
2005-09-22 12:41 PDT, Henrik Skupin (:whimboo) [away 09/30 - 10/06]
neil: review+
asa: approval1.8b5+
Details | Diff | Splinter Review
Patch for Toolkit (1.32 KB, patch)
2005-09-22 12:42 PDT, Henrik Skupin (:whimboo) [away 09/30 - 10/06]
mconnor: review+
asa: approval1.8b5+
Details | Diff | Splinter Review

Description DooShan 2005-09-22 04:08:47 PDT
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b4) Gecko/20050910 MultiZilla/1.8.0.1h SeaMonkey/1.0a
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b4) Gecko/20050910 MultiZilla/1.8.0.1h SeaMonkey/1.0a

When there appears a message that a file wasn't found and the path to the file
is too long, the string isn't wrapped and goes way off the table that shows this
message. It's not a big deal, it just doesn't look nice. The table was displayed
in seamonkey mailnews when my antivirus has deleted a message that contained a
virus.
Here's an image of that bug: <a
href="http://img292.imageshack.us/img292/2536/fnfbug1xf.gif"> fnfbug </a>

Reproducible: Always

Actual Results:  
the table showed a long path that wasn't wrapped


Expected Results:  
the path string should be wrapped
Comment 1 Hermann Schwab 2005-09-22 07:22:40 PDT
This happens only when a local file (on a windows partition) isn't found:
file://c:/ABCDEFG010ABCDEFG020ABCDEFG030ABCDEFG040ABCDEFG050ABCDEFG060ABCDEFG070ABCDEFG080ABCDEFG090ABCDEFG100

http://www.nomozilla.org/ABCDEFG010ABCDEFG020ABCDEFG030ABCDEFG040ABCDEFG050ABCDEFG060ABCDEFG070ABCDEFG080ABCDEFG090ABCDEFG100
gives the same message without the long string, telling only: 
www.nomozilla.org could not be found. Please check the name and try again.

http://www.mozilla.org/ABCDEFG010ABCDEFG020ABCDEFG030ABCDEFG040ABCDEFG050ABCDEFG060ABCDEFG070ABCDEFG080ABCDEFG090ABCDEFG100
gives mozilla.org's custom 404 page.

Comment 2 DooShan 2005-09-22 08:16:37 PDT
Hmmm, don't try to do that (copying an infected email to your local folder) in
your live mailbox or you will lose all your emails. The antivirus will delete
the whole file where the emails are stored and if you have other emails there,
it will delete them as well :).
Comment 3 Henrik Skupin (:whimboo) [away 09/30 - 10/06] 2005-09-22 12:35:42 PDT
Confirmed with Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a1) Gecko/20050920
Firefox/1.6a1

The following location shows the same behavior:
file:///home/asdfjasdlfjlsadflsadjflasdflasdhflsfsdfsdfsdfsdfsfaw4rwerwrwwerwerwerwe345234234234234

We should use 'overflow: auto' for '#errorShortDesc > p' to prevent text
overflowing.
Comment 4 Henrik Skupin (:whimboo) [away 09/30 - 10/06] 2005-09-22 12:41:08 PDT
Created attachment 197071 [details] [diff] [review]
Patch for SeaMonkey
Comment 5 Henrik Skupin (:whimboo) [away 09/30 - 10/06] 2005-09-22 12:42:30 PDT
Created attachment 197072 [details] [diff] [review]
Patch for Toolkit
Comment 6 Jesse Ruderman 2005-09-22 19:31:50 PDT
*** Bug 309407 has been marked as a duplicate of this bug. ***
Comment 7 neil@parkwaycc.co.uk 2005-09-25 11:05:47 PDT
Comment on attachment 197071 [details] [diff] [review]
Patch for SeaMonkey

r+sr=me if mconnor's happy with attachment 197072 [details] [diff] [review].
Comment 8 Scott MacGregor 2005-09-30 16:56:34 PDT
this checkin needs to happen ASAP if this is going to get in for 1.8. Do you
need a check in buddy Henrik?
Comment 9 :Gavin Sharp [email: gavin@gavinsharp.com] 2005-09-30 22:24:32 PDT
Trunk:
mozilla/toolkit/themes/winstripe/global/netError.css; new revision: 1.3;
mozilla/toolkit/themes/pinstripe/global/netError.css; new revision: 1.3;
mozilla/themes/classic/global/netError.css; new revision: 1.3;
mozilla/themes/modern/global/netError.css; new revision: 1.3;

1.8 Branch:
mozilla/toolkit/themes/winstripe/global/netError.css; new revision: 1.1.2.2;
mozilla/toolkit/themes/pinstripe/global/netError.css; new revision: 1.1.2.2;
mozilla/themes/classic/global/netError.css; new revision: 1.1.2.2;
mozilla/themes/modern/global/netError.css; new revision: 1.1.2.2;
Comment 10 Jesse Ruderman 2005-11-21 20:10:47 PST
This caused bug 317372, "XUL error page allows error message to accept focus".

Note You need to log in before you can comment on or make changes to this bug.