remove info labels

Author: xLPMG

plugins/SkyCultureMaker/src/SkyCultureMaker.cpp

@@ -428,14 +428,6 @@ void SkyCultureMaker::updateSkyCultureDialog()
 	scmSkyCultureDialog->setConstellations(currentSkyCulture->getConstellations());
 }
 
-void SkyCultureMaker::setSkyCultureDialogInfoLabel(const QString &text)
-{
-	if (scmSkyCultureDialog != nullptr)
-	{
-		scmSkyCultureDialog->setInfoLabel(text);
-	}
-}
-
 void SkyCultureMaker::setSkyCultureDescription(const scm::Description &description)
 {
 	if (currentSkyCulture != nullptr)
@@ -585,3 +577,24 @@ void SkyCultureMaker::initSetting(QSettings *conf, const QString key, const QVar
 		conf->setValue(key, defaultValue);
 	}
 }
+
+void SkyCultureMaker::showUserInfoMessage(QWidget *parent, const QString &dialogName, const QString &message)
+{
+	const QString level = q_("INFO");
+	const QString title = dialogName.isEmpty() ? level : dialogName + ": " + level;
+	QMessageBox::information(parent, title, message);
+}
+
+void SkyCultureMaker::showUserWarningMessage(QWidget *parent, const QString &dialogName, const QString &message)
+{
+	const QString level = q_("WARNING");
+	const QString title = dialogName.isEmpty() ? level : dialogName + ": " + level;
+	QMessageBox::warning(parent, title, message);
+}
+
+void SkyCultureMaker::showUserErrorMessage(QWidget *parent, const QString &dialogName, const QString &message)
+{
+	const QString level = q_("ERROR");
+	const QString title = dialogName.isEmpty() ? level : dialogName + ": " + level;
+	QMessageBox::critical(parent, title, message);
+}

plugins/SkyCultureMaker/src/SkyCultureMaker.hpp

@@ -37,6 +37,7 @@
 #include <QDir>
 #include <QFile>
 #include <QFont>
+#include <QMessageBox>
 
 class QPixmap;
 class StelButton;
@@ -177,12 +178,6 @@ class SkyCultureMaker : public StelModule
 	 */
 	void updateSkyCultureDialog();
 
-	/**
-	 * @brief Sets the info label text in the sky culture dialog.
-	 * @param text The text to set in the info label.
-	 */
-	void setSkyCultureDialogInfoLabel(const QString &text);
-
 	/**
 	 * @brief Sets the current sky culture description.
 	 * @param description The description to set.
@@ -238,6 +233,35 @@ class SkyCultureMaker : public StelModule
 	 */
 	void openConstellationDialog(const QString &constellationId);
 
+	/**
+	 * @brief Displays an information message to the user.
+	 * 
+	 * @param parent The parent widget of the message box.
+	 * @param dialogName The name of the dialog to be shown in the title bar.
+	 * @param message The message to be displayed.
+	 */
+	void showUserInfoMessage(QWidget *parent, const QString &dialogName, const QString &message);
+
+	/**
+	 * @brief Displays a warning message to the user.
+	 * 
+	 * @param parent The parent widget of the message box.
+	 * @param dialogName The name of the dialog to be shown in the title bar.
+	 * @param message The message to be displayed.
+	 */
+	void showUserWarningMessage(QWidget *parent, const QString &dialogName, const QString &message);
+
+	/**
+	 * @brief Displays an error message to the user.
+	 * 
+	 * @param parent The parent widget of the message box.
+	 * @param dialogName The name of the dialog to be shown in the title bar.
+	 * @param message The message to be displayed.
+	 */
+	void showUserErrorMessage(QWidget *parent, const QString &dialogName, const QString &message);
+
+	
+
 signals:
 	void eventIsScmEnabled(bool b);
 

plugins/SkyCultureMaker/src/gui/ScmConstellationDialog.cpp

@@ -115,7 +115,7 @@ void ScmConstellationDialog::setIsDarkConstellation(bool isDark)
 	}
 
 	// the value changed, so we should reset some data from the previous mode
