Bug 173062

Summary:	Long Track or Artist name makes tab "Summary" in Tag Dialog clean.
Product:	[Applications] amarok	Reporter:	Andrey Esin <gmlastik>
Component:	general	Assignee:	Amarok Developers <amarok-bugs-dist>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	2.0-SVN
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	Example file to test this bug Patch to solve this problem Patch to solve this problem try 2

Description Andrey Esin 2008-10-17 20:42:02 UTC

Version:           2.0-svn rev.872644 (using KDE 4.1.2)
Compiler:          gcc 4.2.4 
OS:                Linux
Installed from:    Ubuntu Packages

If file has long (more than 500 symbols) Track or Artist name and file added to playlist and you watch tab "Summary" in Tag Dialog you will see clean dialog without any buttons, tabs and other widgets.
To reproduce this bug i have made little mp3 file, which in attach.
And write little patch, may be, it will be useful.

P.S. I have never seen songs with so long names, but i think it is a bug :)
P.S.S May be, this bug, only on my computer....

Comment 1 Andrey Esin 2008-10-17 20:44:01 UTC

Created attachment 27977 [details]
Example file to test this bug

Add this file to playlist, open Tag Dialog and click on "Summary" if it's ok then this my bug, if it's clean it's bug of Amarok or QT :)

Comment 2 Andrey Esin 2008-10-17 20:48:03 UTC

Created attachment 27978 [details]
Patch to solve this problem

In this patch, long names will be truncate to more small names (truncCount is variable which determines how much symbols leave in name)

Comment 3 Mark Kretschmann 2008-10-17 21:07:30 UTC

Andrey, there is a simpler way to do this:

http://doc.trolltech.com/4.4/qfontmetrics.html#elidedText

Comment 4 Andrey Esin 2008-10-18 11:28:40 UTC

Oh...sorry :( I didn't know about this :(
Ok i will remake the patch, but i think elidedText not useful for this, because "width is specified in pixels, not characters", may be i mistaken...

So, did you reproduce this bug?

Comment 5 Lydia Pintscher 2008-10-28 22:42:01 UTC

What is the status of this?

Comment 6 Andrey Esin 2008-10-29 16:12:17 UTC

Nobody didn't confirm this bug :(

Comment 7 Mark Kretschmann 2008-10-29 19:21:16 UTC

Sorry Andrey, just tested it with your file: I can confirm it.

What happened was that I got one giant (full width of the screen) black filled rectangle. I had to kill Amarok to get rid of it. Pretty bad.

Comment 8 Andrey Esin 2008-10-30 12:24:10 UTC

Created attachment 28232 [details]
Patch to solve this problem try 2

Track Name or Album Name or Artist Name will stretch as much as possible in dialog with '...' in the end :)

Comment 9 Mark Kretschmann 2008-10-30 12:31:06 UTC

SVN commit 877787 by markey:

Fix for very long track or artist names messing up the tag dialog. Thanks to Andrey Esin <gmlastik@gmail.com> for the patch.

Andrey, as you've already contributed so many good patches, are you interested in getting a KDE SVN account?

BUG: 173062


 M  +14 -7     TagDialog.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=877787