Launchpad is in denial about <wbr>'s void status
Bug #971273 reported by
William Grant
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
William Grant | ||
Bug Description
Launchpad currently spits out <wbr></wbr> sequences instead of the classical <wbr>. Amusingly, while having a separate closing tag makes it well-formed XML, because it's a void element it's neither valid HTML nor XHTML.
We should use <wbr /> instead. HTML5 and XML both like that, as do all the browsers we support. The only thing I've found that doesn't is BeautifulSoup; it assumes the closing tag was forgotten, so extends the element to the next tag.
Related branches
lp:~wgrant/launchpad/wbr-is-void
- Ian Booth (community): Approve
-
Diff: 448 lines (+99/-93)10 files modifiedlib/lp/app/__init__.py (+8/-2)
lib/lp/app/browser/stringformatter.py (+2/-2)
lib/lp/app/browser/tests/test_stringformatter.py (+23/-23)
lib/lp/app/doc/displaying-paragraphs-of-text.txt (+45/-45)
lib/lp/app/doc/tales-email-formatting.txt (+10/-10)
lib/lp/app/doc/tales.txt (+5/-5)
lib/lp/blueprints/workitemmigration.py (+1/-1)
lib/lp/bugs/stories/bugs/xx-bug-comments-truncated.txt (+2/-2)
lib/lp/registry/browser/tests/person-views.txt (+2/-2)
lib/lp/registry/stories/announcements/xx-announcements.txt (+1/-1)
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| tags: | added: disclosure sharing |
To post a comment you must log in.
Fixed in stable r15051 <http://