Skip to content

New links in "Help" menu for issues and forum #1210

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Aug 22, 2025
Merged

New links in "Help" menu for issues and forum #1210

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3d79fc0
help menu urls #1208
lassevonpfeil Aug 15, 2025
9a96dd6
added language support #1208
lassevonpfeil Aug 15, 2025
f4d8557
Merge branch 'processing:main' into main
lassevonpfeil Aug 20, 2025
37db555
updated from desktop.browse() to Platform.openURL()
lassevonpfeil Aug 20, 2025
c1a3cec
removed unused imports
lassevonpfeil Aug 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions build/shared/lib/languages/PDE.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,8 @@ menu.help.reference.download = Download Offline Reference
menu.help.libraries_reference = Libraries Reference
menu.help.tools_reference = Tools Reference
menu.help.empty = (empty)
menu.help.report = Report a bug
menu.help.ask = Ask on the Forum
menu.help.online = Online

# Only include the .url lines in the translation file if there
Expand All @@ -167,6 +169,8 @@ menu.help.foundation = The Processing Foundation
menu.help.foundation.url = https://processing.foundation/
menu.help.visit = Visit Processing.org
menu.help.visit.url = https://processing.org/
menu.help.report.url = https://github.com/processing/processing4/issues
menu.help.ask.url = https://discourse.processing.org


# ---------------------------------------
Expand Down
2 changes: 2 additions & 0 deletions build/shared/lib/languages/PDE_de.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,8 @@ menu.help.troubleshooting = Fehlerbehandlung
menu.help.faq = Häufig gestellte Fragen (FAQ)
menu.help.foundation = "The Processing Foundation"
menu.help.visit = Processing.org besuchen
menu.help.report = Einen Fehler melden
menu.help.ask = Im Forum fragen


# ---------------------------------------
Expand Down
15 changes: 15 additions & 0 deletions java/src/processing/mode/java/JavaEditor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@
import java.util.zip.ZipEntry;
import java.util.zip.ZipOutputStream;

import java.net.URI;
import java.net.URISyntaxException;

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these necessary anymore? If not please remove, and then this is ready to be approved! 🎉

import javax.swing.*;
import javax.swing.event.*;
import javax.swing.text.BadLocationException;
Expand Down Expand Up @@ -327,6 +330,18 @@ public JMenu buildHelpMenu() {

menu.addSeparator();

// Report a bug link opener
item = new JMenuItem(Language.text("menu.help.report"));
item.addActionListener(e -> Platform.openURL(Language.text("menu.help.report.url")));
menu.add(item);

// Ask on the Forum link opener
item = new JMenuItem(Language.text("menu.help.ask"));
item.addActionListener(e -> Platform.openURL(Language.text("menu.help.getting_started.url")));
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @lassevonpfeil. I just caught while testing: the URL here should have been menu.help.ask.url instead of menu.help.getting_started.url. Would you mind making another PR for this small fix? Thanks! cc @catilac

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @lassevonpfeil just checking in. Would you be able to confirm whether or not you'll be able to make this small PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, sorry for the delay. Of course, will work on it asap.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem at all! Just wanted to check in. Thank you so much, @lassevonpfeil :)

menu.add(item);

menu.addSeparator();

final JMenu libRefSubmenu = new JMenu(Language.text("menu.help.libraries_reference"));

// Adding this in case references are included in a core library,
Expand Down