diff --git a/configuredialog.cpp b/configuredialog.cpp index 99748209d..e342a44ad 100644 --- a/configuredialog.cpp +++ b/configuredialog.cpp @@ -2173,20 +2173,22 @@ void AppearancePage::MessageTagTab::slotRecordTagSettings( int aIndex ) KMMessageTagDescription *tmp_desc = mMsgTagList->at( aIndex ); - //First row tmp_desc->setName( mTagListBox->item( aIndex )->text() ); - //Second row + tmp_desc->setTextColor( mTextColorCheck->isChecked() ? mTextColorCombo->color() : QColor() ); - //Third row + + tmp_desc->setBackgroundColor( mBackgroundColorCheck->isChecked() ? + mBackgroundColorCombo->color() : QColor() ); + tmp_desc->setTextFont( mTextFontCheck->isChecked() ? mFontRequester->font() : QFont() ); - //Fourth row + tmp_desc->setIconName( mIconButton->icon() ); - //Fifth row + mKeySequenceWidget->applyStealShortcut(); tmp_desc->setShortcut( KShortcut(mKeySequenceWidget->keySequence()) ); - //Sixth row + tmp_desc->setInToolbar( mInToolbarCheck->isChecked() ); } @@ -2202,6 +2204,7 @@ void AppearancePage::MessageTagTab::slotUpdateTagSettingWidgets( int aIndex ) mTagNameLineEdit->setEnabled( false ); mTextColorCheck->setEnabled( false ); + mBackgroundColorCheck->setEnabled( false ); mTextFontCheck->setEnabled( false ); mInToolbarCheck->setEnabled( false ); mTextColorCombo->setEnabled( false ); @@ -2220,11 +2223,9 @@ void AppearancePage::MessageTagTab::slotUpdateTagSettingWidgets( int aIndex ) KMMessageTagDescription *tmp_desc = mMsgTagList->at( mTagListBox->currentRow() ); - //1st row mTagNameLineEdit->setEnabled( true ); mTagNameLineEdit->setText( tmp_desc->name() ); - //2nd row QColor tmp_color = tmp_desc->textColor(); mTextColorCheck->setEnabled( true ); if ( tmp_color.isValid() ) { @@ -2234,22 +2235,27 @@ void AppearancePage::MessageTagTab::slotUpdateTagSettingWidgets( int aIndex ) mTextColorCheck->setChecked( false ); } - //3rd row + tmp_color = tmp_desc->backgroundColor(); + mBackgroundColorCheck->setEnabled( true ); + if ( tmp_color.isValid() ) { + mBackgroundColorCombo->setColor( tmp_color ); + mBackgroundColorCheck->setChecked( true ); + } else { + mBackgroundColorCheck->setChecked( false ); + } + QFont tmp_font = tmp_desc->textFont(); mTextFontCheck->setEnabled( true ); mTextFontCheck->setChecked( ( tmp_font != QFont() ) ); mFontRequester->setFont( tmp_font ); - //4th row mIconButton->setEnabled( true ); mIconButton->setIcon( tmp_desc->toolbarIconName() ); - //5th row mKeySequenceWidget->setEnabled( true ); mKeySequenceWidget->setKeySequence( tmp_desc->shortcut().primary(), KKeySequenceWidget::NoValidate ); - //6th row mInToolbarCheck->setEnabled( true ); mInToolbarCheck->setChecked( tmp_desc->inToolbar() ); diff --git a/messageactions.cpp b/messageactions.cpp index ecc194051..bf6aa3721 100644 --- a/messageactions.cpp +++ b/messageactions.cpp @@ -274,7 +274,8 @@ void MessageActions::updateActions() } } mMailingListActionMenu->menu()->clear(); - mMailingListActionMenu->menu()->addTitle( listId ); + if ( !listId.isEmpty() ) + mMailingListActionMenu->menu()->addTitle( listId ); if ( mailList.features() & MailingList::ArchivedAt ) // IDEA: this may be something you want to copy - "Copy in submenu"?