Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

wip: table of contents draft #882

Closed
wants to merge 1 commit into from
Closed

Conversation

0x-r4bbit
Copy link
Contributor

No description provided.

@0x-r4bbit
Copy link
Contributor Author

@wardbell you'll notice that all i've done is adding a heading for the section list. I think I was lucky that the template syntax guide is already strucutred in a way that it works very well. To me it feels like add the "Table Of Contents" header makes a huge difference.

Of course, it'll take a bit more work to apply that to other chapters.

@@ -29,9 +31,6 @@ include ../../../../_includes/_util-fns
* [Template expression operators](#expression-operators)
// #enddocregion intro

:marked
[Run the live example](/resources/live-examples/template-syntax/ts/plnkr.html)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Notice that I've removed the "Live example" here because.... Which live example are we even talking about? I find this rather misplaced.

Copy link
Contributor

Choose a reason for hiding this comment

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

Disagree about removing the example which is enormously valuable in exploring the ideas expressed here and closely tracks the ToC here.

But I do agree that the purpose should be made clear. Doing that.

@wardbell
Copy link
Contributor

wardbell commented Mar 8, 2016

Incorporating this change. Opened companion issue #935

@wardbell wardbell closed this in 5619356 Mar 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants