Allow circle-basic in options.js - improved demo page

dshongphuc / magic-mouse-js

A lightweight javascript library to create some amazing effects for the mouse (cursor) on your website - MagicMouse.js : https://magicmousejs.web.app/

MIT License

389 stars 60 forks source link

Allow circle-basic in options.js - improved demo page #57

Closed alessiochiffi closed 3 months ago

alessiochiffi commented 3 months ago

Changes

Added 'circle-basic' to allowed options. It was missing from allowedStyle object in relation to outerStyle, resulting in an error in the console.
The style change from the dropdown is now immediate on the demo page, without the need of clicking try now.

dshongphuc commented 3 months ago

Hi @alessiochiffi thank you for your fix. But the "circle-basic" is outdated (and the ones from demo page are also outdated). Can you please remove it from allow options, and also change all "circle-basic" in the demo page to "circle" instead? Thanks

alessiochiffi commented 3 months ago

That make sense, I removed circle-basic @dshongphuc 👍