Fix up the loading of locales through the StartMenu/Session. Now it will properly make the locale changes *if* it could properly load the locale (ensuring consistency between system/session).

4 files changed, 11 insertions, 6 deletions

diff --git a/lumina-desktop/LSession.cpp b/lumina-desktop/LSession.cpp
index e92dc5a4..12d5b831 100644
--- a/lumina-desktop/LSession.cpp
+++ b/lumina-desktop/LSession.cpp
@@ -689,8 +689,10 @@ WId LSession::activeWindow(){
 
 //Temporarily change the session locale (nothing saved between sessions)
 void LSession::switchLocale(QString localeCode){
-  LUtils::setLocaleEnv(localeCode); //will set everything to this locale (no custom settings)
   currTranslator = LUtils::LoadTranslation(this, "lumina-desktop", localeCode, currTranslator); 
+  if(currTranslator!=0 || localeCode=="en_US"){ 
+    LUtils::setLocaleEnv(localeCode); //will set everything to this locale (no custom settings)
+  }
   emit LocaleChanged();
 }
 
diff --git a/lumina-desktop/panel-plugins/appmenu/LAppMenuPlugin.cpp b/lumina-desktop/panel-plugins/appmenu/LAppMenuPlugin.cpp
index 5e8dc7ac..318d03fa 100644
--- a/lumina-desktop/panel-plugins/appmenu/LAppMenuPlugin.cpp
+++ b/lumina-desktop/panel-plugins/appmenu/LAppMenuPlugin.cpp
@@ -21,7 +21,7 @@ LAppMenuPlugin::LAppMenuPlugin(QWidget *parent, QString id, bool horizontal) : L
 
   connect(mainmenu, SIGNAL(aboutToHide()), this, SIGNAL(MenuClosed()));
   connect(mainmenu, SIGNAL(triggered(QAction*)), this, SLOT(LaunchItem(QAction*)) );
-  connect(LSession::handle()->applicationMenu(), SIGNAL(AppMenuUpdated()), this, SIGNAL(UpdateMenu()));
+  connect(LSession::handle()->applicationMenu(), SIGNAL(AppMenuUpdated()), this, SLOT(UpdateMenu()));
   QTimer::singleShot(0,this, SLOT(OrientationChange())); //Update icons/sizes
   QTimer::singleShot(0,this, SLOT(UpdateMenu()) );
 }
diff --git a/lumina-desktop/panel-plugins/systemdashboard/SysMenuQuick.cpp b/lumina-desktop/panel-plugins/systemdashboard/SysMenuQuick.cpp
index 8ce01c06..534b1b41 100644
--- a/lumina-desktop/panel-plugins/systemdashboard/SysMenuQuick.cpp
+++ b/lumina-desktop/panel-plugins/systemdashboard/SysMenuQuick.cpp
@@ -79,7 +79,7 @@ void LSysMenuQuick::UpdateMenu(){
     QLocale curr;
     for(int i=0; i<locales.length(); i++){
       QLocale loc(locales[i]);
-      ui->combo_locale->addItem(loc.nativeLanguageName(), locales[i]); //Make the display text prettier later
+      ui->combo_locale->addItem(loc.nativeLanguageName()+" ("+locales[i]+")", locales[i]); //Make the display text prettier later
       if(locales[i] == curr.name() || locales[i] == curr.name().section("_",0,0) ){
         //Current Locale
 	ui->combo_locale->setCurrentIndex(ui->combo_locale->count()-1); //the last item in the list right now
diff --git a/lumina-desktop/panel-plugins/systemstart/StartMenu.cpp b/lumina-desktop/panel-plugins/systemstart/StartMenu.cpp
index 043cbba0..c817b1a3 100644
--- a/lumina-desktop/panel-plugins/systemstart/StartMenu.cpp
+++ b/lumina-desktop/panel-plugins/systemstart/StartMenu.cpp
@@ -99,11 +99,13 @@ void StartMenu::UpdateAll(){
   QStringList locales = LUtils::knownLocales();
   ui->stackedWidget->setCurrentWidget(ui->page_main); //need to ensure the settings page is not active
   ui->combo_locale->clear();
-  QLocale curr;
+  QString curr = LUtils::currentLocale();
+  qDebug() << "Update Locales:" << locales;
+  qDebug() << "Current Locale:" << curr;
   for(int i=0; i<locales.length(); i++){
     QLocale loc(locales[i]);
-    ui->combo_locale->addItem(loc.nativeLanguageName(), locales[i]); //Make the display text prettier later
-    if(locales[i] == curr.name() || locales[i] == curr.name().section("_",0,0) ){
+    ui->combo_locale->addItem(loc.nativeLanguageName() +" ("+locales[i]+")", locales[i]); //Make the display text prettier later
+    if(locales[i] == curr || locales[i] == curr.section("_",0,0) ){
       //Current Locale
       ui->combo_locale->setCurrentIndex(ui->combo_locale->count()-1); //the last item in the list right now
     }
@@ -553,6 +555,7 @@ void StartMenu::on_combo_locale_currentIndexChanged(int){
   if(ui->stackedWidget->currentWidget()!=ui->page_settings){ return; }
   QString locale = ui->combo_locale->currentData().toString();
   emit CloseMenu();
+  LSession::processEvents();
   LSession::handle()->switchLocale(locale);
 }