Skip to content

Update codemirror to 5.65.21#1105

Merged
anthonygego merged 2 commits into
INGInious:mainfrom
tbarbette:PRfixcodemirror
Apr 2, 2026
Merged

Update codemirror to 5.65.21#1105
anthonygego merged 2 commits into
INGInious:mainfrom
tbarbette:PRfixcodemirror

Conversation

@tbarbette
Copy link
Copy Markdown
Contributor

Fixes #912

Tried on my phone and now it works, you can fix a small problem on your phone and especially use inginious for one-liner quizz (very practical for SQL queries). Probably fixes tablets too.

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Apr 1, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

font-family: monospace;
height: 300px;
color: black;
direction: ltr;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this necessary ? For code it makes sense, but I don't know in case people want to write exercises statements in a RTL language such as hebrew if this is convenient.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the default css from codemirror, I don't really know...

Copy link
Copy Markdown
Member

@anthonygego anthonygego left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great. Could you push the minified version instead ?

Thanks.

@anthonygego anthonygego merged commit 82cdd95 into INGInious:main Apr 2, 2026
5 checks passed
@tbarbette tbarbette deleted the PRfixcodemirror branch April 2, 2026 13:59
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.

The text editor does not work on (Android+Chrome Mobile) smartphones

2 participants