# HG changeset patch
# User Matti Hamalainen <ccr@tnsp.org>
# Date 1490890228 -10800
# Node ID 8282142605e080caf71b9d4860638953153e0301
# Parent  d40345d6373318c45bea1978710fa134a655ffdc
Cleanups.

diff -r d40345d63733 -r 8282142605e0 main.cpp
--- a/main.cpp	Thu Mar 30 18:37:49 2017 +0300
+++ b/main.cpp	Thu Mar 30 19:10:28 2017 +0300
@@ -11,9 +11,18 @@
 #include "ui_editperson.h"
 
 
-void errorMsg(QString title, QString msg)
+int errorMsg(QString title, QString msg)
 {
-    QMessageBox::critical(0, title, msg, QMessageBox::Ok);
+    QMessageBox dlg;
+
+    dlg.setText(title);
+    dlg.setInformativeText(msg);
+    dlg.setTextFormat(Qt::RichText);
+    dlg.setIcon(QMessageBox::Critical);
+    dlg.setStandardButtons(QMessageBox::Ok);
+    dlg.setDefaultButton(QMessageBox::Ok);
+    
+    return dlg.exec();
 }
 
 
@@ -115,7 +124,7 @@
     {
         errorMsg(
             QObject::tr("Tietokantaa ei voitu avata"),
-            QObject::tr("Yhteyttä SQL-tietokantaan ei saatu.\n\nVirhe: %1\n\n").
+            QObject::tr("Yhteyttä SQL-tietokantaan ei saatu.<br><br>Virhe: %1<br><br>").
             arg(db.lastError().text())
             );
         return 1;
@@ -337,15 +346,22 @@
         return;
     }
 
-    QMessageBox::StandardButton ret =
-        QMessageBox::question(this,
-        tr("Varmistus"),
-        tr("\nHaluatko varmasti poistaa henkilön:\n\n'%1, %2' (ID #%3)?\n\n"
-        "Tämä poistaa sekä henkilön ja hänen koko tapahtumahistoriansa PYSYVÄSTI!\n").
-        arg(info.lastName).arg(info.firstName).arg(info.id),
-        QMessageBox::Yes | QMessageBox::No);
+    QMessageBox dlg;
+    dlg.setText(tr("Varmistus"));
+    dlg.setInformativeText(
+        tr("<br>Haluatko varmasti poistaa henkilön:<br>"
+        "<br>"
+        "<b>'%1, %2'</b> <i>(ID #%3)</i>?<br>"
+        "<br>"
+        "Tämä poistaa sekä henkilön ja hänen koko tapahtumahistoriansa PYSYVÄSTI!<br>").
+        arg(info.lastName).arg(info.firstName).arg(info.id));
 
-    if (ret == QMessageBox::Yes)
+    dlg.setTextFormat(Qt::RichText);
+    dlg.setIcon(QMessageBox::Question);
+    dlg.setStandardButtons(QMessageBox::Yes | QMessageBox::No);
+    dlg.setDefaultButton(QMessageBox::No);
+
+    if (dlg.exec() == QMessageBox::Yes)
     {
         int rv = model_People->deletePerson(info.id);
         updatePersonList();
@@ -669,10 +685,9 @@
 {
     if (!validateForm())
     {
-        QMessageBox::critical(0,
+        errorMsg(
             tr("Virhe!"),
-            tr("Vaaditut kentät (etunimi, sukunimi) eivät ole täytetty."),
-            QMessageBox::Ok);
+            tr("Vaaditut kentät (etunimi, sukunimi) eivät ole täytetty."));
 
         return;
     }