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

Bugfix/syntax highlighting site #147

Merged
merged 4 commits into from
Dec 4, 2019
Merged

Bugfix/syntax highlighting site #147

merged 4 commits into from
Dec 4, 2019

Conversation

anton-karlovskiy
Copy link
Contributor

@anton-karlovskiy anton-karlovskiy commented Dec 3, 2019

  1. highlight.js is up-to-date (v9.16.2) so some broken syntax highlighting is fixed by upgrading the lib
  2. syntax highlighting is restored
  3. linked CDN css for syntax highlighting and removed local css file

@addyosmani addyosmani merged commit 0f644e7 into GoogleChromeLabs:master Dec 4, 2019
@addyosmani
Copy link
Collaborator

Thanks!

@addyosmani
Copy link
Collaborator

Hmm. I need to dig into this further when not on mobile, but it looks like our latest deploys are having challenges with syntax highlighting again (this fix was included).

I'll debug later in the day, but just wanted to highlight this.

https://quicklink-6a87b.firebaseapp.com/

@anton-karlovskiy
Copy link
Contributor Author

@addyosmani

Let me debug it too.

@anton-karlovskiy
Copy link
Contributor Author

@addyosmani

quicklink-syntax-highlighting-mobile
quicklink-syntax-highlighting

On my side, I think syntax-highlighting looks okay.
Could you please let me know what is broken?

@addyosmani
Copy link
Collaborator

Hey Anton. After retesting syntax highlighting on the current deployment it does appear to be working. I'll chalk up my previous tests to there being a caching issue of some sort :)

@anton-karlovskiy anton-karlovskiy deleted the bugfix/syntax-highlighting-site branch December 10, 2019 04:32
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