Skip to content

update the css to match dartlang.org #1416

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

Merged
merged 3 commits into from
May 12, 2017
Merged

update the css to match dartlang.org #1416

merged 3 commits into from
May 12, 2017

Conversation

devoncarew
Copy link
Member

This does not change page layout of the top-of-page navigation.

@jcollins-g
screen shot 2017-05-12 at 3 16 16 am

@googlebot googlebot added the cla: yes Google CLA check succeeded. label May 12, 2017
List get navLinks;
Documentable get navLinksLast => navLinks.last;
Copy link
Contributor

Choose a reason for hiding this comment

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

Might we instead make name and kind getters from the templates and let the template classes define how we get them?

Copy link
Member Author

Choose a reason for hiding this comment

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

Renamed to parent, and added some more semantic accessors on TemplateData (name and kind).

<h5><a href="ex/ex-library.html">ex</a></h5>
<h5><a href="ex/Deprecated-class.html">Deprecated</a></h5>

<h5>class Deprecated</h5>
Copy link
Contributor

Choose a reason for hiding this comment

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

Was it a conscious choice to drop the link to class even though this is a member page?

Copy link
Member Author

Choose a reason for hiding this comment

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

Discussed off-line - this was part of simplifying the LHS navigation.

@devoncarew
Copy link
Member Author

ptal, the new diff is pretty small

Copy link
Contributor

@jcollins-g jcollins-g left a comment

Choose a reason for hiding this comment

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

lgtm, much nicer.

@devoncarew devoncarew merged commit 131c556 into master May 12, 2017
@kevmoo kevmoo deleted the dartdoc_css branch May 19, 2017 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Google CLA check succeeded.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants