Skip to content

fix(tooling) developer.html does not render highlighted language properly #2672

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
Sep 5, 2020

Conversation

egor-rogov
Copy link
Collaborator

Our developer.html tool correctly shows markup using specified language, but always displays rendering in autodetection mode.
To reproduce, enter e. g.

for i in *.sh; do echo $i; done

and choose Language: bash. The code is (erroneously) rendered for nginx, while markup is (correctly) for bash.

I'm not familiar with vue and somehow managed to fix the issue although in a disgusting way. There should be a better solution for sure.

@joshgoebel
Copy link
Member

joshgoebel commented Sep 5, 2020

Oh wow, that was a bad mistake on my part. :-/ You fix isn't bad at all, but I'm sure there is a way to move that into a single method and then just call it from both... I'll see.

@joshgoebel joshgoebel changed the title Fix inconsistency in developer.html fix(tooling) developer.html does not render highlighted language properly Sep 5, 2020
@joshgoebel joshgoebel requested review from allejo and removed request for joshgoebel September 5, 2020 00:30
@joshgoebel
Copy link
Member

@egor-rogov Back to you. :-) If that looks good I will merge.

@joshgoebel joshgoebel added this to the 10.2 milestone Sep 5, 2020
@egor-rogov
Copy link
Collaborator Author

Yes, that's good, thanks!

@joshgoebel joshgoebel merged commit f3f29ac into master Sep 5, 2020
@joshgoebel joshgoebel deleted the egor-rogov-patch-1 branch September 11, 2020 11: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