altair icon indicating copy to clipboard operation
altair copied to clipboard

Line numbers overlap query pane

Open febeling opened this issue 3 years ago • 7 comments

Describe the bug After the in-app update to 4.6.2 on macos 11.6.8, there's a visual bug. Also an error notification is shown for a moment.

To Reproduce Steps to reproduce the behavior: Just open the app and click and "Add new"

Expected behavior No overlap of code and line numbers.

Screenshots Screenshot 2022-08-26 at 20 42 30

Screenshot 2022-08-26 at 20 42 05 Screenshot 2022-08-26 at 20 38 01

Desktop (please complete the following information):

  • OS: macOS 11.6.8
  • Browser: n/a
  • Platform: Electron app (download from website)
  • Version: 4.6.2

Additional context Happens with existing query as well as new pane.

febeling avatar Aug 26 '22 18:08 febeling

Do you also have this issue if you set enableExperimental to true in the settings?

imolorhe avatar Aug 27 '22 12:08 imolorhe

@imolorhe Yes, it seems to behave correctly sometimes under experimental, and more often, but still is also sometimes broken.

febeling avatar Aug 29 '22 19:08 febeling

If possible, could you share screenshots of it broken under experimental? I expect the editor under experimental to fix this issue completely so I'm surprised to hear that it's broken there as well

imolorhe avatar Aug 30 '22 18:08 imolorhe

I also have this issue on altair-express-middleware. I will try enableExperimental to see if it helps. edit: seems to have helped. I will keep updating if it breaks with experimental on.

code-is-art avatar Aug 31 '22 13:08 code-is-art

@imolorhe I can't reproduce the problem rn, with experimental enabled. I seem to remember to also experience with that setting, but now have doubts that I checked correctly.

febeling avatar Sep 01 '22 08:09 febeling

Under experimental, we have a new editor (code mirror 6) for the query editor. The variables section still uses the old editor though, so maybe that's where you saw it?

imolorhe avatar Sep 01 '22 09:09 imolorhe

@imolorhe no, I'm pretty certain I didn't work with vars, at that point.

Feel free to close, it still can be re-opened if more info emerges.

febeling avatar Sep 01 '22 09:09 febeling

Closing this issue as it should be fixed in the new editor

imolorhe avatar Nov 20 '22 22:11 imolorhe