Skip to content
This repository has been archived by the owner on May 1, 2020. It is now read-only.

Including Patterns in Header & Footer #24

Closed
dmolsen opened this issue Jul 11, 2016 · 6 comments
Closed

Including Patterns in Header & Footer #24

dmolsen opened this issue Jul 11, 2016 · 6 comments

Comments

@dmolsen
Copy link
Member

dmolsen commented Jul 11, 2016

A user wants to include an atom or other pattern as part of their global header and/or footer.


I believe this is an issue that only affects PL/PHP. I wanted to bring it up as a spec change just in case. This way we'll know how each port renders the header and footer. PL/PHP renders the header and footer as strings so it can't load patterns/partials. This was done for performance reasons. I will try to find a way to minimize the impact of using the full pattern loader over and over again for really simple content.

System Output

When rendered a full pattern's header and footer will include content referenced by includes.

Extra

The timeline for this feature is near term just because I think it's an easy change.

Tagging core and spec-question because it'll be implemented in core and i'm unsure if PHP is the only one not doing this.

This vote will close at 12PM on July 25 or once two voting members have voted yay or nay.

/cc @pattern-lab/voting-members

@dmolsen
Copy link
Member Author

dmolsen commented Jul 11, 2016

👍

@bmuenzenmeyer
Copy link
Member

I'd need to try it in PL/Node - but I think it won't work either - although all the pieces are in place.

Makes sense to me - as long as people understand the ramifications of adding something to every pattern.

👍

@alkaithil
Copy link

Hi, I'm trying to include a pattern in the footer and it seems it's not working. I stumbled into this page and it seems it's not yet implemented. Can someone help me with this issue? Is it going to be implemented any time soon? Any other workarounds I could use? Thanks!

@bmuenzenmeyer
Copy link
Member

Hello @alkaithil which tool are you using?

@alkaithil
Copy link

Patternlabs PHP. I know it's marked as solved but I can't make it work.

@bmuenzenmeyer
Copy link
Member

bmuenzenmeyer commented Aug 2, 2017 via email

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

3 participants