? Success

User tests: Successful: Unsuccessful:

avatar oe1tkt
oe1tkt
3 Jan 2014

An invalid attribute within

was found. In addition, I removed several attributes which its use is discouraged in HTML5.
avatar oe1tkt oe1tkt - open - 3 Jan 2014
avatar oe1tkt
oe1tkt - comment - 3 Jan 2014
avatar Bakual
Bakual - comment - 3 Jan 2014

I can understand removing the width from the <tr> tags as they indeed don't make much sense. They probably should have gone to the <td> elements instead.
But what I don't understand is why you want to remove the width and align from the <table>.

Also please can you add test instructions?

avatar Hackwar
Hackwar - comment - 2 Mar 2014

As @oe1tkt correctly states, those attributes are not valid. Neither width nor align are valid attributes for table tags anymore. See here: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/table
So I agree that this is a correct fix. Testing instructions would be to apply this patch and simply see if the pagebreak button below the editor in com_content still works and looks good. ;-)

Tested and works as expected.

avatar infograf768 infograf768 - reference | - 3 Mar 14
avatar infograf768 infograf768 - merge - 3 Mar 2014
avatar infograf768 infograf768 - close - 3 Mar 2014
avatar infograf768 infograf768 - change - 3 Mar 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-03-03 11:28:09
avatar infograf768 infograf768 - close - 3 Mar 2014
avatar Bakual Bakual - reference | - 4 Mar 14
avatar oe1tkt oe1tkt - head_ref_deleted - 4 Mar 2014
avatar oe1tkt
oe1tkt - comment - 4 Mar 2014

thank you all.
Tom, http://f0x.at

avatar Bakual Bakual - reference | ba5b318 - 12 May 14

Add a Comment

Login with GitHub to post a comment