Netlify-cms: Can't un-blockquote a quote in editor

[bert-bruggeman]

- Do you want to request a feature or report a bug?
Bug
- What is the current behavior?
Selecting an existing blockquote in the editor doesn't highlight the blockquote icon, after clicking on the icon the editor creates a blockquote in the existing blockquote...

- What is the expected behavior?
the blockquote icon should be highlighted when an existing quote is selected, the blockquote should be un-quoted after clicking it..
- Please mention your CMS, node.js, and operating system version.
Cms 0.6, chrome on fedora linux 23

[erquhart]

@bert-bruggeman having a hard time reproducing - here's what I'm seeing:

[bert-bruggeman]

The issue only appears on existing entries...

[erquhart]

Ah, yep, got it. Looks like it doesn't recognize when a blockquote is active on load for some reason.

[cwahlfeldt]

I'm working on this!

[cwahlfeldt]

It seems that the HTML is changed when viewing a Markdown widget in a post that already has a blockquote. When you make a blockquote using the editor the original html that it produces is:

<blockquote>
  <span>text</span>
</blockquote>

Upon revisiting the post edit page from the dashboard, the html changes to a blockquote with a paragraph as its child instead of a span and thus thinks its a paragraph not a quote.

<blockquote>
  <p>text</p>
</blockquote>

Not sure how it "re-hydrates" data when its viewed again? Also note that this isn't retrieving from a file, I believe its retrieving the data from localstorage or some other storage (localforage?). Still learning a ton about the ENV so bear with me. Cheers!

[erquhart]

@cwahlfeldt so glad you're digging into this!

The storage is actually whatever backend Netlify CMS is using (which currently can only be GitHub for live projects), it loads the entry fresh each time it's opened.

From a high level:

1. the markdown widget is a Slate editor
2. we use Remark as our Markdown AST
3. when a change occurs in the editor, Slate stores it (in it's own Slate AST)
4. we manually create our own Remark AST from that - e.g., quote node is parsed here
5. the Remark AST is the source of truth for the widget and the preview
6. the preview is updated by a Remark module that converts the Remark AST to HTML
7. when we save and persist the entry, the Remark AST is serialized to a Markdown string
8. on entry load, the Markdown is converted to Remark AST by Remark (not manually as before)

I'm betting there's a difference between the Remark AST quote node that we create ourselves and the node that Remark creates when parsing from the Markdown string on load.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.