multivnc icon indicating copy to clipboard operation
multivnc copied to clipboard

Add scale-to-fit view mode

Open bk138 opened this issue 2 years ago • 3 comments

TODOs

  • [x] add button
  • [x] add menu entry
  • [x] wire up
  • [x] harmonise fullscreen button: make toggle as well
  • [ ] get correct factor for 1:1
  • [ ] impl scaling logic
  • [ ] impl centering logic
  • [ ] scale input as well

Resources

  • https://docs.wxwidgets.org/3.2/classwx_d_c.html

bk138 avatar May 10 '23 19:05 bk138

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

muava12 avatar Aug 06 '23 03:08 muava12

WIP, see the PR 🙂

bk138 avatar Aug 06 '23 05:08 bk138

example: https://docs.wxwidgets.org/3.2/classwx_d_c.html#aee6d97d655f20039745b5f6fb43a2d51

bk138 avatar Oct 18 '23 12:10 bk138

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.

geekley avatar Mar 07 '24 20:03 geekley

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

bk138 avatar Apr 09 '24 16:04 bk138