[TIP] Freshmedia Template
Posted: Sun Jun 14, 2009 7:50 am
Hi!
Link to that template: http://themes.cmsmadesimple.org/index.php?mact=Uploads,cntnt01,default,0&cntnt01category=Full%20Themes&cntnt01sortorder=date_desc&cntnt01mode=single&cntnt01count=10&cntnt01detailpage=theme-detail&cntnt01template=summary&cntnt01detailtemplate=themes-detail&cntnt01upload_id=182&cntnt01returnid=63
As I integrated this template into a project, I found 2 points which could be fixed easily to get better output:
1) check out the template and delete the second time that the div "wrap" is set, that is a mistake, as the name suggests, "wrap" wraps the complete output and should not be set a second time
2) the stylesheet "Freshmedia : Accessibility and cross-browser tools" declares the horizontal ruler as hidden:
that is not useful, as by this all elements with the class "accessibility" PLUS all horizontal rulers are hidden
as the horizontal rule in the accessibility-div is declared with that class, there is no need to declare it here again
change this to
and you have the complete power on visible horizontal rulers in your content, which might be useful ;=)
Link to that template: http://themes.cmsmadesimple.org/index.php?mact=Uploads,cntnt01,default,0&cntnt01category=Full%20Themes&cntnt01sortorder=date_desc&cntnt01mode=single&cntnt01count=10&cntnt01detailpage=theme-detail&cntnt01template=summary&cntnt01detailtemplate=themes-detail&cntnt01upload_id=182&cntnt01returnid=63
As I integrated this template into a project, I found 2 points which could be fixed easily to get better output:
1) check out the template and delete the second time that the div "wrap" is set, that is a mistake, as the name suggests, "wrap" wraps the complete output and should not be set a second time
Code: Select all
<div id="wrap">Code: Select all
/*
accessibility divs are hidden by default
text, screenreaders and such will show these
*/
.accessibility, hr {
position: absolute;
top: -999em;
left: -999em;
}as the horizontal rule in the accessibility-div is declared with that class, there is no need to declare it here again
change this to
Code: Select all
/*
accessibility divs are hidden by default
text, screenreaders and such will show these
*/
.accessibility {
position: absolute;
top: -999em;
left: -999em;
}