goldendict-ng icon indicating copy to clipboard operation
goldendict-ng copied to clipboard

After adjusting the width of the "query panel" and other panels, restarting the software will restore the default width

Open blucca opened this issue 1 year ago • 7 comments

描述 bug 调整“查询面版”等面版的宽度后,重启软件会恢复默认宽度

测试视频 https://github.com/xiaoyifang/goldendict-ng/assets/127029904/e1bf8e64-a3bf-4961-8192-c0a7d5a00ce3

软件版本 GoldenDict-ng-v23.07.25-alpha.230805.1ee42bd5

额外信息 之前使用的版本是 GoldenDict-ng-v23.07.25-alpha.230727.a1e62f62,这个版本会记住调整后的宽度

blucca avatar Aug 06 '23 12:08 blucca

Bot detected the issue body's language is not English, translate it automatically.

Describe the bug After adjusting the width of the "query panel" and other panels, restarting the software will restore the default width

TEST VIDEO https://github.com/xiaoyifang/goldendict-ng/assets/127029904/e1bf8e64-a3bf-4961-8192-c0a7d5a00ce3

Software version GoldenDict-ng-v23.07.25-alpha.230805.1ee42bd5

extra information The version used before is GoldenDict-ng-v23.07.25-alpha.230727.a1e62f62, this version will be remembered hold the adjusted width

github-actions[bot] avatar Aug 06 '23 12:08 github-actions[bot]

seems a regression of commit https://github.com/xiaoyifang/goldendict-ng/commit/1ffd7296149e02dd83c49dd3ca2cf6f984d04fe9

@shenlebantongying Can you help to take a look?

xiaoyifang avatar Aug 06 '23 13:08 xiaoyifang

Bisecting by moving restoreState/restoreGeometry between the code in this change tells me that restoreState/restoreGeometry have to be called before show(). If called after show(), DockWidget's geometry cannot be applied.

https://github.com/xiaoyifang/goldendict-ng/commit/1ffd7296149e02dd83c49dd3ca2cf6f984d04fe9#diff-56c809abf8615ec9b8833e8e0ddc82d72ff689ed428040d4551b7cb2518b6a0fR865

shenlebantongying avatar Aug 06 '23 13:08 shenlebantongying

fixed on Windows 11 ,but still persisted on Windows 10.

xiaoyifang avatar Aug 07 '23 12:08 xiaoyifang

no effect in my win11 either.

atauzki avatar Aug 07 '23 12:08 atauzki

我这里 win11 上 portable mode 重启软件会恢复默认宽度 不用 portable mode 就正常

czz404 avatar Apr 13 '24 00:04 czz404

Bot detected the issue body's language is not English, translate it automatically.

I'm on win11 here Restarting the software in portable mode will restore the default width. It's normal without portable mode

github-actions[bot] avatar Apr 13 '24 00:04 github-actions[bot]

@xiaoyifang +1

Windows 11 Portable Mode Goldendict-ng 24.09.0.316ec900 at 2024-09-15T14:12:19Z Qt 6.7.2 Visual C++ Compiler 194134120 windows winnt 10.0.22000 x86_64-little_endian-llp64 Flags: MAKE_ZIM_SUPPORT MAKE_CHINESE_CONVERSION_SUPPORT NO_TTS_SUPPORT no_ffmpeg_player

LakeishaKowalczyk avatar Sep 26 '24 06:09 LakeishaKowalczyk