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

Small window size hides buttons and content #146

Closed
kusterjs opened this issue Jul 28, 2024 · 14 comments
Closed

Small window size hides buttons and content #146

kusterjs opened this issue Jul 28, 2024 · 14 comments

Comments

@kusterjs
Copy link

Upon installing the 1.3.0 beta, my windows default size was quite small, resulting in no buttons and neither the right column of the main window being displayed. While the resizing seems to work as designed, wouldn't it make sense to require a minimum size for the large window mode, so that no parts of the application are hidden?
From a few tests, I seems like the mini mode is already a defined resizing, which results in hiding parts of the application. Since the button to go back to the large window mode is hidden by default, my initial user reaction was just to resize the window manually, which lead to the same situation where not all content was shown.

@neilenns
Copy link
Contributor

Have you ever had TrackAudio installed before? Or was the beta your first install?

@kusterjs
Copy link
Author

I had 1.2.0 installed before.

@neilenns
Copy link
Contributor

Ok. Just so I understand, on first launch it was narrower than you expected (which hid the sidebar), but once you resized it bigger it's properly remembering that size across app launches?

@kusterjs
Copy link
Author

No, it doesn't. It's always launched in smaller size.

@neilenns
Copy link
Contributor

No, it doesn't. It's always launched in smaller size.

Hmmmm. What platform are you on?

@kusterjs
Copy link
Author

Windows 11. In fact the window get's smaller with every time. So if I don't resize, it will be even smaller the next time. Looks like a wrong value is saved for the window size.

28-07-2024_20-38-53.mp4

@neilenns
Copy link
Contributor

Thanks for the video! A few follow-ups:

  1. Do you have multiple monitors?
  2. Is this on your primary or your secondary monitor?
  3. Is the monitor 4k?

@kusterjs
Copy link
Author

Yes, multiple monitors. This is my secondary, which is not a 4K. The primary though is one.

@neilenns
Copy link
Contributor

Yes, multiple monitors. This is my secondary, which is not a 4K. The primary though is one.

Ok I was afraid of that.

Does it work correctly if you do this on your primary monitor?

@kusterjs
Copy link
Author

Yes, there it seems to work.

Also just noticed that the mini window version will keep its old place (if I don't specifically move it). But I guess this is a desired behaviour?

@neilenns
Copy link
Contributor

neilenns commented Jul 28, 2024

The remembering position is by design, implemented here: #103. It matches the vatis behaviour and means you don't have to re-position the mini-mode to your favourite place every time you run TrackAudio.

Your issue is, unfortunately, a bug. You can see my mention of it in that PR.

@neilenns
Copy link
Contributor

Digging into this a bit, it appears to be an underlying issue with Electron and multiple monitors that have different scaling percentages. I've found at least two open issues in their github with similar symptoms, neither of which have any response from an Electron developer :(

Will poke around and see if I can find a workaround.

@pierr3
Copy link
Owner

pierr3 commented Sep 8, 2024

Please try beta 2 to see if Neil's fix worked

@pierr3
Copy link
Owner

pierr3 commented Nov 5, 2024

Closing as stale

@pierr3 pierr3 closed this as completed Nov 5, 2024
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

Successfully merging a pull request may close this issue.

3 participants