Commit 09630cc1 authored by Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf

Qt4 - Hotkeys Preferences: * Create the QLabel warning only if needed,

                           * Fix the "empty value, empty text" key bug.
parent f437e6eb
...@@ -1039,7 +1039,7 @@ void KeySelectorControl::finish() ...@@ -1039,7 +1039,7 @@ void KeySelectorControl::finish()
module_config_t *p_item = p_main->p_config + i; module_config_t *p_item = p_main->p_config + i;
if( p_item->i_type & CONFIG_ITEM && p_item->psz_name && if( p_item->i_type & CONFIG_ITEM && p_item->psz_name &&
strstr( p_item->psz_name , "key-" ) ) strstr( p_item->psz_name , "key-" ) && !EMPTY_STR( p_item->psz_text ) )
{ {
QTreeWidgetItem *treeItem = new QTreeWidgetItem(); QTreeWidgetItem *treeItem = new QTreeWidgetItem();
treeItem->setText( 0, qtr( p_item->psz_text ) ); treeItem->setText( 0, qtr( p_item->psz_text ) );
...@@ -1124,11 +1124,9 @@ KeyInputDialog::KeyInputDialog( QList<module_config_t*>& _values, ...@@ -1124,11 +1124,9 @@ KeyInputDialog::KeyInputDialog( QList<module_config_t*>& _values,
setWindowTitle( qtr( "Hotkey for " ) + qfu( keyToChange) ); setWindowTitle( qtr( "Hotkey for " ) + qfu( keyToChange) );
QVBoxLayout *l = new QVBoxLayout( this ); vLayout = new QVBoxLayout( this );
selected = new QLabel( qtr("Press the new keys for ") + qfu(keyToChange) ); selected = new QLabel( qtr("Press the new keys for ") + qfu( keyToChange ) );
warning = new QLabel(); vLayout->addWidget( selected , Qt::AlignCenter );
l->addWidget( selected , Qt::AlignCenter );
l->addWidget( warning, Qt::AlignCenter );
buttonBox = new QDialogButtonBox; buttonBox = new QDialogButtonBox;
QPushButton *ok = new QPushButton( qtr("OK") ); QPushButton *ok = new QPushButton( qtr("OK") );
...@@ -1136,7 +1134,7 @@ KeyInputDialog::KeyInputDialog( QList<module_config_t*>& _values, ...@@ -1136,7 +1134,7 @@ KeyInputDialog::KeyInputDialog( QList<module_config_t*>& _values,
buttonBox->addButton( ok, QDialogButtonBox::AcceptRole ); buttonBox->addButton( ok, QDialogButtonBox::AcceptRole );
buttonBox->addButton( cancel, QDialogButtonBox::RejectRole ); buttonBox->addButton( cancel, QDialogButtonBox::RejectRole );
l->addWidget( buttonBox ); vLayout->addWidget( buttonBox );
buttonBox->hide(); buttonBox->hide();
CONNECT( buttonBox, accepted(), this, accept() ); CONNECT( buttonBox, accepted(), this, accept() );
...@@ -1160,10 +1158,12 @@ void KeyInputDialog::checkForConflicts( int i_vlckey ) ...@@ -1160,10 +1158,12 @@ void KeyInputDialog::checkForConflicts( int i_vlckey )
if( conflicts ) if( conflicts )
{ {
warning->setText( QLabel *warning = new QLabel(
qtr("Warning: the key is already assigned to \"") + qtr("Warning: the key is already assigned to \"") +
qfu( p_current->psz_text ) + "\"" ); qfu( p_current->psz_text ) + "\"" );
buttonBox->hide(); warning->setWordWrap( true );
vLayout->insertWidget( 1, warning );
buttonBox->show();
} }
else accept(); else accept();
} }
......
...@@ -51,6 +51,7 @@ class QTreeWidgetItem; ...@@ -51,6 +51,7 @@ class QTreeWidgetItem;
class QGroupBox; class QGroupBox;
class QGridLayout; class QGridLayout;
class QDialogButtonBox; class QDialogButtonBox;
class QVBoxLayout;
class ConfigControl : public QObject class ConfigControl : public QObject
{ {
...@@ -421,8 +422,8 @@ private: ...@@ -421,8 +422,8 @@ private:
void keyPressEvent( QKeyEvent *); void keyPressEvent( QKeyEvent *);
void wheelEvent( QWheelEvent *); void wheelEvent( QWheelEvent *);
QLabel *selected; QLabel *selected;
QLabel *warning; QVBoxLayout *vLayout;
const char * keyToChange; const char *keyToChange;
QList<module_config_t*> values; QList<module_config_t*> values;
QDialogButtonBox *buttonBox; QDialogButtonBox *buttonBox;
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment