livebook-dev / livebook

Automate code & data workflows with interactive Elixir notebooks
https://livebook.dev
Apache License 2.0
4.86k stars 415 forks source link

Code editor doesn't support my keyboard layout #350

Closed jaimeiniesta closed 3 years ago

jaimeiniesta commented 3 years ago

Environment

Erlang/OTP 23 [erts-11.2.2.1] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [hipe] Elixir 1.12.0 (compiled with Erlang/OTP 23)

Current behavior

While in Firefox 89.0 it works fine, I can't get the bracket characters like { and } in the code editor when working in Chrome or Safari. I have a Spanish ISO Keyboard layout. This seems an issue or misconfiguration with Monaco Editor.

Captura de pantalla 2021-06-16 a las 10 09 11

Expected behavior

I'd like to be able to include all characters in my keyboard layout! :)

jaimeiniesta commented 3 years ago

Tailwind Play also uses Monaco Editor - brackets work fine there in Chrome. Repo.

josevalim commented 3 years ago

Thanks @jaimeiniesta! Quick question: if you are running from master, can you please share your commit?

jaimeiniesta commented 3 years ago

Yes, I just downloaded it today, commit is https://github.com/elixir-nx/livebook/commit/2b2d6c9d509b02d64d428f93844598168bb65e60

jonatanklosko commented 3 years ago

@jaimeiniesta is it the case only for Elixir cells, or for Markdown cells as well? Also, does it work on the Monaco homepage?

jaimeiniesta commented 3 years ago

@jonatanklosko it happens both on Elixir and Mardown cells.

It works fine on the editor in Monaco homepage, and on Tailwind Play.

jonatanklosko commented 3 years ago

There's a recent issue that looks relevant (https://github.com/microsoft/monaco-editor/issues/2533) and it a has been addressed in monaco 0.25.2. I've just updated to this version in e49aaa62d2250f837ffd4b8b5ed582621377bcfb, please let me know if this helps :)

jaimeiniesta commented 3 years ago

Yay! It works great now, thanks!

Captura de pantalla 2021-06-23 a las 11 12 21

Verified in Chrome, Safari and Firefox.

jonatanklosko commented 3 years ago

Thanks for verifying! :cat: