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 scale-to-fit view mode #219

Closed
10 tasks done
bk138 opened this issue May 10, 2023 · 5 comments · Fixed by #227
Closed
10 tasks done

Add scale-to-fit view mode #219

bk138 opened this issue May 10, 2023 · 5 comments · Fixed by #227

Comments

@bk138
Copy link
Owner

bk138 commented May 10, 2023

TODOs

  • add button
  • add menu entry
  • wire up
  • harmonise fullscreen button: make toggle as well
  • get correct factor for scaled view
  • impl scaling logic
  • impl centering logic
  • scale input as well
  • 2nd viewerwindow does not setup mode correctly
  • set mode should be remembered globally across restarts

Resources

@muava12
Copy link

muava12 commented Aug 6, 2023

Yes we need this feature. for now is there a way to make it fit on window?

@bk138
Copy link
Owner Author

bk138 commented Aug 6, 2023

WIP, see the PR 🙂

@bk138
Copy link
Owner Author

bk138 commented Oct 18, 2023

@bk138 bk138 self-assigned this Oct 18, 2023
@bk138 bk138 mentioned this issue Oct 19, 2023
2 tasks
@geekley
Copy link

geekley commented Mar 7, 2024

There is so much wasted spaced too... for starters, you could unify the tab bar, toolbar, status bar and menus in a single row. You could even have drop-downs for the server and bookmarks too, so everything is in a single row.

Also, what is the point of fullscreen if most elements will still occupy space?
It doesn't remove toolbar, status bar, statistics and the tabs bar.
The only thing that should be in the fullscreen is the other device's screen, and a way to exit fullscreen, which could be:

  • a key combo (e.g. Ctrl+F11) that's never grabbed by the device, showed when you enter fullscreen (like Firefox does)
  • a button to exit fullscreen that appears when you hover the top edge of the screen (like Firefox); or that is always visible on top of the content (without taking space away from it)

This way you could maximize usable screen space, which would be great along with this feature request.

@bk138
Copy link
Owner Author

bk138 commented Apr 9, 2024

@geekley please open a separate issue about this or better yet, a full PR :-)

bk138 added a commit that referenced this issue Oct 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants