Skip to content

Added Blockquote element#6773

Merged
sfmskywalker merged 2 commits into
devfrom
feature/6772/blockquote-element
Apr 15, 2016
Merged

Added Blockquote element#6773
sfmskywalker merged 2 commits into
devfrom
feature/6772/blockquote-element

Conversation

@sfmskywalker

Copy link
Copy Markdown
Member

For issue #6772

This adds a Blockquote element to the toolbox, making it awesome for users to add beautifully styled block quote elements to their page.

image

@DaRosenberg

DaRosenberg commented Apr 15, 2016

Copy link
Copy Markdown
Member

Tested and reviewed, looks and works fine, both code and functionality.

However, I think we should also add a "citation" property to the element, allowing editor control of the rendered cite attribute:

<blockquote cite="http://developer.mozilla.org">
  <p>This is a quotation taken from the Mozilla Developer Center.</p>
</blockquote>

If the citation is not specified, the cite attribute should be omitted.

@sfmskywalker

Copy link
Copy Markdown
Member Author

Thanks for reviewing! I added the Citation property.

@sfmskywalker
sfmskywalker merged commit 7d60368 into dev Apr 15, 2016
@sfmskywalker
sfmskywalker deleted the feature/6772/blockquote-element branch April 15, 2016 14:27
@sebastienros

Copy link
Copy Markdown
Member

Why not an HTML section instead of content ?

@sfmskywalker

Copy link
Copy Markdown
Member Author

Section, as in a new Category called HTML containing HTML elements? I would not be opposed to that.

sfmskywalker pushed a commit that referenced this pull request Apr 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants