pattern-lab / styleguidekit-assets-default

The static assets for the default StyleguideKit for Pattern Lab. Contains styles and mark-up for Pattern Lab's front-end.
http://patternlab.io/
MIT License
35 stars 67 forks source link

Deprecate jQuery? #78

Open bradfrost opened 6 years ago

bradfrost commented 6 years ago

This is part of a larger effort refactoring of Pattern Lab's dependencies, but currently jQuery is being used intermittently throughout the codebase. This is more a question than anything, but is it possible to remove that jQuery dependency? My gut says yes, but it would be be great to get a more JS-savvy developer to review.

bmuenzenmeyer commented 6 years ago

All of these FE dependencies have a jQuery reference. Not to say it's impossible - just trying to understand the footprint.

sghoweri commented 6 years ago

Yeah... jQuery really shouldn't be needed given what the relatively simple JS requirements of Pattern Lab's Styleguidekit. There's plenty of options available w/o a jQuery dependency that should work just as good if not better!

sghoweri commented 6 years ago

Alternatively, if the navigation was a vertical panel on the side with search at the very top OR if the nav items were horizontal (ala InVision's interface) we could always just combine fuzzy search with filtering the nav items that match up... http://listjs.com/examples/fuzzy-search/

bradfrost commented 6 years ago

@sghoweri thanks for the roundup of non-jQuery options. I'd definitely be on board with migrating away from jQuery to complete those features.

Alternatively, if the navigation was a vertical panel on the side with search at the very top OR if the nav items were horizontal (ala InVision's interface) we could always just combine fuzzy search with filtering the nav items that match up... http://listjs.com/examples/fuzzy-search/

Interesting concept! Let's park that thought for now as I think there might be some other things we might do with search beyond just pattern navigation in the future. But love all these great ideas!

bradfrost commented 6 years ago

TODO: turn these non-jQuery suggestions into actionable tasks