Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove em tag from "Lists" in Demo post content #1558

Closed

Conversation

dixitadusara
Copy link
Contributor

As per my suggestion, em tag not suitable for "Lists" word.

@dixitadusara dixitadusara changed the title remove em tag from Lists Remove em tag from Lists Jun 28, 2017
@dixitadusara dixitadusara changed the title Remove em tag from Lists Remove em tag from "Lists" from Demo post content Jun 29, 2017
@dixitadusara dixitadusara changed the title Remove em tag from "Lists" from Demo post content Remove em tag from "Lists" in Demo post content Jun 29, 2017
@dixitadusara
Copy link
Contributor Author

@jasmussen

Please review this PR and let me know why "Lists" word is different from the other words in The Inserter Tool.

@jasmussen
Copy link
Contributor

@mtias, any thoughts on this one? I've no strong opinion.

@mtias
Copy link
Member

mtias commented Jun 29, 2017

It was to showcase inline formatting on lists and to make sure it remains working correctly.

@dixitadusara
Copy link
Contributor Author

Ok, i got it.

@ellatrix
Copy link
Member

ellatrix commented Jul 3, 2017

Let's close then, if it's there intentionally.

@ellatrix ellatrix closed this Jul 3, 2017
dratwas pushed a commit that referenced this pull request Nov 14, 2019
….1_to_develop

Merge master 1.16.1 to develop
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.

4 participants