Skip to content

Replace ember-prism with highlight.js #3878

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 1 commit into from
Sep 4, 2021
Merged

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Aug 31, 2021

highlight.js is easier to integrate with the app (via ember-auto-import) and doesn't require us to use any global variables or an ember-addon at all.

The remaining two <pre> usages got a .terminal CSS class in this commit to avoid styling conflicts.

@Turbo87 Turbo87 added C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works A-frontend 🐹 labels Aug 31, 2021
highlight.js is easier to integrate with the app (via ember-auto-import) and doesn't require us to use any global variables etc.

The remaining two `<pre>` usages got a `.terminal` CSS class in this commit to avoid styling conflicts.
Copy link
Member

@Rustin170506 Rustin170506 left a comment

Choose a reason for hiding this comment

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

Tested locally and looks great!

@Rustin170506
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 4, 2021

📌 Commit 7ad1136 has been approved by hi-rustin

@bors
Copy link
Contributor

bors commented Sep 4, 2021

⌛ Testing commit 7ad1136 with merge f8ab731...

@bors
Copy link
Contributor

bors commented Sep 4, 2021

☀️ Test successful - checks-actions
Approved by: hi-rustin
Pushing f8ab731 to master...

@bors bors merged commit f8ab731 into rust-lang:master Sep 4, 2021
@Turbo87 Turbo87 deleted the highlight-js branch September 4, 2021 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-frontend 🐹 C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants