Skip to content

Implement box markdown header attributes parsing - #1025

Merged
yamgent merged 1 commit into
MarkBind:masterfrom
ang-zeyu:box-header-parsing
Feb 22, 2020
Merged

Implement box markdown header attributes parsing#1025
yamgent merged 1 commit into
MarkBind:masterfrom
ang-zeyu:box-header-parsing

Conversation

@ang-zeyu

Copy link
Copy Markdown
Contributor

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

• [x] Enhancement to an existing feature

Resolves#962

Requires MarkBind/vue-strap#118
Follow up to #999

What is the rationale for this request?
To add markdown parsing for box headings

What changes did you make? (Give an overview)

  • Added non-inline markdown parsing for box headers using componentParser
  • Add new unit tests
  • Update an existing test template

Provide some example code that this change will affect:

Markdown parsing for headers in componentParser:

_parseAttributeWithoutOverride(element,'header',false,'_header');// TODO deprecate heading attribute for box_parseAttributeWithoutOverride(element,'heading',false,'_header');

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

Testing instructions:

Proposed commit message: (wrap lines at 72 characters)
Implement box markdown header attributes parsing

@ang-zeyu
ang-zeyuforce-pushed the box-header-parsing branch 3 times, most recently from 6410440 to fb21dd0CompareFebruary 4, 2020 18:48
@marvinchinmarvinchin self-assigned this Feb 11, 2020
@ang-zeyu
ang-zeyuforce-pushed the box-header-parsing branch 2 times, most recently from cc4ff29 to bf0ac14CompareFebruary 15, 2020 05:28

@marvinchinmarvinchin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 🚀

@marvinchinmarvinchin removed their assignment Feb 18, 2020
@yamgentyamgent added this to the v2.10.1 milestone Feb 22, 2020
@yamgent
yamgent merged commit 5a99762 into MarkBind:masterFeb 22, 2020
@damithc

Copy link
Copy Markdown
Contributor

Works. Good work @ang-zeyu and reviewers.

Tejas2805 added a commit to Tejas2805/markbind that referenced this pull request Feb 27, 2020
* '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
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

x of the dismissible boxes it not at the top-right corner

4 participants

@ang-zeyu@damithc@marvinchin@yamgent