-	if(isDarkConstellation != isDark)
+	if (isDarkConstellation != isDark)
 	{
 		// reset drawn lines as they are not compatible between modes
 		draw->resetDrawing();
@@ -241,10 +241,6 @@ void ScmConstellationDialog::createDialogContent()
 
 void ScmConstellationDialog::handleFontChanged()
 {
-	QFont infoLblFont = QApplication::font();
-	infoLblFont.setBold(true);
-	ui->infoLbl->setFont(infoLblFont);
-
 	QFont labelsTitleFont = QApplication::font();
 	labelsTitleFont.setPixelSize(labelsTitleFont.pixelSize() + 2);
 	labelsTitleFont.setBold(true);
@@ -300,21 +296,20 @@ void ScmConstellationDialog::triggerUploadImage()
 
 	if (!fileInfo.isFile())
 	{
-		ui->infoLbl->setText(q_("Choosen path is not a valid file:\n") + filePath);
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("Chosen path is not a valid file:\n") + filePath);
 		return;
 	}
 
 	if (!(fileInfo.suffix().compare("PNG", Qt::CaseInsensitive) == 0 ||
 	      fileInfo.suffix().compare("JPG", Qt::CaseInsensitive) == 0 ||
 	      fileInfo.suffix().compare("JPEG", Qt::CaseInsensitive) == 0))
 	{
-		ui->infoLbl->setText(q_("Chosen file is not a PNG, JPG or JPEG image:\n") + filePath);
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("Chosen file is not a PNG, JPG or JPEG image:\n") + filePath);
 		return;
 	}
 
-	// Reset text
-	ui->infoLbl->setText("");
-
 	QPixmap image = QPixmap(fileInfo.absoluteFilePath());
 	imageItem->setImage(image);
 	imageItem->show();
@@ -337,7 +332,8 @@ void ScmConstellationDialog::bindSelectedStar()
 {
 	if (!imageItem->hasAnchorSelection())
 	{
-		ui->infoLbl->setText(q_("WARNING: Select an anchor to bind to."));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("No anchor was selected. Please select an anchor to bind to."));
 		qDebug() << "SkyCultureMaker: No anchor was selected.";
 		return;
 	}
@@ -347,38 +343,41 @@ void ScmConstellationDialog::bindSelectedStar()
 
 	if (!objectMgr.getWasSelected())
 	{
-		ui->infoLbl->setText(q_("WARNING: Select a star to bind to the current selected anchor."));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("No star was selected to bind to the current selected anchor."));
 		qDebug() << "SkyCultureMaker: No star was selected to bind to.";
 		return;
 	}
 
 	StelObjectP stelObj = objectMgr.getLastSelectedObject();
 	assert(stelObj != nullptr); // Checked through getWasSelected
-	if (stelObj->getType().compare("star", Qt::CaseInsensitive) != 0)
+	if (stelObj->getType().compare("star", Qt::CaseInsensitive) != 0 &&
+	    stelObj->getType().compare("nebula", Qt::CaseInsensitive) != 0)
 	{
-		ui->infoLbl->setText(q_("WARNING: The selected object must be of type star."));
-		qDebug() << "SkyCultureMaker: The selected object is not of type start, got " << stelObj->getType();
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("The selected object must be of type Star or Nebula."));
+		qDebug() << "SkyCultureMaker: The selected object is not of type Star, got " << stelObj->getType();
 		return;
 	}
 
 	ScmConstellationImageAnchor *anchor = imageItem->getSelectedAnchor();
 	if (anchor == nullptr)
 	{
-		ui->infoLbl->setText(q_("WARNING: No anchor is selected."));
-		qDebug() << "SkyCultureMaker: No anchor is selected";
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("No anchor was selected. Please select an anchor to bind to."));
+		qDebug() << "SkyCultureMaker: No anchor was selected";
 		return;
 	}
 
 	bool success = anchor->trySetStarHip(stelObj->getID());
 	if (success == false)
 	{
-		ui->infoLbl->setText(q_("WARNING: The selected object must contain a HIP number."));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("The selected object must contain a HIP number."));
 		qDebug() << "SkyCultureMaker: The object does not contain a HIP, id = " << stelObj->getID();
 		return;
 	}
 
-	ui->infoLbl->setText(""); // Reset
-
 	updateArtwork();
 }
 
@@ -396,14 +395,16 @@ bool ScmConstellationDialog::canConstellationBeSaved() const
 	scm::ScmSkyCulture *currentSkyCulture = maker->getCurrentSkyCulture();
 	if (currentSkyCulture == nullptr)
 	{
-		ui->infoLbl->setText(q_("WARNING: Could not save: Sky Culture is not set"));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("Could not save: Sky Culture is not set"));
 		qDebug() << "SkyCultureMaker: Could not save: Sky Culture is not set";
 		return false;
 	}
 
 	if (constellationEnglishName.isEmpty())
 	{
-		ui->infoLbl->setText(q_("WARNING: Could not save: English name is empty"));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("Could not save: English name is empty"));
 		qDebug() << "SkyCultureMaker: Could not save: English name is empty";
 		return false;
 	}
