sioyek icon indicating copy to clipboard operation
sioyek copied to clipboard

toggle_titlebar causes the window to vanish

Open cmaspi opened this issue 1 year ago • 8 comments

When I use toggle_titlebar the following happens image

I'm using the latest release from flatpak.

cmaspi avatar Apr 13 '24 18:04 cmaspi

I don't think I can fix this on sioyek's end (probably it is a window-manager specific thing), this probably should be fixed in Qt side.

ahrm avatar Apr 13 '24 18:04 ahrm

Try this release: https://github.com/Aman9das/com.github.ahrm.sioyek/actions/runs/8679761337

And tell if the issue persists

Aman9das avatar Apr 16 '24 11:04 Aman9das

@Aman9das it persists

cmaspi avatar Apr 16 '24 15:04 cmaspi

Are you on Wayland? Try Xorg n check .

I have tried to reproduce, but unable to. kindly re-install and disable extensions..

Aman9das avatar Apr 16 '24 15:04 Aman9das

Since wayland mandates Client side decoration. The toggle should be disabled on wayland. In my testing the titlebar isn't hidden afterall.

Regardless, @cmaspi the previous install may be causing issue, uninstall the previous install and kindly test the beta release afterwards..

Aman9das avatar Apr 17 '24 12:04 Aman9das

I have this problem too, but in my case this happens only if i run toggle_titlebar using the command list after opening sioyek. If i run

$ sioyek --execute-command toggle_titlebar

or if i set toggle_titlebar as a startup command in prefs_user.config or in prefs.config everything works as expected.

fabiooo4 avatar Apr 20 '24 17:04 fabiooo4

@Aman9das On removing the stable version, toggle_titlebar doesn't cause the window to vanish but as you mentioned it does nothing. I'm fine with it since the beta release has a dark titlebar.

Running the command that @fabiooo4 suggested doesn't hide the titlebar for me.

I didn't check it on Xorg since I don't plan to switch to it anytime soon. Thanks for your support!

cmaspi avatar Apr 20 '24 17:04 cmaspi

The white title bar is a result of Flatpak packaging method used, not sioyek itself. Kindly report the issue in the Flatpak manifest repo..

https://github.com/flathub/com.github.ahrm.sioyek

Aman9das avatar Apr 21 '24 00:04 Aman9das