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

chore(styles): apply material light / dark styles to highlightjs #213

Merged
merged 2 commits into from
Jul 9, 2017

Conversation

amcdnl
Copy link
Contributor

@amcdnl amcdnl commented Jul 8, 2017

Update highlightjs to use material style syntax for light and dark themes.

light
light

dark
dark

@amcdnl amcdnl requested a review from jelbourn July 8, 2017 21:20
@@ -0,0 +1,114 @@
/** From https://github.com/Kelbster/highlightjs-material-dark-theme */
Copy link
Member

Choose a reason for hiding this comment

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

Can we consume this from npm? Just copying like this would require the license to be included as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have some slight modifications, I'll reference the license.

padding: 1em;
background: #2B2B2D;
color: #CDD3D8;
font-family: "Fira Mono", monospace;
Copy link
Member

Choose a reason for hiding this comment

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

We should use Roboto Mono (for both)

@amcdnl
Copy link
Contributor Author

amcdnl commented Jul 9, 2017

@jelbourn - I've addressed the items and ready for another review.

Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn jelbourn merged commit ac3f30e into angular:master Jul 9, 2017
@amcdnl amcdnl deleted the highlightjs branch July 9, 2017 18:55
amcdnl added a commit to amcdnl/material.angular.io that referenced this pull request Jul 26, 2017
commit b684808
Author: Austin <[email protected]>
Date:   Mon Jul 17 15:35:08 2017 -0600

    feat(PWA): add service worker angular#175 (angular#223)

commit ec7c637
Author: Austin <[email protected]>
Date:   Mon Jul 17 13:52:28 2017 -0600

    feat(docs): add page titles to doc pages angular#189 (angular#220)

commit 6170057
Author: Rafael Santana <[email protected]>
Date:   Wed Jul 12 20:05:29 2017 -0300

    Fix exception being thrown on invalid routes (angular#218)

commit 101aeff
Author: Michael Prentice <[email protected]>
Date:   Tue Jul 11 19:38:17 2017 -0400

    fix(footer): extra w in www.angular.io (angular#217)

    just use https://angular.io since that is where www.angular.io goes

commit 135c38f
Author: Rafael Santana <[email protected]>
Date:   Sun Jul 9 12:37:39 2017 -0300

    Add handling for invalid routes (angular#191)

commit ad1400e
Author: Will Howell <[email protected]>
Date:   Sun Jul 9 11:35:25 2017 -0400

    Set max width for component and guide viewers (angular#207)

commit ac3f30e
Author: Austin <[email protected]>
Date:   Sun Jul 9 10:18:02 2017 -0500

    chore(styles): apply material light / dark styles to highlightjs (angular#213)
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