Skip to content
This repository has been archived by the owner on Feb 23, 2019. It is now read-only.

Malformed HTML in Generated Item UIs (Admin Pages) #343

Merged
merged 1 commit into from
Feb 2, 2017
Merged

Conversation

amiga-500
Copy link
Collaborator

@amiga-500 amiga-500 commented Feb 2, 2017

When using the Util_Ui::config_item function and setting it's style attribute to 2 (this tells it to span two columns for its container table header), it causes the HTML to be malformed, resulting in the generated <label> tag to be inserted incorrectly, potentially destroying the page layout in less popular (or older) browsers, which might even include Internet Explorer. This fix resolves the problem and the page validates correctly. :octocat:

For those reading and curious what the problem looks like, here is a snapshot -- note the red area:

https_0ssd_com_wp-admin_admin_php_page_w3tc_browsercache_-_2017-02-02_02_10_05

@amiga-500 amiga-500 merged commit 6fdc62b into v0.9.5.x Feb 2, 2017
@amiga-500 amiga-500 deleted the malformed branch February 2, 2017 07:27
@amiga-500 amiga-500 changed the title Malformed HTML in Admin Pages Malformed HTML in Generated Item UIs (Admin Pages) Feb 2, 2017
@nigrosimone
Copy link
Collaborator

Eath time we make a fix i sead a mail to w3tc support for infom it. Just for curiosity, you already send a mail? I want avoid double communications.

@amiga-500
Copy link
Collaborator Author

amiga-500 commented Feb 2, 2017

Hi @nigrosimone . Nope i have never sent them an email nor communicated with them ever. So, all good.

I've only been updating the Wiki page with the fixes, updates, and new features.

I assume you are letting them know about the new features and updates too, not just the bugs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants