Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Distraction free plugin does not work on OSX #1864

Closed
Enrico68 opened this issue Jan 14, 2022 · 22 comments
Closed

Distraction free plugin does not work on OSX #1864

Enrico68 opened this issue Jan 14, 2022 · 22 comments
Labels

Comments

@Enrico68
Copy link

I have just installed Zim 0.74.3 using homebrew on OSX 10.15. I have tried to use the distraction free plugin with no success. When I press F11 I have no effect. Even from the menu-bar -->Fullscreen.

@dehesselle
Copy link
Member

While I don't use it personally, there's definitely a difference in fullscreen when I enable that plugin. Can you test our app https://zim-wiki.org/downloads.html ?

@Enrico68
Copy link
Author

Just tried but still cannot get this option working. It behaves in the same way as the brew version with eventually more problems (not recognizing R, Gnuplot, LaTex for equations, DiTaa, seqdiag). I will remain with brew version using Emacs with zim-mode (perfect integration).

@introt
Copy link
Collaborator

introt commented Jan 16, 2022

Could you post a screen recording and debug logs?

@Enrico68
Copy link
Author

Here you get the debug logs and here the screen.
P.S. the F-11 key on OSX (like the other functions keys) are not available.

@introt
Copy link
Collaborator

introt commented Jan 16, 2022

Thank you! The plugin seems to lack any and all logging, but your log also shows a concerning lack of DEBUG: Action: toggle_fullscreen(True) (which should show up whether you toggle via the function key or the menu item, regardless of the plugin).

The window seems to be maximized in your recording. Could you try resizing the window to be tiny and then triggering the menu item?

@Enrico68
Copy link
Author

The window seems to be maximized in your recording. Could you try resizing the window to be tiny and then triggering the menu item?

Minimizing and maximizing the window is working correctly using the window frame button. But the menu item is not responding.

@introt
Copy link
Collaborator

introt commented Jan 16, 2022

Seems there is a larger (not limited to the plugin) issue here. Are you using some window management related software? The menu item is really basic, so there might perhaps be an upstream issue. I can only extrapolate so much, as I don't have a Mac to test with :/

@Enrico68
Copy link
Author

could be interesting but with the Zim Markdown version is working. Here is the debug

@introt
Copy link
Collaborator

introt commented Jan 18, 2022

Doesn't look like toggle_fullscreen is present there either. I'm not familiar with the fork's code base, but hope it offers clues for the devs with OS X.

@dehesselle
Copy link
Member

This is a comparison of "normal mode" and "distraction free mode":
zim_distraction_free

(Don't pay attention to some artifacts, especially around window borders, that's the GIF compression.)

Is this not what it's supposed to do?

@introt
Copy link
Collaborator

introt commented Jan 18, 2022

I was under the impression that the menu item didn't make the window go full screen at all - thus the inquiry upon the usage of window management software. Might have something to do with the window decorations -- didn't 0.74 introduce client side decorations? @dehesselle has them disabled, you might want to try that @Enrico68

@Enrico68
Copy link
Author

@introt window decoration ? I am not aware that is possible under OSX. @dehesselle under which version of OSX are you testing ?

@dehesselle
Copy link
Member

The GIF was recorded on High Sierra since that was the easiest thing for me to do (=my test system). I've also tested on Monterey.
Decorations can be disabled by enabling the toolbar plugin and unchecking the first item there ("show elements in title bar" or something, I don't have the English version in front of me right now). If people miss out on this option, I can add a note to the readme (I already have instructions on how to enable plugins with external dependencies).

@Enrico68
Copy link
Author

I cannot see any toolbar plugin. Are you referringto the Tray Icon?

@dehesselle
Copy link
Member

toolbar_plugin

If you don't have that, you can't possibly be on 0.74.3, it's part of Zim.

For the sake of testing, please use our app (Homebrew is unsupported at this time). I'm not convinced yet that there is even an issue with distraction free editing.

@Enrico68
Copy link
Author

Thanks @dehesselle it is working. I have installed the HEAD version and everything is in Ordnung

@introt
Copy link
Collaborator

introt commented Jan 19, 2022

Glad that the issue got resolved 👍

@dehesselle would you mind checking if toggling fullscreen works with the decorations enabled? If not, that's a separate issue not related to the plugin and needs its own ticket.

@dehesselle
Copy link
Member

dehesselle commented Jan 19, 2022

Fullscreen in general does not work with CSD enabled, regardless of distraction free mode. Clicking the menu item simply doesn't do anything. I wasn't aware of that.

@introt I'll create an issue and link it here later.
Also, please don't assign issues to me. I prefer to do that once I'm really working on something, not before.

@introt
Copy link
Collaborator

introt commented Jan 19, 2022

@dehesselle Roger that! (sorry!)

@jaap-karssenberg Could we get operating system labels for platform-dependant issues?

@jaap-karssenberg
Copy link
Member

jaap-karssenberg commented Jan 19, 2022 via email

@introt
Copy link
Collaborator

introt commented Jan 19, 2022

Roger. Triagers can't edit titles, so some cooperation is required -- unless you want to complicate things by introducing GitHub Teams to assign issues to (and then have the team member assign themselves when taking it up). Not sure whether that's worth the hassle given our active contributor count, but it could help new devs find stuff to work on (eg. issues assigned to GTK/UI team vs. packaging vs. OS integration etc) or non-devs find issues which need investigating (reproducibility & writing clear(er) steps).

@jaap-karssenberg
Copy link
Member

jaap-karssenberg commented Jan 25, 2022 via email

@introt introt added the macOS Issue that is *specific* for mac OS label Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants