Last modified: 2011-01-25 01:01:49 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T23544, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 21544 - Text/URLs not wrapped in commit summary
Text/URLs not wrapped in commit summary
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
CodeReview (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Brion Vibber
http://www.mediawiki.org/w/index.php?...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-11-17 00:50 UTC by Sam Reed (reedy)
Modified: 2011-01-25 01:01 UTC (History)
1 user (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Screen shot of overflowing URL in commit section in list (Chrome 8.0.552.237) (68.91 KB, image/png)
2011-01-22 19:04 UTC, Brion Vibber
Details

Description Sam Reed (reedy) 2009-11-17 00:50:47 UTC
But its fine in IE 8/FF.3.5

So it goes over the Author and looks a bit of a mess...

Not sure if this was a bug for Google, or for here
Comment 1 Sam Reed (reedy) 2009-11-17 00:51:55 UTC
Revisions 58452, 58454-58457, fyi
Comment 2 Sam Reed (reedy) 2009-11-20 00:49:12 UTC
59260 which is just text is the same
Comment 3 Sam Reed (reedy) 2010-07-31 22:59:56 UTC
Text one (59260) occurs on FF 3.6.8

URL's still not fine in Chrome 6
Comment 4 Brion Vibber 2011-01-22 19:04:08 UTC
Created attachment 8019 [details]
Screen shot of overflowing URL in commit section in list (Chrome 8.0.552.237)

Whether browsers are able to wrap very long words & URLs or just let them stick out varies with the browser, and has been a long source of annoyance!

Firefox 4.0b9 seems to wrap the long URLs nicely, but I can still see the reported issue in Chrome 8.0.552.237.

The good news is that while we probably can't force it to wrap them, we can stop it from overflowing the text into the next cell.

The stylesheets have a 20em 'max-width' for .TablePager_col_cr_message, which is what forces the cell to be so small. (This prevents the entire table from being insanely huge when there's a giant unwrapped URL in the comment.)

Adding 'overflow: hidden' to this crops the overflowed text, which is pretty much what we want here. Should be easy to add to the styles.
Comment 5 Brion Vibber 2011-01-22 20:39:16 UTC
Committed on trunk in r80767
Comment 6 Sam Reed (reedy) 2011-01-23 17:30:13 UTC
Amusingly, on Chrome 10.0.642.2 dev it seems to have been fixed. :P

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


Navigation
Links