Remove v-html from app/assets/javascripts/notebook/cells/output/latex.vue
As part of the effort to remove v-html
from the codebase, we need to replace it with a safer alternative.
See parent epic for more details and examples.
Implementation Plan
-
Replace v-html
withv-safe-html
by using GlSafeHTMLDirective -
Is the value being passed to v-html
sanitized before in frontend?-
Remove that part, and -
Replace every other v-html
that uses that value withv-safe-html
-
-
Remove the comment /* eslint-disable vue/no-v-html */
-
Follow standard code review process