Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Conversation

@marcelgerber
Copy link
Contributor

Note you can only ever get into GFM mode using the Language Switcher.
Also added some more fileExtensions for Markdown. Source: https://github.com/github/markup#markups

Note you can only ever get into GFM mode using the Language Switcher.
fileExtensions source: https://github.com/github/markup#markups
@Mark-Simulacrum
Copy link
Contributor

Looks good to me.

@redmunds
Copy link
Contributor

Comment to test gfm comment (edit comment to see)

@Mark-Simulacrum
Copy link
Contributor

@redmunds An interesting part about your comment is that it displays the hidden portion in the email sent:

Comment to test gfm comment <!-- shouldn't see this -->

@redmunds
Copy link
Contributor

@Mark-Simulacrum inline comments don't seem to work -- only block-level comments.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't there be at least 1 file extension?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since GitHub Flavored Markdown is a subset of markdown, I'm not sure if we can/should list any extensions: if we do, they would have to be the same as the (regular) markdown extensions, but that could create conflicts inside Brackets. I'm not sure, but perhaps just leaving it to the user to switch to GitHub Flavored Markdown mode is the best option.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIK GFM has the exact same file extensions Markdown has, and I'm not sure what happens if two modes use the same file extensions.
You can still use GFM using the Language Switcher, where you can define it as default mode as well.

@redmunds redmunds self-assigned this Nov 13, 2014
@redmunds
Copy link
Contributor

Done with code review.

@marcelgerber
Copy link
Contributor Author

Code review comments adressed.
I tried adding fileExtensions (the same Markdown has), but it always takes normal Markdown mode.

@peterflynn
Copy link
Member

Right, you can't have two languages with the same set of extensions. One will just overwrite the other. So as I said above, I don't think this is ideal in terms of discoverability... but I think it's fair to merge it since it's still better than nothing.

@redmunds
Copy link
Contributor

Merging.

redmunds added a commit that referenced this pull request Nov 14, 2014
Add GitHub Flavored Markdown mode, Add some more Markdown fileExtensions
@redmunds redmunds merged commit f93177d into adobe:master Nov 14, 2014
@marcelgerber marcelgerber deleted the gfm-mode branch November 14, 2014 13:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants