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

Apply theme to API docs #724

Merged
merged 1 commit into from
Jan 24, 2019
Merged

Apply theme to API docs #724

merged 1 commit into from
Jan 24, 2019

Conversation

ghost
Copy link

@ghost ghost commented Dec 21, 2018

@ghost ghost requested a review from justinfagnani as a code owner December 21, 2018 23:09
@ghost ghost requested a review from arthurevans December 21, 2018 23:10
@arthurevans
Copy link
Contributor

Thanks! I notice you went back to the default theme, with the nav on the right side. Was that just easier to extend, or were there other things you wanted from the default theme?

@ghost
Copy link
Author

ghost commented Dec 22, 2018 via email

@justinfagnani
Copy link
Collaborator

@katejeffreys did you mean to close this PR, or are you going to update it?

@ghost
Copy link
Author

ghost commented Jan 3, 2019

I did mean to update it. I spent some more time on it this morning but couldn't get the minimal theme to build with the main docs theme.

@arthurevans Sorry I couldn't get this working.

@arthurevans
Copy link
Contributor

Going to merge this as-is and iterate. Thanks @katejeffreys!

@arthurevans arthurevans merged commit 9a8796e into master Jan 24, 2019
@justinfagnani justinfagnani deleted the api-theme branch February 9, 2019 23:38
neuronetio pushed a commit to neuronetio/lit-html that referenced this pull request Dec 2, 2019
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.

2 participants