cytoscape / cytoscape.js-cxtmenu

Context menu for Cytoscape.js
MIT License
165 stars 83 forks source link

CSS Variable support for fillColor and activeFillColor #151

Closed mecharmor closed 7 months ago

mecharmor commented 8 months ago

Hi cytoscape.js-cxtmenu team!

I wanted to see if its possible to support CSS variables for the keys: fillColor, and activeFillColor.

This will help enhance the modularity of this extension when trying to apply dynamic themes site-wide. I did however notice that the Command.contentStyle.* color supports CSS variables so i'm guessing that is injecting some native CSS into the page rather than fillColor which i'm guessing parses the colors or something.

Here is a screenshot of what i'm trying to do: image

Thanks!

maxkfranz commented 8 months ago

This sounds like a great idea for an enhancement PR.

mecharmor commented 8 months ago

if I have time i'll try to put something together

stale[bot] commented 7 months ago

This issue has been automatically marked as stale, because it has not had activity within the past 14 days. It will be closed if no further activity occurs within the next 7 days. If a feature request is important to you, please consider making a pull request. Thank you for your contributions.