Closed Bug 1293226 Opened 7 years ago Closed 2 years ago

Broken HTML in translation string


(Firefox for Android Graveyard :: General, defect, P5)



(Not tracked)



(Reporter: stoyan, Unassigned)



(Whiteboard: [TPE-1])


(1 file)

The entity is "malformedURI.longDesc2" and the HTML content of it is not valid HTML. The error from the validator is "Element div not allowed as child of element ul in this context."
Blocks: 1042199
Entity "dnsNotFound.longDesc4" is broken the same way.
Priority: -- → P1
Whiteboard: [TPE-1]
This seems to have been added in 2014 in Bug 1042199:

I'm slightly worried that changing this would cause translation churn.
The patch I've uploaded would seem to fix this (based on playing around with the w3 validator), however I haven't tested in a local build yet.

As mentioned I'm worried about the translation churn, especially seeing as the current code works despite being non-compliant - do we want to continue with this?
Let's bring in flod to understand if updating this string is a problem.

(In reply to Andrzej Hunt :ahunt from comment #4)
> The patch I've uploaded would seem to fix this (based on playing around with
> the w3 validator), however I haven't tested in a local build yet.

Okay, we should make sure that everything's working correctly by just triggering this error page. And we should try to keep the indentation correct/clean. :)
Flags: needinfo?(francesco.lodolo)
Delphine owns mobile l10n, it's her decision ;-)

Updating the string would require localizers to "re-translate" it, since the source changes. The point to evaluate is: how bad is to have broken HTML in an internal page (not exposed to other browsers, and working as expected in Firefox)?
Flags: needinfo?(francesco.lodolo) → needinfo?(lebedel.delphine)
Thanks for forwarding flod. I think it makes sense to correct this
Flags: needinfo?(lebedel.delphine)
Can you take care of getting this tested and landed?
Flags: needinfo?(ahunt)
The patch as it is it's not OK: if you want other locales to fix the HTML you will need a new string ID.

That's unless we don't care about locales keeping the broken HTML.
Flags: needinfo?(ahunt)
[triage] Given engineering resources, only critical bugs will remain P1.
Keywords: stale-bug
Priority: P1 → P3
Re-triaging per

Needinfo :susheel if you think this bug should be re-triaged.
Priority: P3 → P5
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly]( an issue can be reported at the [Fenix GitHub project]( If you want to discuss your report please use [Mozilla's chat]( server and join the [#fenix]( channel.
Closed: 2 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.