Merge pull request #75 from HugeNoob/lingxi/help-command

feat: Update help command
AY2324S1-CS2103T-T10-2 · Oct 12, 2023 · a23a154 · a23a154
2 parents c0b7493 + 80deab9
commit a23a154
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 5 deletions.
diff --git a/src/main/java/seedu/address/ui/HelpWindow.java b/src/main/java/seedu/address/ui/HelpWindow.java
@@ -1,5 +1,9 @@
 package seedu.address.ui;
 
+import java.awt.Desktop;
+import java.io.IOException;
+import java.net.URI;
+import java.net.URISyntaxException;
 import java.util.logging.Logger;
 
 import javafx.fxml.FXML;
@@ -9,13 +13,14 @@
 import javafx.scene.input.ClipboardContent;
 import javafx.stage.Stage;
 import seedu.address.commons.core.LogsCenter;
+import seedu.address.commons.util.StringUtil;
 
 /**
  * Controller for a help page
  */
 public class HelpWindow extends UiPart<Stage> {
 
-    public static final String USERGUIDE_URL = "https://se-education.org/addressbook-level3/UserGuide.html";
+    public static final String USERGUIDE_URL = "https://ay2324s1-cs2103t-t10-2.github.io/tp/UserGuide.html";
     public static final String HELP_MESSAGE = "Refer to the user guide: " + USERGUIDE_URL;
 
     private static final Logger logger = LogsCenter.getLogger(HelpWindow.class);
@@ -24,6 +29,9 @@ public class HelpWindow extends UiPart<Stage> {
     @FXML
     private Button copyButton;
 
+    @FXML
+    private Button openButton;
+
     @FXML
     private Label helpMessage;
 
@@ -99,4 +107,21 @@ private void copyUrl() {
         url.putString(USERGUIDE_URL);
         clipboard.setContent(url);
     }
+
+    /**
+     * Opens the user guide URL in the default web browser.
+     */
+    @FXML
+    private void openInBrowser() {
+        try {
+            Desktop.getDesktop().browse(new URI(USERGUIDE_URL));
+        } catch (IOException e) {
+            // Handle the IOException, which may occur when the system cannot open the
+            // default browser.
+            logger.warning("Failed to open URL in default browser: " + StringUtil.getDetails(e));
+        } catch (URISyntaxException e) {
+            // Handle the URISyntaxException, which may occur when the URL is not valid.
+            logger.warning("Invalid URL specified: " + StringUtil.getDetails(e));
+        }
+    }
 }
diff --git a/src/main/resources/view/HelpWindow.css b/src/main/resources/view/HelpWindow.css
@@ -1,16 +1,16 @@
-#copyButton, #helpMessage {
+#copyButton, #openButton, #helpMessage {
     -fx-text-fill: white;
 }
 
-#copyButton {
+#copyButton, #openButton {
     -fx-background-color: dimgray;
 }
 
-#copyButton:hover {
+#copyButton:hover, #openButton:hover {
     -fx-background-color: gray;
 }
 
-#copyButton:armed {
+#copyButton:armed, #openButton:armed {
     -fx-background-color: darkgray;
 }
 

diff --git a/src/main/resources/view/HelpWindow.fxml b/src/main/resources/view/HelpWindow.fxml
@@ -31,6 +31,11 @@
               <Insets left="5.0" />
             </HBox.margin>
           </Button>
+          <Button fx:id="openButton" mnemonicParsing="false" onAction="#openInBrowser" text="Open in browser">
+            <HBox.margin>
+              <Insets left="5.0" />
+            </HBox.margin>
+          </Button>
         </children>
         <opaqueInsets>
           <Insets bottom="10.0" left="5.0" right="10.0" top="5.0" />