Skip to content

Conversation

@ggazzo
Copy link
Member

@ggazzo ggazzo commented Sep 17, 2019

No description provided.

@tassoevan tassoevan self-requested a review September 18, 2019 03:28
@MartinSchoeler MartinSchoeler marked this pull request as ready for review October 16, 2019 18:39
@MartinSchoeler
Copy link
Member

@ggazzo I've tested this and these are the results:

Before:

Before 1

before 2

After:

After 1

after 2

@MartinSchoeler MartinSchoeler changed the title lazyload katex [IMPROVE] Lazyload Katex Package Oct 16, 2019
@ggazzo ggazzo merged commit 40eb927 into develop Oct 16, 2019
@ggazzo ggazzo deleted the lazy-katex branch October 16, 2019 19:29
@ggazzo ggazzo added this to the 2.2.0 milestone Oct 16, 2019
@ggazzo ggazzo added the area: ui Touches the code on client side label Oct 16, 2019
@lgtm-com
Copy link

lgtm-com bot commented Oct 16, 2019

This pull request introduces 4 alerts when merging 5a74816 into a6080f7 - view on LGTM.com

new alerts:

  • 4 for Property access on null or undefined

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui Touches the code on client side

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants