Closed Bug 1433299 Opened 3 years ago Closed 3 years ago
Link in summary is broken
Steps to reproduce: 1. open https://bugzilla.mozilla.org/show_bug.cgi?id=1430979 2. click "bug 1193394" in summary Actual result: it opens https://bugzilla.mozilla.org/show%3Cwbr%3E_bug%3Cwbr%3E.cgi?id=1193394 Expected result: it opens https://bugzilla.mozilla.org/show_bug.cgi?id=1193394
Why don't you use `overflow-wrap: break-word` instead of inserting <wbr> everywhere?
Just noticed this and was about to file a bug - Kohei, can you take this on?
Yes this is my regression... will fix it.
Assignee: nobody → kohei.yoshino
Status: NEW → ASSIGNED
remove the wbr FILTER to restore link functionality to the summary fields.
Assignee: dylan → glob
Attachment #8954267 - Flags: review?(dylan)
i agree with :emk that using `overflow-wrap: word-break` is the better solution; however, i keep on getting bit by this and removing the injection of <wbr> elements is a trivial change.
Using `overflow-wrap` is an easy solution, not a better solution. That's why MDN and my FxSiteCompat.com have opted for `<wbr>`. This issue can be solved by modifying the regular expression.
(In reply to Kohei Yoshino [:kohei] from comment #13) > Using `overflow-wrap` is an easy solution, not a better solution. Why?
Already explained in Bug 1428270.
Er, if I could explain a bit more, `overflow-wrap: word-break` doesn't always work well, e.g. Bug 1392433 that led to the original Bug 1428270. `<wbr>` meanwhile has been proved by other sites to work better especially with camel cases that can often been seen on BMO. Basically the `overflow-wrap` property needs a new value handling such cases properly, but I don't think it will happen soonish :S (Well, it took 2 _decades_ for CSS editors to add the `calc()` function)
so the wbr filter just has to be smarter.
The fix is coming today.
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.