cuyahoga-tap / cuyasurvey-drupal

Cuyahoga County customer feedback mobile site in Drupal
GNU General Public License v2.0
2 stars 2 forks source link

Adherence to W3C mobileOK Checker specs #6

Open skorasaurus opened 12 years ago

fndtn357 commented 12 years ago

it is definitely getting much better - not quite there yet. http://validator.w3.org/mobile/check?async=false&docAddr=http%3A%2F%2Fprototypesurvey.info

fndtn357 commented 12 years ago

Embedded resources must be retrieved for the page to be rendered correctly. Triggered by http://cuyasurvey.websubstrate.com/sites/all/libraries/jquery.mobile/images/ajax-loader.png. Triggered by http://cuyasurvey.websubstrate.com/sites/all/libraries/jquery.mobile/images/icons-18-black.png. Triggered by http://cuyasurvey.websubstrate.com/sites/all/libraries/jquery.mobile/images/icons-18-white.png. Triggered by http://cuyasurvey.websubstrate.com/sites/all/libraries/jquery.mobile/images/icons-36-black.png. Triggered by http://cuyasurvey.websubstrate.com/sites/all/libraries/jquery.mobile/images/icons-36-white.png. Triggered by http://cuyasurvey.websubstrate.com/sites/all/libraries/jquery.mobile/jquery.mobile.structure-1.0.css.

and it looks like both locations report the same issue - Triggered by http://prototypesurvey.info/sites/all/libraries/jquery.mobile/images/ajax-loader.png. Triggered by http://prototypesurvey.info/sites/all/libraries/jquery.mobile/images/icons-18-black.png. Triggered by http://prototypesurvey.info/sites/all/libraries/jquery.mobile/images/icons-18-white.png. Triggered by http://prototypesurvey.info/sites/all/libraries/jquery.mobile/images/icons-36-black.png. Triggered by http://prototypesurvey.info/sites/all/libraries/jquery.mobile/images/icons-36-white.png. Triggered by http://prototypesurvey.info/sites/all/libraries/jquery.mobile/jquery.mobile.structure-1.0.css.

w3 says - these links are broken... there aren't any images on the dev site - in locations specified or otherwise

looks like we are not calling the right style sheet images exist in jquery.custom

fndtn357 commented 12 years ago

severe, not critical, issue number 1 - size of page and I am thinking this report (which is related) is fairly normal with the new jquery-mobile themes. what do you guys think? see this link for results: http://jigsaw.w3.org/css-validator/validator?uri=http%3A%2F%2Fcuyasurvey.websubstrate.com%2F&profile=css21&usermedium=all&warning=1&vextwarning=&lang=en

I am thinking a lot of these issues on css validation are directly related to using css3. I mean, should we be providing opacity in this instance [rgba(0,0,0,.4)] instead of what is called out?

How far should we clean up?

jeffschuler commented 12 years ago

Just added the mobile structure file. I believe we have those images in the jquery.custom directory if you want to copy them into jquery.mobile.

fndtn357 commented 12 years ago

we need to convert all our images into gif and jpeg format for the mobile experience.

fndtn357 commented 12 years ago

document markup size is about 14% larger than suggested.

jeffschuler commented 12 years ago

can you decide what you think the priorities are here and break them into separate issues?

jeffschuler commented 12 years ago

I changed over to the minified versions of all of the jquery.mobile CSS & JS files, and turned CSS+JS aggregation on. That cut the sizes of those down by about 50%. We might be able to trim a little more, but I'm pretty confident we're not going to get in under 20KB. Wasn't able to turn aggregation on on the prototypesurvey.info site for some reason...

fndtn357 commented 12 years ago

thanks for your responsiveness. i will segregate and approach each issue individually. I can't seem to see the base font size here - 13px?

jeffschuler commented 12 years ago

Any progress here? Would be great to break these out into separate issues so folks could tackle them individually.