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

add fullscreen mode #152

Open
cgahr opened this issue Apr 4, 2023 · 3 comments
Open

add fullscreen mode #152

cgahr opened this issue Apr 4, 2023 · 3 comments

Comments

@cgahr
Copy link

cgahr commented Apr 4, 2023

Hi,

I'd love to have a fullscreen mode for rmview.

I use rmView to present my slides via my laptop using the remarkable. The nice thing is that I can take notes or explain things directly using the pen.

However, I cannot put rmview into fullscreen, which wastes a bit of space on the bottom and top of the screen.

BR, cgahr

PS: really love rmview!

@bordaigorl
Copy link
Owner

Just pushed a fix on the devel branch df79616
It wasn't implemented earlier since in OSX maximizing = fullscreen but an explicit toggle is a good idea anyway.
Which system are you on? If you can confirm it works I'll test it some more and then merge in vnc

@cgahr
Copy link
Author

cgahr commented Apr 5, 2023

Just tested it, and fullscreen mode seems to work on Ubuntu. However, you cannot exit it by pressing esc, so that's something that is still missing.

@bordaigorl
Copy link
Owner

bordaigorl commented Apr 5, 2023

Yeah, it's a toggle so the same shortcut that makes it full screen also exists it.
On GNOME it should be Ctrl+F11, on KDE just F11 might also work.
The app uses QKeySequence.FullScreen as a shortcut the interpretation of which is platform-dependent, see the Qt docs.
There's always the option of using the context menu.
I can see that ESC could be the instinctive thing to try, I'll see what I can do.

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

No branches or pull requests

2 participants