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

add HTML section #11

Merged
merged 3 commits into from
Jan 12, 2017
Merged

add HTML section #11

merged 3 commits into from
Jan 12, 2017

Conversation

waldyrious
Copy link
Contributor

No description provided.

@waldyrious
Copy link
Contributor Author

@templeman ping :)

@waldyrious
Copy link
Contributor Author

@templeman friendly reminder, hoping to catch you around!

@waldyrious
Copy link
Contributor Author

Any news, @templeman?

@templeman
Copy link
Owner

Hi @waldyrious. Thanks for the pull request 😄
It makes sense to point out which generators support HTML output, but I think it's better to identify such entries with a simple HTML tag, rather than create a new category. That way entries from all different categories can be included. Since you submitted this request I've changed the way entries are tagged to hopefully be more useful and searchable.

RoughDraft.js is unique and I think it best belongs under "Tools, Apps, and Extensions".

If you'd care to update your PR then I would gladly merge.

@waldyrious
Copy link
Contributor Author

Just to be sure, does updating the PR mean:

  • adding RoughDraft.js to the "Tools, Apps, and Extensions" section, with the HTML tag
  • adding HTML5-ipsum next to the HTML-ipsum entry, with a HTML tag
  • anything else?

@templeman
Copy link
Owner

That should do it.

@waldyrious
Copy link
Contributor Author

I'm loving github's new features. Conflict resolution in the browser ftw!

@templeman templeman merged commit 8bdff7b into templeman:master Jan 12, 2017
@waldyrious waldyrious deleted the patch-2 branch January 12, 2017 21:18
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.

None yet

2 participants