Skip to content

Html converter - #194

Open
nalbion wants to merge 9 commits into
memspace:masterfrom
nalbion:html_converter
Open

Html converter#194
nalbion wants to merge 9 commits into
memspace:masterfrom
nalbion:html_converter

Conversation

@nalbion

@nalbion nalbion commented Oct 27, 2019

Copy link
Copy Markdown

No description provided.

@codecov

codecov Bot commented Oct 27, 2019

Copy link
Copy Markdown

Codecov Report

Merging #194 into master will decrease coverage by 2.24%.
The diff coverage is 82.39%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #194      +/-   ##
==========================================
- Coverage   90.63%   88.38%   -2.25%     
==========================================
  Files          12       13       +1     
  Lines        1089     1498     +409     
==========================================
+ Hits          987     1324     +337     
- Misses        102      174      +72
Impacted Files Coverage Δ
packages/notus/lib/src/document/attributes.dart 75% <ø> (ø) ⬆️
packages/notus/lib/convert.dart 0% <0%> (ø) ⬆️
packages/notus/lib/src/convert/html.dart 82.59% <82.59%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7f102b9...1db03aa. Read the comment docs.

@JEuler

JEuler commented Dec 18, 2019

Copy link
Copy Markdown

Do you have plans to merget it (also)? Thx for you great editor!

@chocovish

Copy link
Copy Markdown

There is problem with this. if a line is bulleted (unordered list) and that line contains different styles per word like

<ul>
<li> This <i>is</i> <b>bold</b> </li>
</ul>

it gives output like

  • This
  • is
  • bold

@stuartwk

Copy link
Copy Markdown

Any progress on this? Looking to convert zefyr to html.

@nalbion

nalbion commented May 28, 2020

Copy link
Copy Markdown
Author

@chocovish Delta doesn't support nested formatting - but that could change...
slab/delta#48

Sign up for free to 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.

6 participants