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

Render plugins on dynamic resources #1051

Merged
merged 6 commits into from
Mar 3, 2020

Conversation

openorclose
Copy link
Contributor

@openorclose openorclose commented Feb 21, 2020

What is the purpose of this pull request? (put "X" next to an item, remove the rest)

• [x] Bug fix?
• [x] New feature?

Fixes #1032

Fixes #1050

What is the rationale for this request?

What changes did you make? (Give an overview)

Render plugins on dynamically included content.

Provide some example code that this change will affect:

//index.md
<panel src="inc.md"></panel>
<panel><span heading><b>header</b></span> Content of panel</panel>



//inc.md
<panel><span heading><b>header</b></span> Content of panel</panel>

Is there anything you'd like reviewers to focus on?

Reasons for not rendering plugins

Testing instructions:

Proposed commit message: (wrap lines at 72 characters)

Render plugins on dynamically included content

Plugins are not rendered on dynamically included content.

Let's do so by calling the prerender and postrender functions on them.

@openorclose
Copy link
Contributor Author

Edited the main page to demo:

https://5e4f850363f97500071b045f--markbind-master.netlify.com/

@ang-zeyu
Copy link
Contributor

LGTM, does plugin source live reloading and asset collection work correctly as well for dynamic includes?

@openorclose
Copy link
Contributor Author

LGTM, does plugin source live reloading and asset collection work correctly as well for dynamic includes?

Yup just tested it works fine

@yamgent yamgent added this to the v2.11.1 milestone Mar 2, 2020
@yamgent yamgent added the pr.BugFix 🐛 Fixes correct a programming error/assumption label Mar 3, 2020
@yamgent yamgent merged commit a0ae237 into MarkBind:master Mar 3, 2020
Tejas2805 added a commit to Tejas2805/markbind that referenced this pull request Mar 7, 2020
…nvert-to-code-block

* 'master' of https://github.com/MarkBind/markbind:
  Allow changing parameter properties (MarkBind#1075)
  Custom timezone for built-in timestamp (MarkBind#1073)
  Fix reload inconsistency when updating frontmatter (MarkBind#1068)
  Implement an api to ignore content in certain tags (MarkBind#1047)
  Enable AppVeyor CI (MarkBind#1040)
  Add heading and line highlighting to code blocks (MarkBind#1034)
  Add dividers and fix bug in siteNav (MarkBind#1063)
  Fixed navbar no longer covers modals (MarkBind#1070)
  Add copy code-block plugin (MarkBind#1043)
  Render plugins on dynamic resources (MarkBind#1051)
  Documentation for Implement no-* attributes for <box>  (MarkBind#1042)
  Migrate to bootstrap-vue popovers (MarkBind#1033)
  Refactor preprocess and url processing functions (MarkBind#1026)
  Add pageNav to Using Plugins Page (MarkBind#1062)

# Conflicts:
#	docs/userGuide/syntax/siteNavigationMenus.mbdf
Tejas2805 added a commit to Tejas2805/markbind that referenced this pull request Mar 9, 2020
* 'master' of https://github.com/MarkBind/markbind:
  2.12.0
  Update outdated test files
  Update vue-strap version to v2.0.1-markbind.37
  Fix refactor to processDynamicResources (MarkBind#1092)
  Implement lazy page building for markbind serve (MarkBind#1038)
  Add warnings for conflicting/deprecated component attribs (MarkBind#1057)
  Allow changing parameter properties (MarkBind#1075)
  Custom timezone for built-in timestamp (MarkBind#1073)
  Fix reload inconsistency when updating frontmatter (MarkBind#1068)
  Implement an api to ignore content in certain tags (MarkBind#1047)
  Enable AppVeyor CI (MarkBind#1040)
  Add heading and line highlighting to code blocks (MarkBind#1034)
  Add dividers and fix bug in siteNav (MarkBind#1063)
  Fixed navbar no longer covers modals (MarkBind#1070)
  Add copy code-block plugin (MarkBind#1043)
  Render plugins on dynamic resources (MarkBind#1051)
  Documentation for Implement no-* attributes for <box>  (MarkBind#1042)
  Migrate to bootstrap-vue popovers (MarkBind#1033)
  Refactor preprocess and url processing functions (MarkBind#1026)
  Add pageNav to Using Plugins Page (MarkBind#1062)
marvinchin pushed a commit that referenced this pull request Apr 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr.BugFix 🐛 Fixes correct a programming error/assumption
Projects
None yet
Development

Successfully merging this pull request may close these issues.

puml files not rendered under some conditions Plugins not used on included content
3 participants