Opened 11 years ago
Closed 11 years ago
#13555 closed defect (fixed)
Fix Module Path Regression in deviceTheme in Mobile Gallery
Reported by: | Ming Zhe Huang | Owned by: | ykami |
---|---|---|---|
Priority: | high | Milestone: | 1.7 |
Component: | DojoX Mobile | Version: | 1.7.0b1 |
Keywords: | mobile-gallery | Cc: | Chris Mitchell |
Blocked By: | Blocking: |
Description
This is a recent regression. Previously when specifying the package name of a theme file, "." is used as separator. However, now "/" is used as separator. So the deviceTheme in mobile gallery does NOT work, and there will be an HTTP 404 error during loading.
The fix is very simple. Just replace "." with "/" in the main HTML page. Please see the attached fix.
Sorry I have to put the fix here, since there's no better component to file against.
Chris, would you please help to check in the patch if necessary? Thanks.
Attachments (1)
Change History (5)
Changed 11 years ago by
Attachment: | mobileGallery-2011-08-02.patch added |
---|
comment:1 Changed 11 years ago by
comment:3 Changed 11 years ago by
Well, I didn't know [25886] had such a side-effect. I fixed deviceTheme.js to allow both '.' and '/' since '.' delimiter was external spec, and people may have already been using it. Hopefully your patch is no longer necessary. Please have a try.
comment:4 Changed 11 years ago by
Milestone: | tbd → 1.7 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Confirmed with Archer.
Proposed patch