Page MenuHomePhabricator

[dagit] Upgrade CodeMirror, fix error rendering

Authored by dish on Fri, Nov 13, 5:47 PM.



A minor upgrade to CodeMirror, but mostly this fixes a bug with how errors are displayed. A CodeMirror change introduced a new way of setting classes on lint marks, so we just need to reflect that in our own code.

Relevant CM change:

Test Plan

View Playground in Dagit with errors, verify that backgrounds render correctly.

Diff Detail

R1 dagster
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

dish requested review of this revision.Fri, Nov 13, 5:54 PM

That old background was quite ugly

This revision is now accepted and ready to land.Fri, Nov 13, 6:00 PM