Opened 12 years ago

Closed 12 years ago

#4735 closed defect (fixed)

Errors in Testcase HTML

Reported by: Jared Jurkiewicz Owned by:
Priority: high Milestone: 1.0
Component: Dijit Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

No start tag (<h3>). dijit/tests test_InlineEditBox.html line 59 No start tag (<h3>). dijit/tests/form test_InlineEditBox.html line 57 No start tag (<h3>). dijit/tests/form test_InlineEditBox.html line 57

As determined by Eclipse HTML validator.

Change History (14)

comment:1 Changed 12 years ago by Jared Jurkiewicz

There are others as well (Multiple problems):

No start tag (<p>). dijit/bench test_button-results.html line 66

No start tag (<p>). dijit/tests/form test_ComboBox_destroy.html line 40

No start tag (<p>). dijit/tests/form test_FilteringSelect.html line 47

No start tag (<p>). dijit/tests/form test_FilteringSelect.html line 54

No start tag (<p>). dijit/tests/form test_Slider.html line 104

No start tag (<p>). dijit/bench test_button-results.html line 66

No start tag (<p>). dijit/tests test_Tooltip.html line 41

No start tag (<p>). dijit/tests/form test_ComboBox_destroy.html line 40

No start tag (<p>). dijit/tests/form test_ComboBox_destroy.html line 40

No start tag (<p>). dijit/tests/form test_FilteringSelect.html line 47

No start tag (<p>). dijit/tests/form test_FilteringSelect.html line 54

No start tag (<p>). dijit/tests/form test_Slider.html line 98

comment:2 Changed 12 years ago by Jared Jurkiewicz

And yet more: :-)

No start tag (<p>). TestDojoWar/WebContent?/dijit/tests/i18n currency.html line 223

No start tag (<p>). TestDojoWar/WebContent?/dijit/tests/i18n currency.html line 232

No start tag (<p>). TestDojoWar/WebContent?/dijit/tests/i18n currency.html line 248

comment:3 Changed 12 years ago by Jared Jurkiewicz

(In [10833]) Fixing HTML errors. refs #4735

comment:4 Changed 12 years ago by Jared Jurkiewicz

Apparently, the Eclipse validator doesn't like the nesting of those in <p>. Why, I don't know. Anyone know how to shut up Eclipse on these? This tends to cause problems when using dojo in a WAR.

comment:5 Changed 12 years ago by Jared Jurkiewicz

(In [10848]) More HTML fixes in the test cases. refs #4735

comment:6 Changed 12 years ago by Jared Jurkiewicz

(In [10849]) Fix to currency test markup. refs #4735

comment:7 Changed 12 years ago by Jared Jurkiewicz

(In [10850]) Fix to tooltip. refs #4735

comment:8 Changed 12 years ago by Jared Jurkiewicz

(In [10851]) Fix for misclosed div. refs #4735

comment:9 Changed 12 years ago by Jared Jurkiewicz

(In [10852]) More invalid HTML fixes. refs #4735

comment:10 Changed 12 years ago by Jared Jurkiewicz

(In [10853]) Fix to io proxy test. refs #4735

comment:11 Changed 12 years ago by bill

Summary: Errors in Testcase HTML. (test_InlineEditBox.html)Errors in Testcase HTML

I see, Eclipse complains if you have an <ol> or <ul> or <div> inside a <p>. I guess that makes sense since they are block level elements.

comment:12 Changed 12 years ago by Jared Jurkiewicz

(In [10854]) Fix for extra script close tag. refs #4735

comment:13 Changed 12 years ago by Jared Jurkiewicz

Bill,

Yep, it's technically invalid HTML to do that according to spec. Eclipse complains loudly about it too, and makes dojo look like it has 40+ errors in its files. Hence why I'm fixing all these.

comment:14 Changed 12 years ago by Jared Jurkiewicz

Resolution: fixed
Status: newclosed

Fixed all of the current complains from Eclipse Validator regarding test files.

Note: See TracTickets for help on using tickets.