Opened 8 years ago

Closed 8 years ago

#13315 closed defect (fixed)

404 errors loading *-compat.css files

Reported by: bill Owned by: ykami
Priority: high Milestone: 1.7
Component: DojoX Mobile Version: 1.7.0b1
Keywords: 1.7-mobile Cc: ykami
Blocked By: Blocking:

Description

Loading mobileGallery demo (I tried firefox) gives these 404 errors (I'm trying in FF5 if that makes a difference):

kernel.js (line 457)
"NetworkError: 404 Not Found - http://localhost/clean/demos/mobileGallery/demo-compat.css"
demo-compat.css
"NetworkError: 404 Not Found - http://localhost/clean/demos/mobileGallery/css/themes/iphone/gallery-compat.css"
galler...pat.css
"NetworkError: 404 Not Found - http://localhost/clean/demos/mobileGallery/images/MobileShow_App_57-compat.css"
Mobile...pat.css
"NetworkError: 404 Not Found - http://localhost/clean/demos/mobileGallery/images/MobileShow_App_114-compat.css"
Mobile...pat.css
"NetworkError: 404 Not Found - http://localhost/clean/demos/mobileGallery/images/MobileShow_Fav_16-compat.css"

Happens on source or on build. Not sure who is requesting those non-existant files.

Change History (6)

comment:1 Changed 8 years ago by bill

Summary: mobileGallery 404 errorsmobileGallery demo 404 errors

comment:2 Changed 8 years ago by ykami

dojox.mobile's compatibility module (dojox.mobile._compat) automatically loads *-compat.css for each loaded css file. This never happens on mobile. Please just ignore this. It is working as designed.

comment:3 Changed 8 years ago by ykami

This never happens on mobile.

More precisely, this never happens on webkit-based browsers, which do not need compat.

comment:4 Changed 8 years ago by bill

Summary: mobileGallery demo 404 errors404 errors loading *-compat.css files

Oh OK. But do you think it would make sense to limit (trying to) load the *-compat.css modules to dojo.isIE < 10? I doubt they are needed for the new firefox 5, although I haven't checked.

It's just that we got many many bug reports from the 404 errors in the working-as-designed NLS code (trying to load en-us and en files that didn't exist).

Anyway, feel free to close this ticket as invalid.

comment:5 Changed 8 years ago by ykami

Keywords: 1.7-mobile added
Milestone: tbd1.7
Owner: changed from Chris Mitchell to ykami
Status: newassigned

I was wrong. It was not an expected behavior. By default, request should be made only for *-compat.css files that correspond to mobile/themes/...*.css. There was a bug in loadCompatCssFiles() in _compat.js.

comment:6 Changed 8 years ago by ykami

Resolution: fixed
Status: assignedclosed

(In [25643]) Fixes #13315 !strict Fixed the condition that checks location.href in loadCompatCssFiles().

Note: See TracTickets for help on using tickets.