Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ability to strip pattern type category prefixes from main menu nav #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bradwade
Copy link

This just cleans up the main menu navigation if you have to prefix your pattern type categories.

Pattern type category prefixes can be defined in config.yml as an array named "prefixes"
containing strings of prefixes. For example if your pattern types are called,
"theme1_atoms", "theme1_molecules", etc. you can add to your config.yml:

prefixes

  • "theme1_"

...then theme1_ will not appear in the main menu navigation of pattern lab.

Needing to prefix the top level pattern type directories is a common use case when working with multiple pattern lab instances integrated with a single Drupal 8 installation for example.

…navigation.

Pattern type category prefixes can be defined in config.yml as an array named "prefixes"
containing strings of prefixes. For example if your pattern types are called,
"theme1_atoms", "theme1_molecules", etc. you can add to your config.yml:

prefixes
  - theme1_

...then theme1_ will not appear in the main menu navigation of pattern lab.: 'upstream' does not appear to be a git repository
fatal: Could not read from remote repository.

Added ability to strip pattern type category prefixes from main menu navigation.

Pattern type category prefixes can be defined in config.yml as an array named "prefixes"
containing strings of prefixes. For example if your pattern types are called,
"theme1_atoms", "theme1_molecules", etc. you can add to your config.yml:

prefixes
  - theme1_

...then theme1_ will not appear in the main menu navigation of pattern lab.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant