Remove "view all" code about patternlab-node HOT 6 CLOSED

Any plans to add the "view all" feature back in?

from patternlab-node.

Hi @heyitsolivia this feature was removed as part of patternlab-php, the official reference implementation of patternlab before I ever had a chance to fully implement it. pl-node does have the "ALL" section however.

I am open to including this as a flag - but it would require me revisiting the code anyways as it was never as functional as I wanted.

Thoughts?

from patternlab-node.

@bmuenzenmeyer -

What was the purpose of the feature? Maybe it's something I need to add to the spec.

from patternlab-node.

@dmolsen Actually, I just double checked the pl-php and sure enough it's there:

I never got support for this working due to some messy code and eventually removed it. I must have been mistaken somewhere along the way because I thought it was removed from pl-php and therefore I abandoned. This should be reopened now as a gap I see. Thanks @heyitsolivia!

I want to devote more time to pl-node, and this should be one more paver on the way to compliance :)

from patternlab-node.

@bmuenzenmeyer If you like, I can take a crack at rewriting the code for the View All

from patternlab-node.

@heyitsolivia I'd love more contributions - there is a decent amount of change in dev right now.

from patternlab-node.