Opened 9 years ago
Closed 8 years ago
#15931 closed defect (fixed)
[regression] 404 error in dojox/editor/plugins
Reported by: | Douglas Hays | Owned by: | Douglas Hays |
---|---|---|---|
Priority: | undecided | Milestone: | 1.7.5 |
Component: | Dojox | Version: | 1.8.0 |
Keywords: | Cc: | Jared Jurkiewicz, Mike Wilcox | |
Blocked By: | Blocking: |
Description
Starting with [14805], there's a 404 error thrown when using dojox/editor/plugins with a theme other than tundra. This rule was changed from:
.tundra .dijitButtonNode .dijitArrowButtonInner{
to
.dijitButtonNode .dijitArrowButtonInner{
but this was left in:
background:url(../../../../dijit/themes/tundra/images/spriteArrows.png) no-repeat left top;
I don't know if this rule should be removed, or if the background property should be removed or if additional rules for each theme should be added.
Attachments (1)
Change History (10)
comment:1 Changed 9 years ago by
Cc: | Jared Jurkiewicz Mike Wilcox added |
---|---|
Owner: | Adam Peller deleted |
Status: | new → assigned |
comment:2 Changed 9 years ago by
comment:3 Changed 9 years ago by
Owner: | set to Mike Wilcox |
---|---|
Summary: | 404 error in dojox/editor/plugins → [regression] 404 error in dojox/editor/plugins |
I don't understand why it gets a 404.
comment:4 Changed 9 years ago by
Any update on this? The 404 error comes since tundra theme is not there in standard package since dojo 1.7.
comment:5 Changed 8 years ago by
Milestone: | tbd → 1.9 |
---|
bill, can you review the attached patch? I didn't see any reason for the wayward rule in editorPlugins.css. dijit.css was being imported AFTER claro.css in a couple of test HTML files causing the .dijitButtonNode .dijitArrowButtonInner selector in dijit.css to have a higher priority than the theme specific rule for .claro .dijitArrowButtonInner.
comment:6 Changed 8 years ago by
I'm testing via dojox/editor/tests/editorTablePlugs.html?theme=claro using the dropdown arrow in the editor toolbar.
comment:7 Changed 8 years ago by
@doughays - At a glance that patch looks good to me. I don't see why that CSS code was there in the first place either.
comment:8 Changed 8 years ago by
Milestone: | 1.9 → 1.7.5 |
---|---|
Owner: | changed from Mike Wilcox to Douglas Hays |
error occurs in dojox/editor/plugins/resources/editorPlugins.css