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

[BUG]: or FEATURE? 4.6.6 always starts settings-window with fixed size - resizing is not remembered #1336

Closed
mohafri opened this issue Mar 15, 2024 · 2 comments
Assignees
Labels
Component-GUI Status-Fixed Ticket is resolved. Type-Defect This is BUG!!!
Milestone

Comments

@mohafri
Copy link

mohafri commented Mar 15, 2024

Brief description of the issue

For me #1333 is only partially solved: size of main-window is ok now, but not the size of the settings-window (I also cannot reproduce the bug described in #1335).

My settings in config.ini:
settings_window_size=@SiZe(1008 474) - the window is too narrow with 4.6.6.
Resizing is ignored with next start of rssguard or close and reopen of the settings-window.
Reverting only the lines with 'SettingsWindowInitialSize' in commit e48677b solved the issue for me:
src/librssguard/gui/dialogs/formsettings.cpp:
62 + resize(qApp->settings()->value(GROUP(GUI), GUI::SettingsWindowInitialSize, size()).toSize());
125 + qApp->settings()->setValue(GROUP(GUI), GUI::SettingsWindowInitialSize, size());
src/librssguard/miscellaneous/settings.cpp:
281 + DKEY GUI::SettingsWindowInitialSize = "settings_window_size";
295 + KEY SettingsWindowInitialSize;

How to reproduce the bug?

  1. change settings regarding size of settings_window
  2. or resize settings-window, close the window and reopen
  3. restart rssguard

What was the expected result?

Manually changed size of settings-window is not reset.

What actually happened?

rssguard always starts the settings-window with fixed size.

Debug log

none

Operating system and version

  • OS: Arch Linux 64-bit - xfce4-desktop
  • RSS Guard version: 4.6.6
@mohafri mohafri added the Type-Defect This is BUG!!! label Mar 15, 2024
@martinrotter martinrotter added this to the 4.6.7 milestone Mar 18, 2024
@martinrotter
Copy link
Owner

Hopefully fixed. please test via devbuild.

Now the solution is universal and it remembers size for all dialogs inside RSS Guard! As for dialogs' positions - I much prefer to have all modal dialogs display on the center of its parent main window or on the center of the "screen" if the dialog is just too big. This was fixed too.

Pls test and let me know.

@mohafri
Copy link
Author

mohafri commented Mar 18, 2024

Tested with current development build.

Everything is now working perfectly!

As for dialogs' positions - I much prefer to have all modal dialogs display on the center of its parent main window or on the center of the "screen" ...>

That is also fine for me.

Thanks again for the quick fix and your great program!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component-GUI Status-Fixed Ticket is resolved. Type-Defect This is BUG!!!
Projects
None yet
Development

No branches or pull requests

2 participants