Bug 276668 - Amarok current track applet layout is broken when the font size is more than 9.
Summary: Amarok current track applet layout is broken when the font size is more than 9.
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: Context View/Current Track (other bugs)
Version First Reported In: 2.8.0
Platform: Arch Linux Linux
: NOR minor
Target Milestone: later
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-28 09:58 UTC by Sergey Malkin
Modified: 2024-03-23 21:17 UTC (History)
12 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Amarok current track applet bag (132.74 KB, image/png)
2011-06-28 09:59 UTC, Sergey Malkin
Details
12px font, corrupted after restart similar to comment 1 (93.63 KB, image/png)
2013-03-19 08:35 UTC, Dmitry Roshchin
Details
2.7 with LANG=ru_RU.UTF-8 and MS Trebuchet 13 (367.24 KB, image/png)
2013-04-02 05:33 UTC, Anton Anikin
Details
screenshot shows buggy current track applet with russian locale (220.36 KB, image/jpeg)
2014-07-07 12:26 UTC, Ильдар Ахметгалеев
Details
Quick and dirty patch (1.17 KB, patch)
2015-08-05 15:05 UTC, Yaroslav Sidlovsky
Details
Another try, still it's very ugly solution (1.63 KB, patch)
2015-08-06 10:43 UTC, Yaroslav Sidlovsky
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Malkin 2011-06-28 09:58:38 UTC
Version:           2.4.1 (using KDE 4.6.3) 
OS:                Linux

With default KDE intrface font (Sans Serif) size 10 and more, Current track applet in Amarok has somehow inverted layout and text is not visible. Screenshot applied. 

Reproducible: Always

Steps to Reproduce:
Increase default KDE font size to 10 or more, restart Amarok

Actual Results:  
Font size is 10 and everything works fine except Amarok

Expected Results:  
Font size is 10 and everyting works fine
Comment 1 Sergey Malkin 2011-06-28 09:59:55 UTC
Created attachment 61404 [details]
Amarok current track applet bag
Comment 2 Myriam Schweingruber 2011-06-29 08:13:41 UTC
I can't reproduce this here at all, tried with font sizes 10, 11 and 12, everything displays as it should. Testing with current Amarok 2.4-git on Kubuntu 11.04 with KDE 4.6.4
Comment 3 Sergey Malkin 2011-06-29 10:37:57 UTC
I've pulled last GIT build and there everything is really Ok. 
I'm looking forward for the next release.
Comment 4 Dmitry Roshchin 2013-03-19 08:31:17 UTC
Reproducible for Amarok 2.7.0. It appears only on Russian locale, on English everything is good.
Comment 5 Dmitry Roshchin 2013-03-19 08:35:20 UTC
Created attachment 78196 [details]
12px font, corrupted after restart similar to comment 1
Comment 6 Myriam Schweingruber 2013-03-19 10:08:43 UTC
Why would this only appear on Russian locale, do you use a different font? Is it a proportional or a fixed-width font?
Comment 7 Dmitry Roshchin 2013-03-19 10:26:51 UTC
(In reply to comment #6)
> Why would this only appear on Russian locale, do you use a different font?
> Is it a proportional or a fixed-width font?

Sans serif (arial) in both cases. It seems words are too big somewhere.  On English locale it corrupted with font size 16.
Comment 8 Edward Hades 2013-03-19 10:40:30 UTC
Confirmed on 2.7 with LANG=ru_RU.UTF-8 and Sans Serif 12
Comment 9 Anton Anikin 2013-04-02 05:32:14 UTC
Confirmed on 2.7 with LANG=ru_RU.UTF-8 and MS Trebuchet 13
Comment 10 Anton Anikin 2013-04-02 05:33:19 UTC
Created attachment 78571 [details]
2.7 with LANG=ru_RU.UTF-8 and MS Trebuchet 13
Comment 11 Mikhail Ivchenko 2013-12-04 07:59:18 UTC
Can't reproduce in v2.8.0
Comment 12 Myriam Schweingruber 2013-12-04 08:37:01 UTC
(In reply to comment #11)
> Can't reproduce in v2.8.0

You did test with the Russian locale? as it is not reproducible with an English one anyway.
Comment 13 Mikhail Ivchenko 2013-12-04 08:42:58 UTC
(In reply to comment #12)
> (In reply to comment #11)
> > Can't reproduce in v2.8.0
> 
> You did test with the Russian locale? as it is not reproducible with an
> English one anyway.

Ooops, sorry, my fault. :( So, I can reproduce it.
Comment 14 Myriam Schweingruber 2013-12-04 13:42:07 UTC
Confirmed with the Russian locale, as before. If somebody has an idea how to solve that...
Comment 15 Jem Orgun 2013-12-04 16:15:15 UTC
This bug is still present for me on 2.8.0 with LANG=en_US.UTF-8 and font size > 15, for what it's worth.
Comment 16 Ильдар Ахметгалеев 2014-07-07 12:26:25 UTC
Created attachment 87617 [details]
screenshot shows buggy current track applet with russian locale

In my case I have only album art and stars and nothing else in current track widget. Once again runing with LANG=C solves problem.
Comment 17 Martin Riethmayer 2014-09-09 20:27:59 UTC
I can confirm this issue with Version 2.8.0, KDE 4.14.0, openSUSE 13.1 with german locale, DPI for fonts forced to 148 (or higher). If I'm not forcing the DPI or forcing it to less than 148 (so 147 is the last setting that works), it goes back to normal. Other font settings are default. 

My screen (as reported by xdpyinfo):
dimensions:    1920x1080 pixels (508x285 millimeters)
resolution:    96x96 dots per inch
Comment 18 Myriam Schweingruber 2014-09-11 10:03:08 UTC
Why do you force DPI to start with? This should be left out so the system can calibrate itself, which is especially useful on large resolutions.
Comment 19 Ильдар Ахметгалеев 2014-09-11 11:39:06 UTC
Because I have screen with high density and not very high eyesight. So I increase scale of all texts in my system to see it.
(In reply to Myriam Schweingruber from comment #18)
> Why do you force DPI to start with? This should be left out so the system
> can calibrate itself, which is especially useful on large resolutions.
Comment 20 Yaroslav Sidlovsky 2015-08-05 15:05:11 UTC
Created attachment 93901 [details]
Quick and dirty patch
Comment 21 Yaroslav Sidlovsky 2015-08-05 15:14:24 UTC
It's better than nothing (7px wide widget I mean).
Sadly but after applying this patch no song info will be displayed, only album cover, song rating and statistics info would be visible.
Comment 22 Myriam Schweingruber 2015-08-05 22:20:14 UTC
(In reply to Yaroslav Sidlovsky from comment #20)
> Created attachment 93901 [details]
> Quick and dirty patch

Thank you for this patch. Could you please submit it to reviewboard.kde.org, and subscribe the group Amarok to it? Thank you.
Comment 23 Yaroslav Sidlovsky 2015-08-06 08:59:52 UTC
Sorry, but that's totally wrong patch.
I can't tell why on one of my machines (with same DPI and font configuration) sizeHint method returns preffered width of 7 px.
Maybe this is a bug in the Qt 4. Who knows...

Also code that relates to widget size calculation was significantly changed in the last Amarok version (2.8.90 version).
Maybe this bug will disappear.
Comment 24 Yaroslav Sidlovsky 2015-08-06 10:43:07 UTC
Created attachment 93915 [details]
Another try, still it's very ugly solution

Commented out line 172 in the CurrentTrack.cpp because without this change I don't see title, artist and album info strings.
Comment 25 Tuomas Nurmi 2024-03-23 21:17:46 UTC
Tried changing language, seems to be fixed in 2.9.71, probably by the port to KF5 and the extensive code changes in context view.