[prev in list] [next in list] [prev in thread] [next in thread] 

List:       amarok-devel
Subject:    Re: extragear/multimedia/amarok/src/dialogs
From:       "Dan Meltzer" <parallelgrapefruit () gmail ! com>
Date:       2008-12-03 17:16:01
Message-ID: 46059ce10812030916x4999051s513f9fbeaea2d98f () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Wed, Dec 3, 2008 at 12:08 PM, Teo Mrnjavac <teo.mrnjavac@gmail.com>wrote:

> SVN commit 892146 by mrnjavac:
>
> Added back the fixed Collection Root token. This was done with our
> usability guy leinir's suggestion, approval and blessing and should not have
> been removed.

I have to disagree with this.

1) There was a thread about this on amarok-devel.  The agreement there was
to remove this (along with the other changes).  If there are objections they
should be raised there first.  The thread was not there for all that long
before Seb commited, but objections should still be raised there even after
committing.

2)  The token itself-- Seb's objection was extremely valid.  It takes up a
_lot_ of horizontal space, and it does not add any new information.  The
preview widget below shows how it will go.  I'm really curious about why you
(and leinir) deem it necessary?

>
> Added back the hint picture. This is open for discussion but I would like
> if it stayed in.
>
>
>
>  M  +56 -7     FilenameLayoutDialog.cpp
>  M  +86 -17    FilenameLayoutDialog.ui
>
>
> --- trunk/extragear/multimedia/amarok/src/dialogs/FilenameLayoutDialog.cpp
> #892145:892146
> @@ -40,14 +40,23 @@
>     filenameLayoutEdit->hide();
>     syntaxLabel->hide();
>     syntaxLabel->setWordWrap( true );
> +    QString hintImagePath = QString( KStandardDirs::locate( "data",
> "amarok/images/FilenameLayoutDialogHint.png" ) );
> +    QPixmap *hintImage = new QPixmap( hintImagePath );
> +    hintPicture->setPixmap( *hintImage );
> +
> +    connect( cbCase, SIGNAL( toggled( bool ) ),
> +             this, SLOT( editStateEnable( bool ) ) );
> +    connect( parent, SIGNAL( accepted() ),
> +             this, SLOT( onAccept() ) );
> +    connect( tokenPool, SIGNAL( onDoubleClick( QString ) ),
> +             filenameLayout, SLOT( addToken( QString ) ) );
> +    connect( kpbAdvanced, SIGNAL( clicked() ),
> +             this, SLOT( toggleAdvancedMode() ) );
> +    connect( filenameLayout, SIGNAL( schemeChanged() ),
> +             this, SIGNAL( schemeChanged() ) );
> +    connect( filenameLayoutEdit, SIGNAL( textChanged( const QString & ) ),
> +             this, SIGNAL( schemeChanged() ) );
>
> -    connect( cbCase, SIGNAL( toggled( bool ) ), this, SLOT(
> editStateEnable( bool ) ) );
> -    connect( parent, SIGNAL( accepted() ), this, SLOT( onAccept() ) );
> -    connect( tokenPool, SIGNAL( onDoubleClick( QString ) ),
> filenameLayout, SLOT( addToken( QString ) ) );
> -    connect( kpbAdvanced, SIGNAL( clicked() ), this, SLOT(
> toggleAdvancedMode() ) );
> -    connect( filenameLayout, SIGNAL( schemeChanged() ), this, SIGNAL(
> schemeChanged() ) );
> -    connect( filenameLayoutEdit, SIGNAL( textChanged( const QString & ) ),
> this, SIGNAL( schemeChanged() ) );
> -
>     //KConfig stuff:
>     int caseOptions = Amarok::config( "TagGuesser" ).readEntry( "Case
> options" ).toInt();
>     if( !caseOptions )
> @@ -73,6 +82,42 @@
>     if( !m_isOrganizeCollection )
>         optionsFrame->show();
>
> +    //INIT for collection root
> +    unsigned int borderColor = static_cast<unsigned int>( KColorScheme(
> QPalette::Active ).decoration( KColorScheme::HoverColor ).color().rgb() );
> +    collectionRootFrame->setStyleSheet( "\
> +        color: palette( Base );\
> +        border: 2px solid #" + QString::number( borderColor, 16 ).remove(
> 0, 2 ) + ";\
> +        border-radius: 4px;\
> +        padding: 2px;\
> +        " );
> +    QHBoxLayout *collectionRootLayout = new QHBoxLayout(
> collectionRootFrame );
> +    QLabel *collectionRootIconLabel = new QLabel( "", this );
> +    QLabel *collectionRootLabel = new QLabel( i18n( "Collection root" ),
> this );
> +    collectionRootLayout->addWidget( collectionRootIconLabel );
> +    collectionRootLayout->addWidget( collectionRootLabel );
> +    collectionRootLabel->setStyleSheet( "border:0px solid #000000;
> border-radius: 0px; padding: 0px;" );
> +    collectionRootIconLabel->setStyleSheet( "border:0px solid #000000;
> border-radius: 0px; padding: 0px;" );
> +    collectionRootLayout->setContentsMargins( 0, 0, 0, 0 );
> +    collectionRootIconLabel->setContentsMargins( 0, 0, 0, 0 );
> +    collectionRootLabel->setContentsMargins( 0, 0, 0, 0 );
> +    collectionRootIconLabel->setSizePolicy( QSizePolicy::Fixed,
> QSizePolicy::Fixed );
> +    collectionRootIconLabel->setFixedSize( 16, 16 );
> +    QPixmap collectionIcon = QPixmap( KIcon( "collection-amarok"
> ).pixmap(16, 16) );
> +    collectionRootIconLabel->setPixmap( collectionIcon );
> +
> +    collectionSlashFrame->setStyleSheet( "\
> +        color: palette( Base );\
> +        border: 2px solid #" + QString::number( borderColor, 16 ).remove(
> 0, 2 ) + ";\
> +        border-radius: 4px;\
> +        padding: 2px;\
> +        " );
> +    QHBoxLayout *collectionSlashLayout = new QHBoxLayout(
> collectionSlashFrame );
> +    QLabel *collectionSlashLabel = new QLabel( "/", this );
> +    collectionSlashLayout->addWidget(collectionSlashLabel);
> +    collectionSlashLabel->setStyleSheet( "border:0px solid #000000;
> border-radius: 0px; padding: 0px;" );
> +    collectionSlashLayout->setContentsMargins( 0, 0, 0, 0 );
> +    collectionSlashLabel->setContentsMargins( 0, 0, 0, 0 );
> +
>     //INIT for tokenPool
>     tokenPool->addItem( new QListWidgetItem( KIcon( "filename-track-amarok"
> ).pixmap( 48, 48 ), i18n( "Track" ) ) );
>     tokenPool->addItem( new QListWidgetItem( KIcon( "filename-title-amarok"
> ).pixmap( 48, 48 ), i18n( "Title" ) ) );
> @@ -92,6 +137,8 @@
>         syntaxLabel->setText( i18nc("Please do not translate the %foo words
> as they define a syntax used internally by a parser to describe a
> filename.",
>                                     "The following tokens can be used to
> define a filename scheme: \
>                                      <br>%track, %title, %artist,
> %composer, %year, %album, %comment, %genre, %ignore." ) );
> +        collectionRootFrame->hide();
> +        collectionSlashFrame->hide();
>     }
>     else
>     {
> @@ -102,6 +149,8 @@
>         syntaxLabel->setText( i18nc("Please do not translate the %foo words
> as they define a syntax used internally by a parser to describe a
> filename.",
>                                     "The following tokens can be used to
> define a filename scheme: \
>                                      <br>%track, %title, %artist,
> %composer, %year, %album, %comment, %genre, %initial, %folder, %filetype,
> %discnumber." ) );
> +        collectionRootFrame->show();
> +        collectionSlashFrame->show();
>     }
>     if( m_isOrganizeCollection )
>     {
> --- trunk/extragear/multimedia/amarok/src/dialogs/FilenameLayoutDialog.ui
> #892145:892146
> @@ -5,7 +5,7 @@
>    <rect>
>     <x>0</x>
>     <y>0</y>
> -    <width>554</width>
> +    <width>690</width>
>     <height>568</height>
>    </rect>
>   </property>
> @@ -87,34 +87,34 @@
>     </widget>
>    </item>
>    <item>
> -    <widget class="QLabel" name="syntaxLabel" >
> +    <widget class="QLabel" name="hintPicture" >
>      <property name="sizePolicy" >
> -      <sizepolicy vsizetype="Preferred" hsizetype="Preferred" >
> +      <sizepolicy vsizetype="Fixed" hsizetype="Expanding" >
>        <horstretch>0</horstretch>
>        <verstretch>0</verstretch>
>       </sizepolicy>
>      </property>
>      <property name="minimumSize" >
>       <size>
> -       <width>0</width>
> -       <height>0</height>
> +       <width>682</width>
> +       <height>44</height>
>       </size>
>      </property>
>      <property name="maximumSize" >
>       <size>
>        <width>16777215</width>
> -       <height>144</height>
> +       <height>44</height>
>       </size>
>      </property>
>      <property name="text" >
> -      <string>The following tokens can be used to define a filename
> scheme: &lt;br>%track, %title, %artist, %composer, %year, %album, %comment,
> %genre&lt;br></string>
> +      <string/>
>      </property>
>     </widget>
>    </item>
>    <item>
> -    <widget class="FilenameLayoutWidget" name="filenameLayout" >
> +    <widget class="QLabel" name="syntaxLabel" >
>      <property name="sizePolicy" >
> -      <sizepolicy vsizetype="Preferred" hsizetype="MinimumExpanding" >
> +      <sizepolicy vsizetype="Preferred" hsizetype="Preferred" >
>        <horstretch>0</horstretch>
>        <verstretch>0</verstretch>
>       </sizepolicy>
> @@ -122,26 +122,95 @@
>      <property name="minimumSize" >
>       <size>
>        <width>0</width>
> -       <height>30</height>
> +       <height>0</height>
>       </size>
>      </property>
>      <property name="maximumSize" >
>       <size>
> -       <width>30000</width>
> -       <height>36</height>
> +       <width>16777215</width>
> +       <height>144</height>
>       </size>
>      </property>
> -     <property name="frameShape" >
> -      <enum>QFrame::StyledPanel</enum>
> +     <property name="text" >
> +      <string>The following tokens can be used to define a filename
> scheme: &lt;br>%track, %title, %artist, %composer, %year, %album, %comment,
> %genre&lt;br></string>
>      </property>
> -     <property name="frameShadow" >
> -      <enum>QFrame::Sunken</enum>
> -     </property>
>     </widget>
>    </item>
>    <item>
>     <layout class="QHBoxLayout" name="horizontalLayout_2" >
>      <item>
> +      <widget class="QFrame" name="collectionRootFrame" >
> +       <property name="sizePolicy" >
> +        <sizepolicy vsizetype="Preferred" hsizetype="Minimum" >
> +         <horstretch>0</horstretch>
> +         <verstretch>0</verstretch>
> +        </sizepolicy>
> +       </property>
> +       <property name="maximumSize" >
> +        <size>
> +         <width>16777215</width>
> +         <height>24</height>
> +        </size>
> +       </property>
> +       <property name="frameShape" >
> +        <enum>QFrame::StyledPanel</enum>
> +       </property>
> +       <property name="frameShadow" >
> +        <enum>QFrame::Raised</enum>
> +       </property>
> +      </widget>
> +     </item>
> +     <item>
> +      <widget class="QFrame" name="collectionSlashFrame" >
> +       <property name="sizePolicy" >
> +        <sizepolicy vsizetype="Preferred" hsizetype="Minimum" >
> +         <horstretch>0</horstretch>
> +         <verstretch>0</verstretch>
> +        </sizepolicy>
> +       </property>
> +       <property name="maximumSize" >
> +        <size>
> +         <width>16777215</width>
> +         <height>24</height>
> +        </size>
> +       </property>
> +       <property name="frameShape" >
> +        <enum>QFrame::StyledPanel</enum>
> +       </property>
> +       <property name="frameShadow" >
> +        <enum>QFrame::Raised</enum>
> +       </property>
> +      </widget>
> +     </item>
> +     <item>
> +      <widget class="FilenameLayoutWidget" name="filenameLayout" >
> +       <property name="sizePolicy" >
> +        <sizepolicy vsizetype="Preferred" hsizetype="MinimumExpanding" >
> +         <horstretch>0</horstretch>
> +         <verstretch>0</verstretch>
> +        </sizepolicy>
> +       </property>
> +       <property name="minimumSize" >
> +        <size>
> +         <width>0</width>
> +         <height>30</height>
> +        </size>
> +       </property>
> +       <property name="maximumSize" >
> +        <size>
> +         <width>30000</width>
> +         <height>36</height>
> +        </size>
> +       </property>
> +       <property name="frameShape" >
> +        <enum>QFrame::StyledPanel</enum>
> +       </property>
> +       <property name="frameShadow" >
> +        <enum>QFrame::Sunken</enum>
> +       </property>
> +      </widget>
> +     </item>
> +     <item>
>       <widget class="KLineEdit" name="filenameLayoutEdit" >
>        <property name="sizePolicy" >
>         <sizepolicy vsizetype="Preferred" hsizetype="MinimumExpanding" >
>

[Attachment #5 (text/html)]

<br><br><div class="gmail_quote">On Wed, Dec 3, 2008 at 12:08 PM, Teo Mrnjavac <span \
dir="ltr">&lt;<a href="mailto:teo.mrnjavac@gmail.com">teo.mrnjavac@gmail.com</a>&gt;</span> \
wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, \
204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> SVN commit 892146 by \
mrnjavac:<br> <br>
Added back the fixed Collection Root token. This was done with our usability guy \
leinir&#39;s suggestion, approval and blessing and should not have been \
removed.</blockquote><div>I have to disagree with this.<br><br>1) There was a thread \
about this on amarok-devel.&nbsp; The agreement there was to remove this (along with \
the other changes).&nbsp; If there are objections they should be raised there \
first.&nbsp; The thread was not there for all that long before Seb commited, but \
objections should still be raised there even after committing.<br> <br>2)&nbsp; The \
token itself-- Seb&#39;s objection was extremely valid.&nbsp; It takes up a _lot_ of \
horizontal space, and it does not add any new information.&nbsp; The preview widget \
below shows how it will go.&nbsp; I&#39;m really curious about why you (and leinir) \
deem it necessary? <br> </div><blockquote class="gmail_quote" style="border-left: 1px \
solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br> Added \
back the hint picture. This is open for discussion but I would like if it stayed \
in.<br> <br>
<br>
<br>
&nbsp;M &nbsp;+56 -7 &nbsp; &nbsp; FilenameLayoutDialog.cpp<br>
&nbsp;M &nbsp;+86 -17 &nbsp; &nbsp;FilenameLayoutDialog.ui<br>
<br>
<br>
--- trunk/extragear/multimedia/amarok/src/dialogs/FilenameLayoutDialog.cpp \
#892145:892146<br> @@ -40,14 +40,23 @@<br>
 &nbsp; &nbsp; filenameLayoutEdit-&gt;hide();<br>
 &nbsp; &nbsp; syntaxLabel-&gt;hide();<br>
 &nbsp; &nbsp; syntaxLabel-&gt;setWordWrap( true );<br>
+ &nbsp; &nbsp;QString hintImagePath = QString( KStandardDirs::locate( \
&quot;data&quot;, &quot;amarok/images/FilenameLayoutDialogHint.png&quot; ) );<br> + \
&nbsp; &nbsp;QPixmap *hintImage = new QPixmap( hintImagePath );<br> + &nbsp; \
&nbsp;hintPicture-&gt;setPixmap( *hintImage );<br> +<br>
+ &nbsp; &nbsp;connect( cbCase, SIGNAL( toggled( bool ) ),<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; this, SLOT( editStateEnable( bool ) ) \
);<br> + &nbsp; &nbsp;connect( parent, SIGNAL( accepted() ),<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; this, SLOT( onAccept() ) );<br>
+ &nbsp; &nbsp;connect( tokenPool, SIGNAL( onDoubleClick( QString ) ),<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; filenameLayout, SLOT( addToken( QString ) \
) );<br> + &nbsp; &nbsp;connect( kpbAdvanced, SIGNAL( clicked() ),<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; this, SLOT( toggleAdvancedMode() ) );<br>
+ &nbsp; &nbsp;connect( filenameLayout, SIGNAL( schemeChanged() ),<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; this, SIGNAL( schemeChanged() ) );<br>
+ &nbsp; &nbsp;connect( filenameLayoutEdit, SIGNAL( textChanged( const QString &amp; \
) ),<br> + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; this, SIGNAL( schemeChanged() ) \
);<br> <br>
- &nbsp; &nbsp;connect( cbCase, SIGNAL( toggled( bool ) ), this, SLOT( \
                editStateEnable( bool ) ) );<br>
- &nbsp; &nbsp;connect( parent, SIGNAL( accepted() ), this, SLOT( onAccept() ) );<br>
- &nbsp; &nbsp;connect( tokenPool, SIGNAL( onDoubleClick( QString ) ), \
                filenameLayout, SLOT( addToken( QString ) ) );<br>
- &nbsp; &nbsp;connect( kpbAdvanced, SIGNAL( clicked() ), this, SLOT( \
                toggleAdvancedMode() ) );<br>
- &nbsp; &nbsp;connect( filenameLayout, SIGNAL( schemeChanged() ), this, SIGNAL( \
                schemeChanged() ) );<br>
- &nbsp; &nbsp;connect( filenameLayoutEdit, SIGNAL( textChanged( const QString &amp; \
                ) ), this, SIGNAL( schemeChanged() ) );<br>
-<br>
 &nbsp; &nbsp; //KConfig stuff:<br>
 &nbsp; &nbsp; int caseOptions = Amarok::config( &quot;TagGuesser&quot; ).readEntry( \
&quot;Case options&quot; ).toInt();<br>  &nbsp; &nbsp; if( !caseOptions )<br>
@@ -73,6 +82,42 @@<br>
 &nbsp; &nbsp; if( !m_isOrganizeCollection )<br>
 &nbsp; &nbsp; &nbsp; &nbsp; optionsFrame-&gt;show();<br>
<br>
+ &nbsp; &nbsp;//INIT for collection root<br>
+ &nbsp; &nbsp;unsigned int borderColor = static_cast&lt;unsigned int&gt;( \
KColorScheme( QPalette::Active ).decoration( KColorScheme::HoverColor ).color().rgb() \
);<br> + &nbsp; &nbsp;collectionRootFrame-&gt;setStyleSheet( &quot;\<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;color: palette( Base );\<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;border: 2px solid #&quot; + QString::number( \
borderColor, 16 ).remove( 0, 2 ) + &quot;;\<br> + &nbsp; &nbsp; &nbsp; \
&nbsp;border-radius: 4px;\<br> + &nbsp; &nbsp; &nbsp; &nbsp;padding: 2px;\<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&quot; );<br>
+ &nbsp; &nbsp;QHBoxLayout *collectionRootLayout = new QHBoxLayout( \
collectionRootFrame );<br> + &nbsp; &nbsp;QLabel *collectionRootIconLabel = new \
QLabel( &quot;&quot;, this );<br> + &nbsp; &nbsp;QLabel *collectionRootLabel = new \
QLabel( i18n( &quot;Collection root&quot; ), this );<br> + &nbsp; \
&nbsp;collectionRootLayout-&gt;addWidget( collectionRootIconLabel );<br> + &nbsp; \
&nbsp;collectionRootLayout-&gt;addWidget( collectionRootLabel );<br> + &nbsp; \
&nbsp;collectionRootLabel-&gt;setStyleSheet( &quot;border:0px solid #000000; \
border-radius: 0px; padding: 0px;&quot; );<br> + &nbsp; \
&nbsp;collectionRootIconLabel-&gt;setStyleSheet( &quot;border:0px solid #000000; \
border-radius: 0px; padding: 0px;&quot; );<br> + &nbsp; \
&nbsp;collectionRootLayout-&gt;setContentsMargins( 0, 0, 0, 0 );<br> + &nbsp; \
&nbsp;collectionRootIconLabel-&gt;setContentsMargins( 0, 0, 0, 0 );<br> + &nbsp; \
&nbsp;collectionRootLabel-&gt;setContentsMargins( 0, 0, 0, 0 );<br> + &nbsp; \
&nbsp;collectionRootIconLabel-&gt;setSizePolicy( QSizePolicy::Fixed, \
QSizePolicy::Fixed );<br> + &nbsp; &nbsp;collectionRootIconLabel-&gt;setFixedSize( \
16, 16 );<br> + &nbsp; &nbsp;QPixmap collectionIcon = QPixmap( KIcon( \
&quot;collection-amarok&quot; ).pixmap(16, 16) );<br> + &nbsp; \
&nbsp;collectionRootIconLabel-&gt;setPixmap( collectionIcon );<br> +<br>
+ &nbsp; &nbsp;collectionSlashFrame-&gt;setStyleSheet( &quot;\<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;color: palette( Base );\<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;border: 2px solid #&quot; + QString::number( \
borderColor, 16 ).remove( 0, 2 ) + &quot;;\<br> + &nbsp; &nbsp; &nbsp; \
&nbsp;border-radius: 4px;\<br> + &nbsp; &nbsp; &nbsp; &nbsp;padding: 2px;\<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&quot; );<br>
+ &nbsp; &nbsp;QHBoxLayout *collectionSlashLayout = new QHBoxLayout( \
collectionSlashFrame );<br> + &nbsp; &nbsp;QLabel *collectionSlashLabel = new QLabel( \
&quot;/&quot;, this );<br> + &nbsp; \
&nbsp;collectionSlashLayout-&gt;addWidget(collectionSlashLabel);<br> + &nbsp; \
&nbsp;collectionSlashLabel-&gt;setStyleSheet( &quot;border:0px solid #000000; \
border-radius: 0px; padding: 0px;&quot; );<br> + &nbsp; \
&nbsp;collectionSlashLayout-&gt;setContentsMargins( 0, 0, 0, 0 );<br> + &nbsp; \
&nbsp;collectionSlashLabel-&gt;setContentsMargins( 0, 0, 0, 0 );<br> +<br>
 &nbsp; &nbsp; //INIT for tokenPool<br>
 &nbsp; &nbsp; tokenPool-&gt;addItem( new QListWidgetItem( KIcon( \
&quot;filename-track-amarok&quot; ).pixmap( 48, 48 ), i18n( &quot;Track&quot; ) ) \
);<br>  &nbsp; &nbsp; tokenPool-&gt;addItem( new QListWidgetItem( KIcon( \
&quot;filename-title-amarok&quot; ).pixmap( 48, 48 ), i18n( &quot;Title&quot; ) ) \
);<br> @@ -92,6 +137,8 @@<br>
 &nbsp; &nbsp; &nbsp; &nbsp; syntaxLabel-&gt;setText( i18nc(&quot;Please do not \
translate the %foo words as they define a syntax used internally by a parser to \
describe a filename.&quot;,<br>  &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; \
&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; \
&quot;The following tokens can be used to define a filename scheme: \<br>  &nbsp; \
&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; \
&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;&lt;br&gt;%track, %title, %artist, \
%composer, %year, %album, %comment, %genre, %ignore.&quot; ) );<br> + &nbsp; &nbsp; \
&nbsp; &nbsp;collectionRootFrame-&gt;hide();<br> + &nbsp; &nbsp; &nbsp; \
&nbsp;collectionSlashFrame-&gt;hide();<br>  &nbsp; &nbsp; }<br>
 &nbsp; &nbsp; else<br>
 &nbsp; &nbsp; {<br>
@@ -102,6 +149,8 @@<br>
 &nbsp; &nbsp; &nbsp; &nbsp; syntaxLabel-&gt;setText( i18nc(&quot;Please do not \
translate the %foo words as they define a syntax used internally by a parser to \
describe a filename.&quot;,<br>  &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; \
&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; \
&quot;The following tokens can be used to define a filename scheme: \<br>  &nbsp; \
&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; \
&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;&lt;br&gt;%track, %title, %artist, \
%composer, %year, %album, %comment, %genre, %initial, %folder, %filetype, \
%discnumber.&quot; ) );<br> + &nbsp; &nbsp; &nbsp; \
&nbsp;collectionRootFrame-&gt;show();<br> + &nbsp; &nbsp; &nbsp; \
&nbsp;collectionSlashFrame-&gt;show();<br>  &nbsp; &nbsp; }<br>
 &nbsp; &nbsp; if( m_isOrganizeCollection )<br>
 &nbsp; &nbsp; {<br>
--- trunk/extragear/multimedia/amarok/src/dialogs/FilenameLayoutDialog.ui \
#892145:892146<br> @@ -5,7 +5,7 @@<br>
 &nbsp; &nbsp;&lt;rect&gt;<br>
 &nbsp; &nbsp; &lt;x&gt;0&lt;/x&gt;<br>
 &nbsp; &nbsp; &lt;y&gt;0&lt;/y&gt;<br>
- &nbsp; &nbsp;&lt;width&gt;554&lt;/width&gt;<br>
+ &nbsp; &nbsp;&lt;width&gt;690&lt;/width&gt;<br>
 &nbsp; &nbsp; &lt;height&gt;568&lt;/height&gt;<br>
 &nbsp; &nbsp;&lt;/rect&gt;<br>
 &nbsp; &lt;/property&gt;<br>
@@ -87,34 +87,34 @@<br>
 &nbsp; &nbsp; &lt;/widget&gt;<br>
 &nbsp; &nbsp;&lt;/item&gt;<br>
 &nbsp; &nbsp;&lt;item&gt;<br>
- &nbsp; &nbsp;&lt;widget class=&quot;QLabel&quot; name=&quot;syntaxLabel&quot; \
&gt;<br> + &nbsp; &nbsp;&lt;widget class=&quot;QLabel&quot; \
name=&quot;hintPicture&quot; &gt;<br>  &nbsp; &nbsp; &nbsp;&lt;property \
                name=&quot;sizePolicy&quot; &gt;<br>
- &nbsp; &nbsp; &nbsp;&lt;sizepolicy vsizetype=&quot;Preferred&quot; \
hsizetype=&quot;Preferred&quot; &gt;<br> + &nbsp; &nbsp; &nbsp;&lt;sizepolicy \
vsizetype=&quot;Fixed&quot; hsizetype=&quot;Expanding&quot; &gt;<br>  &nbsp; &nbsp; \
&nbsp; &nbsp;&lt;horstretch&gt;0&lt;/horstretch&gt;<br>  &nbsp; &nbsp; &nbsp; \
&nbsp;&lt;verstretch&gt;0&lt;/verstretch&gt;<br>  &nbsp; &nbsp; &nbsp; \
&lt;/sizepolicy&gt;<br>  &nbsp; &nbsp; &nbsp;&lt;/property&gt;<br>
 &nbsp; &nbsp; &nbsp;&lt;property name=&quot;minimumSize&quot; &gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;size&gt;<br>
- &nbsp; &nbsp; &nbsp; &lt;width&gt;0&lt;/width&gt;<br>
- &nbsp; &nbsp; &nbsp; &lt;height&gt;0&lt;/height&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;width&gt;682&lt;/width&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;height&gt;44&lt;/height&gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;/size&gt;<br>
 &nbsp; &nbsp; &nbsp;&lt;/property&gt;<br>
 &nbsp; &nbsp; &nbsp;&lt;property name=&quot;maximumSize&quot; &gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;size&gt;<br>
 &nbsp; &nbsp; &nbsp; &nbsp;&lt;width&gt;16777215&lt;/width&gt;<br>
- &nbsp; &nbsp; &nbsp; &lt;height&gt;144&lt;/height&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;height&gt;44&lt;/height&gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;/size&gt;<br>
 &nbsp; &nbsp; &nbsp;&lt;/property&gt;<br>
 &nbsp; &nbsp; &nbsp;&lt;property name=&quot;text&quot; &gt;<br>
- &nbsp; &nbsp; &nbsp;&lt;string&gt;The following tokens can be used to define a \
filename scheme: &amp;lt;br&gt;%track, %title, %artist, %composer, %year, %album, \
%comment, %genre&amp;lt;br&gt;&lt;/string&gt;<br> + &nbsp; &nbsp; \
&nbsp;&lt;string/&gt;<br>  &nbsp; &nbsp; &nbsp;&lt;/property&gt;<br>
 &nbsp; &nbsp; &lt;/widget&gt;<br>
 &nbsp; &nbsp;&lt;/item&gt;<br>
 &nbsp; &nbsp;&lt;item&gt;<br>
- &nbsp; &nbsp;&lt;widget class=&quot;FilenameLayoutWidget&quot; \
name=&quot;filenameLayout&quot; &gt;<br> + &nbsp; &nbsp;&lt;widget \
class=&quot;QLabel&quot; name=&quot;syntaxLabel&quot; &gt;<br>  &nbsp; &nbsp; \
                &nbsp;&lt;property name=&quot;sizePolicy&quot; &gt;<br>
- &nbsp; &nbsp; &nbsp;&lt;sizepolicy vsizetype=&quot;Preferred&quot; \
hsizetype=&quot;MinimumExpanding&quot; &gt;<br> + &nbsp; &nbsp; &nbsp;&lt;sizepolicy \
vsizetype=&quot;Preferred&quot; hsizetype=&quot;Preferred&quot; &gt;<br>  &nbsp; \
&nbsp; &nbsp; &nbsp;&lt;horstretch&gt;0&lt;/horstretch&gt;<br>  &nbsp; &nbsp; &nbsp; \
&nbsp;&lt;verstretch&gt;0&lt;/verstretch&gt;<br>  &nbsp; &nbsp; &nbsp; \
&lt;/sizepolicy&gt;<br> @@ -122,26 +122,95 @@<br>
 &nbsp; &nbsp; &nbsp;&lt;property name=&quot;minimumSize&quot; &gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;size&gt;<br>
 &nbsp; &nbsp; &nbsp; &nbsp;&lt;width&gt;0&lt;/width&gt;<br>
- &nbsp; &nbsp; &nbsp; &lt;height&gt;30&lt;/height&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;height&gt;0&lt;/height&gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;/size&gt;<br>
 &nbsp; &nbsp; &nbsp;&lt;/property&gt;<br>
 &nbsp; &nbsp; &nbsp;&lt;property name=&quot;maximumSize&quot; &gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;size&gt;<br>
- &nbsp; &nbsp; &nbsp; &lt;width&gt;30000&lt;/width&gt;<br>
- &nbsp; &nbsp; &nbsp; &lt;height&gt;36&lt;/height&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;width&gt;16777215&lt;/width&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;height&gt;144&lt;/height&gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;/size&gt;<br>
 &nbsp; &nbsp; &nbsp;&lt;/property&gt;<br>
- &nbsp; &nbsp; &lt;property name=&quot;frameShape&quot; &gt;<br>
- &nbsp; &nbsp; &nbsp;&lt;enum&gt;QFrame::StyledPanel&lt;/enum&gt;<br>
+ &nbsp; &nbsp; &lt;property name=&quot;text&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp;&lt;string&gt;The following tokens can be used to define a \
filename scheme: &amp;lt;br&gt;%track, %title, %artist, %composer, %year, %album, \
%comment, %genre&amp;lt;br&gt;&lt;/string&gt;<br>  &nbsp; &nbsp; \
                &nbsp;&lt;/property&gt;<br>
- &nbsp; &nbsp; &lt;property name=&quot;frameShadow&quot; &gt;<br>
- &nbsp; &nbsp; &nbsp;&lt;enum&gt;QFrame::Sunken&lt;/enum&gt;<br>
- &nbsp; &nbsp; &lt;/property&gt;<br>
 &nbsp; &nbsp; &lt;/widget&gt;<br>
 &nbsp; &nbsp;&lt;/item&gt;<br>
 &nbsp; &nbsp;&lt;item&gt;<br>
 &nbsp; &nbsp; &lt;layout class=&quot;QHBoxLayout&quot; \
name=&quot;horizontalLayout_2&quot; &gt;<br>  &nbsp; &nbsp; &nbsp;&lt;item&gt;<br>
+ &nbsp; &nbsp; &nbsp;&lt;widget class=&quot;QFrame&quot; \
name=&quot;collectionRootFrame&quot; &gt;<br> + &nbsp; &nbsp; &nbsp; &lt;property \
name=&quot;sizePolicy&quot; &gt;<br> + &nbsp; &nbsp; &nbsp; &nbsp;&lt;sizepolicy \
vsizetype=&quot;Preferred&quot; hsizetype=&quot;Minimum&quot; &gt;<br> + &nbsp; \
&nbsp; &nbsp; &nbsp; &lt;horstretch&gt;0&lt;/horstretch&gt;<br> + &nbsp; &nbsp; \
&nbsp; &nbsp; &lt;verstretch&gt;0&lt;/verstretch&gt;<br> + &nbsp; &nbsp; &nbsp; \
&nbsp;&lt;/sizepolicy&gt;<br> + &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;maximumSize&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;size&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &lt;width&gt;16777215&lt;/width&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &lt;height&gt;24&lt;/height&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;/size&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;frameShape&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;enum&gt;QFrame::StyledPanel&lt;/enum&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;frameShadow&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;enum&gt;QFrame::Raised&lt;/enum&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp;&lt;/widget&gt;<br>
+ &nbsp; &nbsp; &lt;/item&gt;<br>
+ &nbsp; &nbsp; &lt;item&gt;<br>
+ &nbsp; &nbsp; &nbsp;&lt;widget class=&quot;QFrame&quot; \
name=&quot;collectionSlashFrame&quot; &gt;<br> + &nbsp; &nbsp; &nbsp; &lt;property \
name=&quot;sizePolicy&quot; &gt;<br> + &nbsp; &nbsp; &nbsp; &nbsp;&lt;sizepolicy \
vsizetype=&quot;Preferred&quot; hsizetype=&quot;Minimum&quot; &gt;<br> + &nbsp; \
&nbsp; &nbsp; &nbsp; &lt;horstretch&gt;0&lt;/horstretch&gt;<br> + &nbsp; &nbsp; \
&nbsp; &nbsp; &lt;verstretch&gt;0&lt;/verstretch&gt;<br> + &nbsp; &nbsp; &nbsp; \
&nbsp;&lt;/sizepolicy&gt;<br> + &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;maximumSize&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;size&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &lt;width&gt;16777215&lt;/width&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &lt;height&gt;24&lt;/height&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;/size&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;frameShape&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;enum&gt;QFrame::StyledPanel&lt;/enum&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;frameShadow&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;enum&gt;QFrame::Raised&lt;/enum&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp;&lt;/widget&gt;<br>
+ &nbsp; &nbsp; &lt;/item&gt;<br>
+ &nbsp; &nbsp; &lt;item&gt;<br>
+ &nbsp; &nbsp; &nbsp;&lt;widget class=&quot;FilenameLayoutWidget&quot; \
name=&quot;filenameLayout&quot; &gt;<br> + &nbsp; &nbsp; &nbsp; &lt;property \
name=&quot;sizePolicy&quot; &gt;<br> + &nbsp; &nbsp; &nbsp; &nbsp;&lt;sizepolicy \
vsizetype=&quot;Preferred&quot; hsizetype=&quot;MinimumExpanding&quot; &gt;<br> + \
&nbsp; &nbsp; &nbsp; &nbsp; &lt;horstretch&gt;0&lt;/horstretch&gt;<br> + &nbsp; \
&nbsp; &nbsp; &nbsp; &lt;verstretch&gt;0&lt;/verstretch&gt;<br> + &nbsp; &nbsp; \
&nbsp; &nbsp;&lt;/sizepolicy&gt;<br> + &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;minimumSize&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;size&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &lt;width&gt;0&lt;/width&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &lt;height&gt;30&lt;/height&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;/size&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;maximumSize&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;size&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &lt;width&gt;30000&lt;/width&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &lt;height&gt;36&lt;/height&gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;/size&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;frameShape&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;enum&gt;QFrame::StyledPanel&lt;/enum&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;property name=&quot;frameShadow&quot; &gt;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;&lt;enum&gt;QFrame::Sunken&lt;/enum&gt;<br>
+ &nbsp; &nbsp; &nbsp; &lt;/property&gt;<br>
+ &nbsp; &nbsp; &nbsp;&lt;/widget&gt;<br>
+ &nbsp; &nbsp; &lt;/item&gt;<br>
+ &nbsp; &nbsp; &lt;item&gt;<br>
 &nbsp; &nbsp; &nbsp; &lt;widget class=&quot;KLineEdit&quot; \
name=&quot;filenameLayoutEdit&quot; &gt;<br>  &nbsp; &nbsp; &nbsp; &nbsp;&lt;property \
name=&quot;sizePolicy&quot; &gt;<br>  &nbsp; &nbsp; &nbsp; &nbsp; &lt;sizepolicy \
vsizetype=&quot;Preferred&quot; hsizetype=&quot;MinimumExpanding&quot; &gt;<br> \
</blockquote></div><br>



_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic