Skip to content
This repository was archived by the owner on May 5, 2021. It is now read-only.

Do Not Merge: Proposed idea around code partial #26

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Do Not Merge: Proposed idea around code partial #26

wants to merge 4 commits into from

Conversation

willium
Copy link

@willium willium commented Sep 8, 2015

I want to present my 'example code' sans any grid I implement. within my code I have an extra stylesheet where I added a grid. for presenting icons on the style guide. Of course, they shouldn't be shown to people who are implementing the style guide (as they aren't available within the compressed css). Therefore I needed a separate partial for only showing the code. My f-item-content looks like

{{#if notes}}<div class="f-item-notes" data-f-toggle="notes">
    {{{notes}}}
</div>{{/if}}
<div class="f-item-preview">
    {{{material @key "preview" @root}}}
</div>
<div class="f-item-code f-item-hidden" data-f-toggle="code">
    <pre><code class="language-markup">{{material @key "code" @root}}</code></pre>
</div>

to better understand. This isn't implemented well, however, so I don't propose this change be merged. However, I'd be interested to know about a potential proper implementation of this kind of use-case within fabricator.

@LukeAskew
Copy link
Member

Hey!

I'm not totally sure what functionality this adds. Do you mind posting a more concrete example of how you're using this functionality?

@willium
Copy link
Author

willium commented Sep 10, 2015

Hey @LukeAskew, this gist might make it more clear

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants