Skip to content

Improve export consistency between HTML and PDF types#761

Merged
Rokt33r merged 1 commit intoBoostIO:masterfrom
Komediruzecki:feature/improve-export-consistency
Mar 20, 2021
Merged

Improve export consistency between HTML and PDF types#761
Rokt33r merged 1 commit intoBoostIO:masterfrom
Komediruzecki:feature/improve-export-consistency

Conversation

@Komediruzecki
Copy link
Copy Markdown
Contributor

@Komediruzecki Komediruzecki commented Jan 9, 2021

Improve export consistency between HTML and PDF types (#754)

  • Add remark-admonitions for html generation
  • Update metadata for html generation
  • Refactor HTML export to use custom HTML generated previously for PDF only
  • Refactor export functions
  • Replace preferences object with actual needed preferences
  • Update conversion functions for saving note document with shortcut
  • Export css attachments for HTML
  • Export css stylesheets
  • Link css stylesheets in HTML string for HTML export
  • Refactor PDF links update in markdown (remove HTML handling)
  • Refactor shortcut saving HTML and Markdown
    • Add saveFolder and saveName
  • Update logic for production paths (use API and dev variable)
  • Add katex fonts exporting and katex.min.css instead of jsdeliver link

Error handling:

  • When writing note attachment or asset fails, it is skipped
  • User is notified of missing assets in console (consider pushMessage?)

Preview:
The export structure as described in #754
ExportsWithKatexFonts

HTML rendering of admonitions:
HTML_RenderingAdmonitions

Exported HTML and then printed PDF in Firefox (emojis work):
MarkdownFunctionalityFirefoxRendering.pdf
HTML & Assets:
Markdown Functionality Test.zip

Tests:

  • Tried exporting with shortcut ctrl+s with type Markdown, PDF and HTML

  • Tried exporting with export buttons with type Markdown, PDF, and HTML

    • HTML is exported with assets (CSS files, CSS fonts, and attachments if any)
    • Markdown is exported only with attachments (no need for CSS)
    • PDF is exported as one file (embedded with attachments, stylesheets are rendered)
  • In electron Linux App (dev)

  • In electron Linux App production version (appImage)

@Komediruzecki Komediruzecki requested a review from Rokt33r January 9, 2021 13:48
@Komediruzecki Komediruzecki added the awaiting review ❇️ Pull request is awaiting a review. label Jan 9, 2021
Add remark-admonitions for html generation
Update metadata for html generation
Refactor HTML export to use custom HTML generated previously for PDF only
Refactor export functions
Replace preferences object with actual needed preferences
Update conversion functions for saving note document with shortcut
Export css attachments for HTML
Export css stylesheets
Link css stylesheets in HTML string for HTML export
Refactor PDF links update in markdown (remove HTML handling)
Refactor shortcut saving HTML and Markdown
 - Add saveFolder and saveName
Update logic for production paths (use API and dev variable)
Add katex fonts exporting and katex.min.css instead of jsdeliver link
@Komediruzecki Komediruzecki force-pushed the feature/improve-export-consistency branch from 560c34c to 9254c9a Compare March 16, 2021 21:47
@Rokt33r Rokt33r merged commit 2bd1035 into BoostIO:master Mar 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review ❇️ Pull request is awaiting a review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants