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.

Sign up for GitHub

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

Jump to bottom

[UI] Obscuring of Main FastFlix Window #568

Closed
vitduck opened this issue Jun 27, 2024 · 4 comments
Closed

[UI] Obscuring of Main FastFlix Window #568

vitduck opened this issue Jun 27, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@vitduck
Copy link

vitduck commented Jun 27, 2024

FastFlix Version:
5.7.0, 5.7.1

Operating System:
Windows 11 21H1, 24H2

Describe the bug
FastFlix main window is always offset to right side of the screen on startup

Screen resolution
1920x1080

Screenshots
Fastflix

Hello,

I am not sure whether this can be classified as a bug since it does not hinder operation, and is just a minor inconvenience.

As shown in the screenshot:

  • FastFlix main window always offset itself to the right side of the screen, obscuring about 15% of its windows.
  • User has to manually drag the window to center it on the screen

At first, I thought that 24H2 is trying to act smart. But the same issue was observed with older version 21H1.
I also tested with dual monitor setup where FastfFlix main window also 'bleeds' into the second monitor as well.

Perhaps coordinates of the main window is currently calculated relatively w.r.t console's ones, causing excessive off set.
@vitduck vitduck added the bug Something isn't working label Jun 27, 2024
@vitduck vitduck changed the title Obscuring of Main FastFlix Window [UI] Obscuring of Main FastFlix Window Jun 27, 2024
cdgriffith added a commit that referenced this issue Aug 7, 2024
@cdgriffith
* Adding #568 center app on startup (thanks to Viet-Duc Le)
88f296b
@cdgriffith cdgriffith mentioned this issue Aug 7, 2024
Merged
@cdgriffith
Copy link
Owner

Thanks for the improvement idea, never even thought of that to be honest. Should be centered in next version, beta: https://github.com/cdgriffith/FastFlix/releases/tag/5.8.0b0

@vitduck
Copy link
Author

vitduck commented Aug 9, 2024

@cdgriffith

Thanks for consider the suggestion.
I would like to confirm that the main UI is now properly centered on my screen.

Can I give a feed back on the beta here instead of opening a new thread ?

fastflix

There is an issue with pandetic_core which prevents applying changes to video / audio settings.
I can confirm that 5.7.4 does not show this error.

Thanks.

@cdgriffith
Copy link
Owner

Thanks for the confirm and the error details! Fix is pushed to prod so won't affect the main version release :)

cdgriffith added a commit that referenced this issue Aug 9, 2024
@cdgriffith
Version 5.8.0 (#593)
f3cfc88 
* Adding #283 support for experimental DTS (dca) audio by adding -strict -2 (thanks to Sub7)
* Adding #354 M1 support (thanks to Nhunz and Anton)
* Adding #536 Improve Profiles - save advanced options (thanks to CelticTaonga and DCNerds)
* Adding #568 center app on startup (thanks to Viet-Duc Le)
* Adding #589 support for pydantic 2.x (thanks to dmo marillat)
* Adding #592 Add alpha channel for VP9 (thjanks to subof)
* Fixing #185 audio channels not being set properly and resetting on encoder change (thanks to Tupsi)
* Fixing #522 add file fails - fixed as of 5.7.0 (thanks to pcl5x2008)
* Fixing #531 list limitation in readme that FFmpeg must support the software encoders listed (thanks to brunoais)
* Fixing #567 Profiles for WebP did not work (nor GIF dither) (thanks to jpert)
* Fixing #582 BT.2020-10 Color transfer not maintained (thanks to Ryushin)
* Fixing #585 error when trying to return a video from queue that has the video track after audio or subtitiles (thanks to Hankuu)
* Fixing #586 audio channels being set incorrectly (thanks to Hankuu)
* Fixing #588 audio and subtitle dispositions were not set from source (thanks to GeZorTenPlotZ)
@cdgriffith
Copy link
Owner

Added in 5.8.0! https://github.com/cdgriffith/FastFlix/releases/tag/5.8.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vitduck @cdgriffith