Opened 10 years ago
Closed 9 years ago
#15078 closed enhancement (patchwelcome)
dojox.fx.highlight is limited to one transparent colour
Reported by: | Nick Fenwick | Owned by: | dante |
---|---|---|---|
Priority: | low | Milestone: | 1.9 |
Component: | Dojox | Version: | 1.7.2 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
dojox.fx.highlight requires you to specify the transparent colour in data-dojo-config, e.g.:
<script src="//ajax.googleapis.com/ajax/libs/dojo/1.7.2/dojo/dojo.js" data-dojo-config="async:true, transparentColor: [255, 255, 255]"></script>
This makes it pretty useless for general purpose highlighting, where elements may be on different colour backgrounds.
See attached test case. Only the example on a white background looks good, because of the [255, 255, 255] in the transparentColor config.
Expected behaviour: dojox.fx.highlight should pick up the element's actual background colour and use that when animating colours. Or, use some kind of alpha or transparency that avoids the need to specify the background colour.
Attachments (1)
Change History (4)
Changed 10 years ago by
Attachment: | dojox.fx.highlight.weirdness.html added |
---|
comment:1 Changed 10 years ago by
Owner: | changed from Adam Peller to dante |
---|---|
Status: | new → assigned |
comment:2 Changed 10 years ago by
See also #6399. Note that (IIUC) the background-color that you need to fade to could come from any ancestor node rather than the node itself.
comment:3 Changed 9 years ago by
Milestone: | tbd → 1.9 |
---|---|
Priority: | undecided → low |
Resolution: | → patchwelcome |
Status: | assigned → closed |
A patch would be welcome to fix this. Otherwise I don't think it will get fixed given the lack of activity.
Simple test case using CDN with three different colour backgrounds.