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
chore(masthead): added masthead template #4683
Conversation
Preview: https://patternfly-pr-4683.surge.sh A11y report: https://patternfly-pr-4683-a11y.surge.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fantastic, couple of comments - neither are blockers.
@@ -1,4 +1,4 @@ | |||
{{#> context-selector context-selector--modifier="pf-m-full-height" context-selector--id=(concat masthead--id '-context-selector') context-selector--label-text="Selected project"}} | |||
{{#> context-selector context-selector--modifier="pf-m-full-height" context-selector--id=(concat masthead--id '-context-selector') context-selector--label-text="Selected project" context-selector--IsExpanded=false}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you were using reset
as the word here to unset a context param?
{{#> context-selector context-selector--modifier="pf-m-full-height" context-selector--id=(concat masthead--id '-context-selector') context-selector--label-text="Selected project" context-selector--IsExpanded=false}} | |
{{#> context-selector context-selector--modifier="pf-m-full-height" context-selector--id=(concat masthead--id '-context-selector') context-selector--label-text="Selected project" context-selector--IsExpanded=reset}} |
src/patternfly//components/Form/examples/Form.md: {{#> reset form-field-group--IsExpanded=reset}}
src/patternfly//components/Form/examples/Form.md: {{#> form-field-group form-field-group--id=(concat form--id '-field-group7') form-field-group--IsExpandable=reset}}
src/patternfly//components/Form/examples/Form.md: {{#> form-field-group form-field-group--id=(concat form--id '-field-group8') form-field-group--IsExpandable=reset}}
src/patternfly//components/Nav/templates/nav--drilldown--nav-item-submenu.hbs: {{#> nav-list nav-item--IsDrilldown=reset nav-item--expanded=reset}}
src/patternfly//components/Nav/templates/nav--drilldown--nav-item-submenu.hbs: {{#> nav-list nav-item--expanded=reset nav-item--IsDrilldown=reset}}
src/patternfly//components/Nav/templates/nav--drilldown--nav-item-submenu.hbs: {{#> nav-item nav-item--IsDrilldown=reset nav-item--IsFlyout="true"}}
src/patternfly//demos/Page/page-template-masthead-drilldown.hbs: {{#> menu-content menu-content--attribute=reset}}
src/patternfly//demos/Masthead/templates/masthead-demo--drilldown.hbs: {{#> menu-content menu-content--attribute=reset}}
{{> page-template-skip-to-content}} | ||
{{#> masthead masthead--id=(concat page--id '-masthead')}} | ||
{{> masthead-toggle}} | ||
{{!-- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fwiw with the other page demo templates, I put them in the page component dir so we can keep the demo stuff separate from the example stuff and it makes it easier to find files. Though I don't feel strongly about it if you think there is a better way.
An aside, I'd like to organize all of that better once we can make hbs changes in subdirs and the file watcher reload the workspace properly.
🎉 This PR is included in version 4.178.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
closes #4579
closes #4686
Masthead template applied to Page and Masthead demos.