Ticket #1040 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Reset Detached Video position if outside visible desktop

Reported by: rayne86 Owned by: nielsm
Priority: high Milestone: 2.1.8
Component: Interface Version: 2.1.6
Severity: major Keywords:
Cc: Platform: All
Sub Component:

Description (last modified by nielsm) (diff)

It's possible to cause the detached video window to pop up on a position outside the visible desktop area, which makes it impossible to interact with it. That includes moving it back to the visible desktop area. This is caused by blindly saving and restoring the last position of the window.

Restoring the window position should only be done if it will be entirely inside the visible desktop area.
Ideally, if the window will be just partially inside the visible desktop, it should be moved so it is entirely inside it, and if it's entirely outside the visible desktop, it should reset to a default position. It might be too hard to do the former, so the fix might just move the detached video window to a default position if any part of it might be invisible.

The current workaround is to delete the config file, or manually edit the saved position in it.

Change History

comment:1 Changed 2 years ago by nielsm

  • Status changed from new to accepted
  • Severity changed from minor to major
  • Component changed from Video to Interface
  • Summary changed from detach video problem to Reset Detached Video position if outside visible desktop
  • Milestone set to 2.1.8
  • Owner set to nielsm
  • Description modified (diff)

This is a long-standing problem that should have been fixed ages ago.

Rewriting the description and summary to detail the solution.

comment:2 Changed 2 years ago by nielsm

(In [3774]) Make detached video display reposition itself if it opens partially or entirely out of view. Updates #1040.

comment:3 Changed 2 years ago by nielsm

  • Status changed from accepted to closed
  • Resolution set to fixed

(In [3775]) Merge r3774, closes #1040.

comment:4 Changed 2 years ago by nielsm

(In [3776]) Missed some headers in r3774, which caused compilation errors when precompiled headers were not used. Updates #1040.

comment:5 Changed 2 years ago by nielsm

(In [3777]) Merge r3776 to fix non-PCH compilation. Updates #1040.

comment:6 Changed 2 years ago by verm

(In [3794]) Include wx/display.h last, it needs some values that aren't inited by display.h itself. I'm not sure if this is a bug as a lot of other headers seem to have this problem too. Fixes compilation errors when not using precompiled headers (which loads about 30 headers before this one.) This fixes non-PCH compilation updates #1040

comment:7 Changed 2 years ago by verm

(In [3795]) Merge r3794 updates #1040.

Note: See TracTickets for help on using tickets.