Closed Bug 980448 Opened 11 years ago Closed 11 years ago

modify-account section missing closing tag

Categories

(Firefox OS Graveyard :: Gaia::Calendar, defect)

x86
macOS
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
1.4 S6 (25apr)

People

(Reporter: Eli, Assigned: mmedeiros)

Details

(Whiteboard: [p=1])

Attachments

(1 file)

The `modify-account` section at the end of the calendar index.html markup is missing its closing tag. https://github.com/mozilla-b2g/gaia/blob/master/apps/calendar/index.html#L171
Assignee: nobody → mmedeiros
Whiteboard: [p=1]
Target Milestone: --- → 1.4 S6 (25apr)
Attachment #8406879 - Flags: review?(gaye) → review+
This is actually the second instance of us creating invalid html in the calendar app. Perhaps we should have some html validation marionette plugin thing that can tap into the integration tests to verify that the html is always compliant...
Such errors wouldn't be reflected into the DOM which will have corrected things using the HTML5 parser rules. You'd need to do the checking at a syntactic level, I think. Since Calendar I think uses templates, that may be trickier than in something like e-mail where the r.js build step or runtime loader would be able to sanity-check HTML as it optimizes/loads.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: