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

Upgraded highlight version to 9.1, removed highlight when sample is t… #1956

Merged
merged 11 commits into from
Feb 22, 2016
Merged

Upgraded highlight version to 9.1, removed highlight when sample is t… #1956

merged 11 commits into from
Feb 22, 2016

Conversation

bodnia
Copy link
Contributor

@bodnia bodnia commented Feb 9, 2016

@fehguy upgraded highlight version to 9.1, which looks a little bit faster, but still causes the blocking screen. It might be placed into workers but browsers restrict running web workers with local files (Chrome allows only when running it with --allow-file-access-from-files flag. So it's not an option when running only index.html without server. So where is added class nohighlight when example length is more than 5000 and highlighter isn't run for this piece of code.

@fehguy
Copy link
Contributor

fehguy commented Feb 9, 2016

looks good. Can you make the highlight threshold use the options.highlightSizeThreshold option instead of hard-coding to 5000?

@fehguy
Copy link
Contributor

fehguy commented Feb 19, 2016

@bodnia can you please rebase?

@bodnia
Copy link
Contributor Author

bodnia commented Feb 22, 2016

@fehguy I rebased and also could you please review the commit bodnia@ed7c726
Here highlightSizeThreshold is a maximum example code length which may be processed with highlighter, which may be changed with configure method.

Thanks!

@fehguy fehguy merged commit 57498de into swagger-api:master Feb 22, 2016
Jonahss pushed a commit to eaze/swagger-ui that referenced this pull request Aug 12, 2016
…ble highlight when length more than highlightSizeThreshold
vincent-zurczak pushed a commit to roboconf/swagger-ui that referenced this pull request Aug 19, 2016
…ble highlight when length more than highlightSizeThreshold
@fehguy fehguy modified the milestone: v2.2.1 Aug 23, 2016
JuanSW18 pushed a commit to Digital-Paw/digital-paw-swagger-ui that referenced this pull request Aug 23, 2024
…ble highlight when length more than highlightSizeThreshold
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.

5 participants