1263: Use this as the parent component for error dialogs

VitaliyBoyko · VitaliyBoyko · commit 5bd9238b8a85 · 2025-03-07T20:35:44.000+02:00
Previously, null was passed as the parent component, which could cause dialog positioning issues. Using `this` ensures the dialog is properly tied to the current window. This improves the user interface consistency and behavior.
diff --git a/src/main/java/com/magento/idea/magento2plugin/actions/generation/dialog/AbstractDialog.java b/src/main/java/com/magento/idea/magento2plugin/actions/generation/dialog/AbstractDialog.java
@@ -218,7 +218,7 @@ protected void showErrorMessage(final String errorMessage) {
             return;
         }
         JOptionPane.showMessageDialog(
-                null,
+                this,
                 errorMessage,
                 errorTitle,
                 JOptionPane.ERROR_MESSAGE

Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,7 @@ protected void showErrorMessage(final String errorMessage) {`
`218`	`218`	`return;`
`219`	`219`	`}`
`220`	`220`	`JOptionPane.showMessageDialog(`
`221`		`- null,`
	`221`	`+ this,`
`222`	`222`	`errorMessage,`
`223`	`223`	`errorTitle,`
`224`	`224`	`JOptionPane.ERROR_MESSAGE`