JvanKatwijk / qt-dab

Qt-DAB, a general software DAB (DAB+) decoder with a (slight) focus on showing the signal
http://www.sdr-j.tk
GNU General Public License v2.0
296 stars 62 forks source link

Long DLS causes window size to strech off screen horizontally #263

Closed alkyl-twists closed 2 years ago

alkyl-twists commented 2 years ago

If the dynamic label gets set to a really long unbroken word the whole window resize far too big instead of wrapping.

I believe you can fix it by setting the maximumSize.width property on the dynamicLabel to something sane like 450 instead of the default.

Screenshot of problem Screenshot_2022-05-20_12-13-19

Label received breaking the window Qt-DAB-dlText-Fri-May-20-11-42-22-2022.txt

JvanKatwijk commented 2 years ago

Interesting text. Wordwrap is "on" so normal texts not fitting on a single line are displayed over more lines. Is the dynamic label a single word?

Op vr 20 mei 2022 om 14:50 schreef J @.***>:

If the dynamic label gets set to a really long unbroken word the whole window resize far too big instead of wrapping.

I believe you can fix it by setting the maximumSize.width property on the dynamicLabel to something sane like 450 instead of the default.

Screenshot of problem [image: Screenshot_2022-05-20_12-13-19] https://user-images.githubusercontent.com/62932591/169531414-49830067-3ebe-4ae8-8a23-3ddfbfb686d5.png

Label received breaking the window Qt-DAB-dlText-Fri-May-20-11-42-22-2022.txt https://github.com/JvanKatwijk/qt-dab/files/8740264/Qt-DAB-dlText-Fri-May-20-11-42-22-2022.txt

— Reply to this email directly, view it on GitHub https://github.com/JvanKatwijk/qt-dab/issues/263, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCPHQGFEKETTWXXPJX6Y6LVK6DBFANCNFSM5WPLZQUA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Jan van Katwijk

alkyl-twists commented 2 years ago

Yes, It doesn't seem to break long words up. I think it's a Qt bug (Above attached file had an example, "wyattwyattwyatt..."

JvanKatwijk commented 2 years ago

OK

I set a constraint on the width of the QLabel for the DL text. The latest sources contain the update, as well as the precompiled versions (AppImage and Windows installer) as stored in the 4.351 release

Let me know if there are still problems

Op vr 20 mei 2022 om 16:58 schreef J @.***>:

Yes, It doesn't seem to break long words up. I think it's a Qt bug

— Reply to this email directly, view it on GitHub https://github.com/JvanKatwijk/qt-dab/issues/263#issuecomment-1133009208, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCPHQDCU2WESW2VD657HHDVK6SCRANCNFSM5WPLZQUA . You are receiving this because you commented.Message ID: @.***>

-- Jan van Katwijk

alkyl-twists commented 2 years ago

Seems to be fixed for me now. Thanks.