@@ -412,15 +413,17 @@ bool ScmConstellationDialog::canConstellationBeSaved() const
 	QString finalId = constellationId.isEmpty() ? constellationPlaceholderId : constellationId;
 	if (finalId.isEmpty())
 	{
-		ui->infoLbl->setText(q_("WARNING: Could not save: Constellation ID is empty"));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("Could not save: Constellation ID is empty"));
 		qDebug() << "SkyCultureMaker: Could not save: Constellation ID is empty";
 		return false;
 	}
 
 	// Not editing a constellation, but the ID already exists
 	if (constellationBeingEdited == nullptr && currentSkyCulture->getConstellation(finalId) != nullptr)
 	{
-		ui->infoLbl->setText(q_("WARNING: Could not save: Constellation with this ID already exists"));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("Could not save: Constellation with this ID already exists"));
 		qDebug() << "SkyCultureMaker: Could not save: Constellation with this ID already exists, id = "
 			 << finalId;
 		return false;
@@ -429,7 +432,8 @@ bool ScmConstellationDialog::canConstellationBeSaved() const
 	else if (constellationBeingEdited != nullptr && constellationBeingEdited->getId() != finalId &&
 	         currentSkyCulture->getConstellation(finalId) != nullptr)
 	{
-		ui->infoLbl->setText(q_("WARNING: Could not save: Constellation with this ID already exists"));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("Could not save: Constellation with this ID already exists"));
 		qDebug() << "SkyCultureMaker: Could not save: Constellation with this ID already exists, id = "
 			 << finalId;
 		return false;
@@ -439,7 +443,8 @@ bool ScmConstellationDialog::canConstellationBeSaved() const
 	auto drawnConstellation = maker->getScmDraw()->getCoordinates();
 	if (drawnConstellation.empty())
 	{
-		ui->infoLbl->setText(q_("WARNING: Could not save: The constellation does not contain any drawings"));
+		maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+		                            q_("Could not save: The constellation does not contain any drawings"));
 		qDebug() << "SkyCultureMaker: Could not save: The constellation does not contain any drawings";
 		return false;
 	}
@@ -449,8 +454,9 @@ bool ScmConstellationDialog::canConstellationBeSaved() const
 	{
 		if (!imageItem->isImageAnchored())
 		{
-			ui->infoLbl->setText(q_("WARNING: Could not save: An artwork is attached, but not all "
-			                        "anchors have a star bound."));
+			maker->showUserErrorMessage(this->dialog, ui->titleBar->title(),
+			                            q_("Could not save: An artwork is attached, but not all "
+			                               "anchors have a star bound."));
 			qDebug() << "SkyCultureMaker: Could not save: An artwork is attached, but not all "
 				    "anchors have a star bound.";
 			return false;

plugins/SkyCultureMaker/src/gui/ScmSkyCultureDialog.cpp

@@ -176,13 +176,13 @@ void ScmSkyCultureDialog::saveSkyCulture()
 	// check if license is set
 	if (desc.license == scm::LicenseType::NONE)
 	{
-		ui->infoLbl->setText(q_("WARNING: Please select a license for the sky culture."));
+		maker->showUserWarningMessage(dialog, ui->titleBar->title(), q_("Please select a license for the sky culture."));
 		return;
 	}
 	// check if description is complete
 	if (!desc.isComplete())
 	{
-		ui->infoLbl->setText(q_("WARNING: The sky culture description is not complete."));
+		maker->showUserWarningMessage(dialog, ui->titleBar->title(), q_("The sky culture description is not complete. Please fill in all required fields."));
 		return;
 	}
 
@@ -321,18 +321,6 @@ scm::Description ScmSkyCultureDialog::getDescriptionFromTextEdit() const
 	return desc;
 }
 
-void ScmSkyCultureDialog::setInfoLabel(const QString &text)
-{
-	if (ui && dialog)
-	{
-		ui->infoLbl->setText(text);
-	}
-	else
-	{
-		qDebug() << "ScmSkyCultureDialog: UI or dialog is not initialized.";
-	}
-}
-
 void ScmSkyCultureDialog::resetDialog()
 {
 	if (ui && dialog)
@@ -357,7 +345,6 @@ void ScmSkyCultureDialog::resetDialog()
 		name.clear();
 		setIdFromName(name);
 		resetConstellations();
-		maker->setSkyCultureDialogInfoLabel("");
 		maker->setSkyCultureDescription(getDescriptionFromTextEdit());
 		updateRemoveConstellationButton();
 	}

plugins/SkyCultureMaker/src/gui/ScmSkyCultureDialog.hpp

@@ -59,13 +59,6 @@ class ScmSkyCultureDialog : public StelDialogSeparate
 	 */
 	void resetConstellations();
 
-	/**
-	 * @brief Sets the info label text.
-	 *
-	 * @param text The text to set in the info label.
-	 */
-	void setInfoLabel(const QString &text);
-
 	/**
 	 * @brief Resets all fields in the dialog to their default values.
 	 */