includes/styleguide/typography/inline.md - Issue 29491561: Issue 5413 - Create basic style guide for website-defaults

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Side by Side Diff: includes/styleguide/typography/inline.md

Issue 29491561: Issue 5413 - Create basic style guide for website-defaults (Closed)

Patch Set: Apply patch against CMS compatible website-defaults Created Aug. 8, 2017, 3:49 p.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
(Empty)
	1

	2 ### Body text
	juliandoucette 2017/08/09 12:06:54 This seems to correlate to the "Body content" sect This seems to correlate to the "Body content" section of the default-content demo page. If there is no good reason for these to be separate then I think we should use the same include on both pages. ire 2017/08/09 16:50:31 What is the purpose of having them on both pages i Show quoted text On 2017/08/09 12:06:54, juliandoucette wrote: > This seems to correlate to the "Body content" section of the default-content > demo page. If there is no good reason for these to be separate then I think we > should use the same include on both pages. What is the purpose of having them on both pages in the first place? Why not only have the styleguide?
	3

	4 This is normal text

	5

	6 This is emphasized text

	7

	8 This is strong text

	9

	10 This is <small>small</small> text
	juliandoucette 2017/08/09 12:06:54 I left this out of default-content because it's no I left this out of default-content because it's not supported by Markdown and I didn't expect content developers to use it semantically correctly. (I may have been being too harsh. And it may be a good idea to add support for this in Markdown later via a CMS "converter" or "hook") ire 2017/08/09 16:50:31 Acknowledged. Show quoted text On 2017/08/09 12:06:54, juliandoucette wrote: > I left this out of default-content because it's not supported by Markdown and I > didn't expect content developers to use it semantically correctly. > > (I may have been being too harsh. And it may be a good idea to add support for > this in Markdown later via a CMS "converter" or "hook") Acknowledged.
	11

	12 This is a [link](https://example.com/)

	13

	14 This is `inline code`
	juliandoucette 2017/08/09 12:06:54 I left inline-code out of default-content because I left inline-code out of default-content because it wasn't relevant to most of our websites. I thought it was a better idea to include it in a separate module with other advanced features like mark, samp, kbd, and syntax highlighting. ire 2017/08/09 16:50:31 Acknowledged. Show quoted text On 2017/08/09 12:06:54, juliandoucette wrote: > I left inline-code out of default-content because it wasn't relevant to most of > our websites. I thought it was a better idea to include it in a separate module > with other advanced features like mark, samp, kbd, and syntax highlighting. Acknowledged.
	15

	16 This is an abbr

	17

	18 *[abbr]: Abbreviation

	19

	20 This has a footnote[^1]

	21

	22 [^1]: This is a footnote content

OLD	NEW

« includes/styleguide/typography/headings.md ('K') | « includes/styleguide/typography/index.md ('k') | pages/index.html » ('j') | pages/styleguide.html » ('J')