From b3106f936967ce68a4ab8e7a8cb7e47151e7b9e9 Mon Sep 17 00:00:00 2001
From: Karol Lewandowski <karol.lewandowski@jetbrains.com>
Date: Thu, 26 Oct 2023 13:33:02 +0200
Subject: [PATCH] working_with_custom_actions.md, PopupDialogAction.java:
 Cleanup code sample and reference it from the page

---
 .../sdk/action/PopupDialogAction.java         | 30 +++++++------------
 1 file changed, 11 insertions(+), 19 deletions(-)

diff --git a/action_basics/src/main/java/org/intellij/sdk/action/PopupDialogAction.java b/action_basics/src/main/java/org/intellij/sdk/action/PopupDialogAction.java
index 683809318..6649eae9a 100644
--- a/action_basics/src/main/java/org/intellij/sdk/action/PopupDialogAction.java
+++ b/action_basics/src/main/java/org/intellij/sdk/action/PopupDialogAction.java
@@ -50,33 +50,25 @@ public class PopupDialogAction extends AnAction {
     super(text, description, icon);
   }
 
-  /**
-   * Gives the user feedback when the dynamic action menu is chosen.
-   * Pops a simple message dialog. See the psi_demo plugin for an
-   * example of how to use {@link AnActionEvent} to access data.
-   *
-   * @param event Event received when the associated menu item is chosen.
-   */
   @Override
   public void actionPerformed(@NotNull AnActionEvent event) {
     // Using the event, create and show a dialog
     Project currentProject = event.getProject();
-    StringBuilder dlgMsg = new StringBuilder(event.getPresentation().getText() + " Selected!");
-    String dialogTitle = event.getPresentation().getDescription();
+    StringBuilder message =
+        new StringBuilder(event.getPresentation().getText() + " Selected!");
     // If an element is selected in the editor, add info about it.
-    Navigatable navigatable = event.getData(CommonDataKeys.NAVIGATABLE);
-    if (navigatable != null) {
-      dlgMsg.append(String.format("\nSelected Element: %s", navigatable));
+    Navigatable selectedElement = event.getData(CommonDataKeys.NAVIGATABLE);
+    if (selectedElement != null) {
+      message.append("\nSelected Element: ").append(selectedElement);
     }
-    Messages.showMessageDialog(currentProject, dlgMsg.toString(), dialogTitle, Messages.getInformationIcon());
+    String title = event.getPresentation().getDescription();
+    Messages.showMessageDialog(
+        currentProject,
+        message.toString(),
+        title,
+        Messages.getInformationIcon());
   }
 
-  /**
-   * Determines whether this menu item is available for the current context.
-   * Requires a project to be open.
-   *
-   * @param e Event received when the associated group-id menu is chosen.
-   */
   @Override
   public void update(AnActionEvent e) {
     // Set the availability based on whether a project is open