adobe / aem-lib

Apache License 2.0
2 stars 7 forks source link

Move `decorateButtons` to scripts.js? #43

Open shsteimer opened 8 months ago

shsteimer commented 8 months ago

I find the implementation of decorateButtons to be rather opinionated and something that frequently needs to be tweaked or modified on a per project basis depending on how buttons are used, how many different button variations there are, etc. A search of project repos, shows that this is not uncommon, https://github.com/search?q=org%3Ahlxsites+path%3A%2F%28%5E%7C%5C%2F%29scripts%5C.js%24%2F+button&type=code

Anyway, just something that I think may warrant some consideration/discussion of the correct place for this?