diff -r 22d3abee08da -r 2883a5458389 controlpanelui/src/cpplugins/profileactivatorplugin/src/cpprofileactivatordialog.cpp --- a/controlpanelui/src/cpplugins/profileactivatorplugin/src/cpprofileactivatordialog.cpp Tue May 25 04:10:19 2010 +0800 +++ b/controlpanelui/src/cpplugins/profileactivatorplugin/src/cpprofileactivatordialog.cpp Thu Jun 10 16:07:41 2010 +0800 @@ -20,7 +20,11 @@ #include #include #include +#include #include "cpprofileactivatorentryitem.h" +#include +#include + CpProfileActivatorDialog::CpProfileActivatorDialog(CpSettingFormItemData *profileActivator, CpProfileModel &profileModel, QGraphicsItem *parent):HbDialog(parent), @@ -28,11 +32,19 @@ mProfileList(0), mProfileActivator(profileActivator) { + mTitleLabel = new HbLabel(this); + mTitleLabel->setPlainText(hbTrId("txt_cp_title_profile")); + mContentWidget = new QGraphicsWidget(this); + setContentWidget(mContentWidget); + mLayout = new QGraphicsLinearLayout(); + mLayout->setOrientation( Qt::Vertical ); mProfileIds << EProfileWrapperGeneralId << EProfileWrapperMeetingId; mProfileList = new HbRadioButtonList(); mProfileList->setItems(profileModel.profileNames()); - + mLayout->addItem(mTitleLabel); + mLayout->addItem(mProfileList); + int currentId = mProfileModel.activeProfileId(); mProfileList->setSelected(mProfileIds.indexOf(static_cast(currentId))); @@ -46,12 +58,11 @@ this->addAction(mConfirmProfile); - this->addAction(mCancelProfile); - - this->setContentWidget(mProfileList); + this->addAction(mCancelProfile); this->setModal(true); this->setDismissPolicy(HbPopup::NoDismiss); this->setTimeout(HbPopup::NoTimeout); + mContentWidget->setLayout( mLayout ); } CpProfileActivatorDialog::~CpProfileActivatorDialog() @@ -61,6 +72,9 @@ void CpProfileActivatorDialog::confirmProfileSelection() { int currentIndex = mProfileList->selected(); + if (currentIndex == -1) { + return; + } // the best choice is no need to convert the index to id mProfileModel.activateProfile(mProfileIds.at(currentIndex));