[#138] - [#28120] XHTML 1.0 Transitional
- Closed
- 22 Mar 2013
- Medium
- Build: master
- # 138
- Diff
- klauswilms:master
- Foreign ID 28120
User tests: Successful: Unsuccessful:
I made some changes in administrator/components/com_admin/models/sysinfo.php.
if u check the html output of this page in w3c validator you get about 48 errors.
after changing the regex pattern line 146 and 147 and added a opening tr tag at line in administrator/components/com_admin/views/sysinfo/tmpl/default_system.php i got zero errors.
check it :)
greetings from klaus
[#28120] XHTML 1.0 Transitional added
ok there is no response, so I close this pull pull request
That's because all patches still have to go trough the issue tracker (http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8103) for testing before they can be accepted. As infograf768 mentioned you need to open an issue there or your patch won't be accepted.
It's not the most elegant system, but the github issue handling just doesn't cut it for the project.
Thanks for coding this, Klaus! :) This is fixed in JoomlaCode.
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=28120
Closing.
Please create a tracker in joomalcode.