Skip to content

[#3420] disable editor when hidden#3423

Merged
t-paul merged 3 commits intoopenscad:masterfrom
eltonlaw:gh-3420-disable-editor
Aug 30, 2020
Merged

[#3420] disable editor when hidden#3423
t-paul merged 3 commits intoopenscad:masterfrom
eltonlaw:gh-3420-disable-editor

Conversation

@eltonlaw
Copy link
Contributor

Addresses #3420

  • Setting the active editor to read only when hidden 94cc6b4
  • If an autocomplete is being shown, cancel it 2b470a0

Copy link
Member

@t-paul t-paul left a comment

Choose a reason for hiding this comment

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

I think that's actually a Qt bug. It might even be fixed as I can't reproduce the issue on Debian/Testing with Qt5.14.2. There's a similar sounding issue https://bugreports.qt.io/browse/QTBUG-82939 which is still open.

As a number of builds use older Qt versions and are affected by the issue, it seems sensible to have this fix, it would just be nice to have a code comment on why this code is here and that it is likely a workaround.

@eltonlaw
Copy link
Contributor Author

eltonlaw commented Aug 29, 2020

Nice find! Looks like the exact issue. Huh, so you can't reproduce it? That's weird, on that ticket it says it affects version 5.14.1 and my local is using 5.15.0 . Not too sure what's going on.

For the comments, is this alright? 8624880

@eltonlaw eltonlaw requested a review from t-paul August 29, 2020 04:37
@t-paul
Copy link
Member

t-paul commented Aug 30, 2020

I'm still puzzled why it does not happen with my local build. It does happen with the Snap package. I do run Wayland but OpenSCAD is not running as native Wayland app.

Anyway, the patch looks good. Thanks.

@t-paul t-paul merged commit d8c4d72 into openscad:master Aug 30, 2020
@t-paul
Copy link
Member

t-paul commented Aug 30, 2020

Hmm, looks like the updated snap does not fix the issue.

@MichaelAtOz
Copy link
Member

Should there be Windows builds there?

@t-paul
Copy link
Member

t-paul commented Aug 30, 2020

Those should show up automatically on the download page at about 8 EDT.

@MichaelAtOz
Copy link
Member

Has the build changed or is it sill running?
It used to show e.g.

ci/circleci: openscad-appimage-64bit Your tests passed on CircleCI!
Details
ci/circleci: openscad-mxe-32bit Your tests passed on CircleCI!
Details
ci/circleci: openscad-mxe-64bit Your tests passed on CircleCI!
Details

and the files accessed via artifacts.

@t-paul
Copy link
Member

t-paul commented Aug 31, 2020

Oh, on the PR. Not sure why, but it looks like CircleCI did not see that PR at all. It's been a bit problematic for a while now. Other PRs just start showing "Unauthorized" and fail to run too, but I can trigger manually.

@t-paul
Copy link
Member

t-paul commented Aug 31, 2020

Looking again at the Snap it does make the editor read-only. It still triggers the autocomplete though, e.g. by typing cu / hide editor / typing b (or even any other key).

@t-paul
Copy link
Member

t-paul commented Sep 1, 2020

And it's possible to trigger call-tips via cube( + pressing (.

@eltonlaw eltonlaw deleted the gh-3420-disable-editor branch September 6, 2020 01:49
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.

3 participants