Skip to content

Conversation

@Tejas2805
Copy link
Contributor

@Tejas2805 Tejas2805 commented Feb 27, 2020

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

• [X] Other, please explain:

Add pageNav to Using Plugins Page

What is the rationale for this request?

The Using Plugins Page is long and having a pageNav will help navigate it.

What changes did you make? (Give an overview)

Added pageNav to the Using Plugins page.

Provide some example code that this change will affect:

<frontmatter>
  title: "User Guide: {{ title }}"
  layout: userGuide
  pageNav: "default"
</frontmatter>

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

N.A.

Testing instructions:

Can check the deploed preview.

* 'master' of https://github.com/MarkBind/markbind:
  Update tests
  Allow using 'none' footer attribute in frontmatter (MarkBind#1002)
  Support line numbers for code blocks (MarkBind#991)
  2.11.0
  Update test files due to changes in PR MarkBind#982
  Update vue-strap version to v2.0.1-markbind.36
  Make highlighting bold (MarkBind#1045)
  Support markdown for header attr in dropdown (MarkBind#1029)
  Add '_site' to the ignored folders in site.json (MarkBind#1046)
  Use path.join instead of string interpolation (MarkBind#1052)
  Implement box markdown header attributes parsing (MarkBind#1025)
  Make the position of top navbar fixed (MarkBind#982)
  Exclude *.md files from being copied over on build (MarkBind#1010)

# Conflicts:
#	docs/css/main.css
@Tejas2805
Copy link
Contributor Author

@marvinchin Can I get your review on this PR? Thanks a lot for your time

Copy link
Contributor

@marvinchin marvinchin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, should help navigating that page a lot!

If you need me to look at something in the future, please explicitly request a review from me :) it makes it much easier to track the stuff I need to look at!

@marvinchin marvinchin added this to the v2.11.1 milestone Feb 28, 2020
@marvinchin marvinchin merged commit 3591bc5 into MarkBind:master Feb 28, 2020
@yamgent yamgent added the pr.DocsUpdate 📃 Pure changes to the documentation, such as typo, restructuring, etc label Mar 3, 2020
@Tejas2805 Tejas2805 deleted the add-pageNav-to-plugin branch March 4, 2020 09:30
nbriannl pushed a commit to nbriannl/markbind that referenced this pull request Mar 7, 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.DocsUpdate 📃 Pure changes to the documentation, such as typo, restructuring, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants