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

docs:(cookbook showing how to create a TreeView) #1145

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

thelgevold
Copy link
Contributor

@wardbell
Here is the TreeView cookbook.

I decided to add an http request as part of this. Mainly because it's the number one ask from people who have been commenting on other TreeViews I've made in the past. It also makes it easier to vary the data since it's relatively painful to hand code tree objects. I linked to the Http Dev guide for the http configurations.

@thelgevold
Copy link
Contributor Author

Updated outdated PR to rc 1

@thelgevold
Copy link
Contributor Author

Updated to use core-js shim

@thelgevold
Copy link
Contributor Author

rebased latest master

working on getting tree view rendering

wiring up tree

more features

styles

added add/cancel

styles

http

fixed styling

added error handling

docs

clean-up

starting article

img

text

updated

e2e

text

text

text

renamed rating

fixed test

typo

update to rc 1

shim

lint
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

1 similar comment
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@Foxandxss
Copy link
Member

Update to this too. @thelgevold could you please update the prose now that there is no directives array so the paragraph where you mention it (for the recursive thing).

I can do it, but I prefer that the original author does it :)

@thelgevold
Copy link
Contributor Author

Thanks! @Foxandxss
I tweaked the point about the recursive template.

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.

3 participants