Bug 121915 - Hidden scrollbar in collection browser
Summary: Hidden scrollbar in collection browser
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: general (show other bugs)
Version: 1.4-beta1
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-13 21:32 UTC by Jakub Kowalski
Modified: 2006-06-11 12:32 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jakub Kowalski 2006-02-13 21:32:01 UTC
Version:           1.4-beta1 (using KDE 3.5.0, Gentoo)
Compiler:          gcc version 3.4.4 (Gentoo 3.4.4-r1, ssp-3.4.4-1.0, pie-8.7.8) 
OS:                Linux

Scrollbar in collection browser is not shown if collection list window isn't as wide as top toolbar. It shows after making it wider.
Comment 1 Alexandre Oliveira 2006-02-14 00:34:07 UTC
Screenshot, please.
Comment 2 Jakub Kowalski 2006-02-14 01:09:38 UTC
http://img119.imageshack.us/img119/540/amarokscrollcombined7ei.png

Left side -- scrollbar in not visible at all.
Center -- collection window is just a bit wider than top toolbar and scrollbar starts to be visible.
Right side -- window wider than before and scrollbar is fully visible.

If button descriptions were shorter it' wouldn't happen.
Maybe in other language versions toolbar width is smaller than fixed minimum width of collection browser window.
Comment 3 Mark Kretschmann 2006-02-14 09:19:00 UTC
Yes, I can reproduce this.
Comment 4 Alexandre Oliveira 2006-03-06 06:53:37 UTC
SVN commit 516167 by aoliveira:

Don't set Collection Browser minimum width to the toolbar width, as it old cause problems when the toolbar is too big.
BUG: 121915


 M  +2 -1      collectionbrowser.cpp  


--- trunk/extragear/multimedia/amarok/src/collectionbrowser.cpp #516166:516167
@@ -289,7 +289,8 @@
 
     m_configureAction->plug( m_toolbar );
 
-    setMinimumWidth( m_toolbar->sizeHint().width() + 2 ); //set a reasonable minWidth
+    //This would break things if the toolbar is too big, see bug #121915
+    //setMinimumWidth( m_toolbar->sizeHint().width() + 2 ); //set a reasonable minWidth
 }