Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#15887 closed defect (invalid)

Tree: Declarative getIconStyle return value appears to be ignored in FireFox

Reported by: tonyerwin Owned by: bill
Priority: undecided Milestone: tbd
Component: Dijit Version: 1.8.0
Keywords: Cc:
Blocked By: Blocking:

Description

For Maqetta, we generate declarative HTML for our users. For Dijit tree, we include script block to override "getIconStyle". It looks like "getIconStyle" is working in Chrome (Mac/Win?), but not Firefox (Mac/Win?).

While it appears that the declarative getIconStyle gets called on both browsers, it seems like nothing is done with the result in FF. I copied the dom elements from both browsers and the styling info is added to Chrome:

` <img src="/maqetta/user/aerwin/ws/workspace/project1/lib/dojo/dojo/resources/blank.gif" alt="" data-dojo-attach-point="iconNode" class="dijitIcon dijitTreeIcon dijitLeaf" role="presentation" style="background-image: url(http://localhost:8081/maqetta/user/aerwin/ws/workspace/project1/samples/sample_data/images/open.gif); "> `

but not FF:

` <img src="/maqetta/user/aerwin/ws/workspace/project1/lib/dojo/dojo/resources/blank.gif" alt="" data-dojo-attach-point="iconNode" class="dijitIcon dijitTreeIcon dijitLeaf" role="presentation"> `

Attachments (2)

getIconStyleTestCaseForBugReport.html (2.4 KB) - added by tonyerwin 7 years ago.
Test case to reproduce
updatedGetIconStyleTest.html (1.8 KB) - added by bill 7 years ago.
self contained test case, working for me on FF

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by tonyerwin

Test case to reproduce

comment:1 Changed 7 years ago by bill

Resolution: invalid
Status: newclosed

You need to say "backgroundImage" not "background-image".

Your test file was referencing some maqetta code, so I removed those parts. And it was also referencing an image file I don't have, so I changed that. See attachment.

Changed 7 years ago by bill

self contained test case, working for me on FF

comment:2 Changed 7 years ago by tonyerwin

Thanks! That makes sense, but guess I was thrown by the fact it worked in Chrome.

comment:3 Changed 7 years ago by bill

Indeed, that is confusing, I think it's gotten a lot of people.

Note: See TracTickets for help on using tickets.