element-hq / element-desktop

A glossy Matrix collaboration client for desktop.
https://element.io
Apache License 2.0
1.08k stars 239 forks source link

v1.11.68 update causes Element window to move upwards and left when toggling visibility #1704

Open MattiasTF opened 3 weeks ago

MattiasTF commented 3 weeks ago

Steps to reproduce

  1. Update Element from 1.11.67 to 1.11.68.
  2. Move the Element window to a known position. I moved it to the bottom right of the screen so that it snaps to the task bar and the edge of the screen.
  3. Click on the tray icon to hide the window.
  4. Click on the tray icon to show the window again.

Outcome

What did you expect?

The Element window should re-open in the previous position, touching the task bar and the edge of the screen in my case. Repeatedly hiding and showing the window will keep the it in the same position.

What happened instead?

The Element window has moved upwards and slightly to the left. The amount it moved upwards is exactly the same as the height of the top window decoration. The amount it moved left is exactly the same as the width of the left/right window decoration. Repeatedly hiding and showing the window will make it crawl across the screen until it reaches the top left corner of the screen. Doing this in a multi-monitor setup will not make the window move to a different monitor.

Further information

The issue appeared after updating the element-desktop package from 1.11.67 to 1.11.68 yesterday and then restarting Element. No other packages were updated at the same time.

Operating system

Debian Linux (Bookworm), with Mate desktop and its Marco window manager

Application version

Element version: 1.11.68; Crypto version: Rust SDK 0.7.0 (068a0af), Vodozemac 0.6.0

How did you install the app?

Element Debian repository (https://packages.element.io/debian/), followed the instructions from https://element.io/download#linux

Homeserver

No response

Will you send logs?

No

t3chguy commented 3 weeks ago

This is likely an upstream issue with Electron 30 given we just updated to it