#1933 Added GUI console pane - #2053

Open

laim2003 wants to merge 89 commits into

devonfw:mainfrom

laim2003:#1933-gui-console-window

laim2003 commented Jun 19, 2026 •

edited

Loading

Contributor

This PR fixes #1933

Implemented changes:

Added a console panel in the GUI.

Refactor usage of ProcessOutput/IdeLogListener/discuss architecture with @hohwille

Testing instructions

Please add conscise, understandable instructions on how a reviewer can test/verify the functionality of your contribution here:

1a. (Team) Review

Run AppLauncher.java

1b. Retesting

Run ide gui

2. Then:

Console pane should be visible if you click the terminal icon in the bottom left
You should be able to resize the console by dragging the top of the panel
When running a commandlet (e.g. launching an IDE), the console should automatically become visible
Log outputs should be shown in the console
The "line count" should update.
When clicking "clear": Console should be cleared
While "Auto scroll" is checked: The console should always show the newest log message.
When clicking the terminal button, this should toggle the visibility of the console.

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

When running mvn clean test locally all tests pass and build is successful
PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat). If no issue ID exists, title only.
PR top-level comment summarizes what has been done and contains link to addressed issue(s)
PR and issue(s) have suitable labels
Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
You followed all coding conventions
You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labeled
with internal
You have formulated clear instructions on how to test your contribution under "Testing instructions"

laim2003 added 30 commits

March 27, 2026 17:52


          devonfw#1296: implemented basic IdeDialog Class

029f03d


          devonfw#1296: Added basic context state manager class

89cfb67

- Added testing class for modals


          devonfw#1785: Added basic question/confirmation modal implementation

a53296d

- Added logging to IdeGuiStateManager.
- Added functionality, that selecting a different project now switches the IdeContext to the new project.


          devonfw#1296: implemented basic IdeDialog Class

37cad74


          devonfw#1296: Added basic context state manager class

55a50ce

- Added testing class for modals


          devonfw#1785: Added basic question/confirmation modal implementation

3f1569f

- Added logging to IdeGuiStateManager.
- Added functionality, that selecting a different project now switches the IdeContext to the new project.


          Merge remote-tracking branch 'origin/devonfw#1296-implement-idecontex…

6ce99b3

…t-for-gui' into devonfw#1785-implement-modals-in-idecontext


          devonfw#1785: - Added logging to IdeGuiStateManager.

7ef65f3

- Added functionality, that selecting a different project now switches the IdeContext to the new project.


          devonfw#1724: implemented basic gui state management

1764b2f


          devonfw#1802: implemented basic gui state management

ddd3e5f


          Merge remote-tracking branch 'origin/devonfw#1802-state-management-im…

9c5a720

…plementation' into devonfw#1802-state-management-implementation


          devonfw#1802: removed a few changes so this branch can be the base fo…

b28e216

…r other ui feature branches


          devonfw#1802: disabled javadoc warning

234e98b


          devonfw#1802: - fixed AppBaseTest being stuck

1c4e766

- added DI for IdeGuiStateManager.switchContext


          Merge branch 'main' into devonfw#1802-state-management-implementation

6fc3884


          devonfw#1802: Added ProjectManager class to handle business logic of …

…reading the list of workspaces/projects instead of reading those from the UI


          devonfw#1802: Updated IdeGuiStateManager to also update its projectMa…

c76e185

…nager, when switchContext(Path rootDirectory, ...) is called.


          devonfw#1802: javafx updated to java 25

6f92d93


          Merge branch 'main' into devonfw#1802-state-management-implementation

9f741a5


          Revert "devonfw#1802: javafx updated to java 25"

48fe018

This reverts commit 6f92d93.


          devonfw#1803: added DI for IDE_ROOT in GuiStateManager

e0ad015


          devonfw#1802: added DI for IDE_ROOT in GuiStateManager

2c9fc55


          Merge remote-tracking branch 'origin/devonfw#1802-state-management-im…

2b62c60

…plementation' into devonfw#1802-state-management-implementation


          devonfw#1802: Workaround for IDE_ROOT=null issue. The ideRoot in GuiS…

40ff54e

…tateManager is now set when calling getInstance(), allowing us to provide a getInstance() method with a DI parameter


          devonfw#1802: removed redundant DI variant of switchContext() (DI via…

0b67210

… getInstance()) (see previous commit)


          devonfw#1802: Added tests for GuiStateManager

2e15d60


          Merge branch 'main' into devonfw#1802-state-management-implementation

10cf940


          devonfw#1802: Extracted mock IDE_ROOT logic into seperate class that …

5f0b40a

…can be extended by tests


          Merge remote-tracking branch 'origin/devonfw#1802-state-management-im…

3b567ef

…plementation' into devonfw#1802-state-management-implementation


          devonfw#1802: added ProjectManagerTest

e406b65

laim2003 added 5 commits

August 3, 2026 16:06


          devonfw#1933: spotless plugin applied

d084f52

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: merged main layout

90af675

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: applied differentiation recommendation by @hohwille

daec30a

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: small layout fixes

98ec109

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: added tooltip & translation

8f5a08d

Signed-off-by: laim2003 <luk.faber@gmx.de>

laim2003 moved this from 👀 In review to 🏗 In progress in IDEasy board

laim2003 added 3 commits

August 3, 2026 20:39


          Merge branch 'main' into devonfw#1933-gui-console-window

c134c63


          devonfw#1933: removed redundant runLater wrapper

019291a

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: switched console to use buffered log messages to increa…

0fff6bd

…se UI performance

Signed-off-by: laim2003 <luk.faber@gmx.de>

laim2003 commented Aug 4, 2026

Contributor Author

I'm currently working on migrating the implementation to a ListView based implementation. This allows us much more flexibility in working with log messages such as filtering etc. I think it makes sense to get this right from the start, as otherwise we will have to completely overhaul this later on.

laim2003 marked this pull request as draft

August 4, 2026 09:05

laim2003 added 10 commits

August 4, 2026 11:56


          devonfw#1933: added translations for console

f38b8d5

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: migrated from TextArea to ListView for performance and …

05ae1cd

…felxibility reasons.

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: updated tests with new ConsoleController Structure

4a6bb22

Signed-off-by: laim2003 <luk.faber@gmx.de>


          Merge remote-tracking branch 'origin/devonfw#1933-gui-console-window'…

e3d2f51

… into devonfw#1933-gui-console-window


          devonfw#1933: applied spotless plugin

13d4896

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: renamed GuiLogEntry for naming convention purposes

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: added log entry wrapping to connect IdeLogEntries with …

219fe2e

…timestamps

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: updated ConsolePanelTest

5a6ca76

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: applied spotless plugin

0b05d96

Signed-off-by: laim2003 <luk.faber@gmx.de>


          devonfw#1933: moved away from java.util.Date

9e919dd

Signed-off-by: laim2003 <luk.faber@gmx.de>

laim2003 requested a review from hohwille

August 4, 2026 14:54

laim2003 moved this from 🏗 In progress to 👀 In review in IDEasy board

laim2003 marked this pull request as ready for review

August 4, 2026 14:55


          devonfw#1933: small fix

c1cc4f9

Signed-off-by: laim2003 <luk.faber@gmx.de>

laim2003 commented Aug 5, 2026 •

edited

Loading

Contributor Author

I just also realized what this new listView-based design allows us to do: Because we handle any Log output in the console via ListItems, we can fully customize what each list item displays. This allows us to integrate dialogues and choices into the console (potentially). As a first step, I will create a Figma design.

hohwille modified the milestones: release:2026.08.001, release:2026.08.002


          Merge remote-tracking branch 'upstream/main' into devonfw#1933-gui-co…

98fb1fb

…nsole-window

# Conflicts:
#	CHANGELOG.adoc

hohwille approved these changes

View reviewed changes

hohwille left a comment

Member

@laim2003 thanks for your PR. This is great progress and you also did some nice cleanups alongside. 👍
Surely in the GUI there is still some experimental code that needs some redesign but this is a good improvement I am happy to merge. However, beside my review comments there are currently some merge conflicts.

gui/src/main/java/com/devonfw/ide/gui/console/GuiLogEntryWrapper.java

+               * The wrapper stores the original log entry and a timestamp (in milliseconds since the epoch). It offers getters for both values and a {@code toString()}
+               * implementation that formats the timestamp and the log entry level and message for display.
+               */
+              public class GuiLogEntryWrapper {

hohwille Aug 21, 2026

Member

Wouldn't it make more sense, to add a timestamp field to IdeLogEntry?
I do not see what is GUI specific about this class.

gui/src/main/java/com/devonfw/ide/gui/MainController.java

+                    protected Void call() {
+                      try {
+                        IdeStartContextImpl startContext = new IdeStartContextImpl(IdeLogLevel.INFO, guiLogListener);

hohwille Aug 21, 2026

Member

Currently the context only supports a single log listener.
You are replacing the default one with your GUI.
If I am not mistaken, this will break features like deactivateLogging and activateLogging.
Maybe we should consider adding further optional listeners...
And we could also think about reusing the IdeLogEvent for that so we do not have to create multiple instances of it for the same log message.

gui/src/main/java/com/devonfw/ide/gui/MainController.java


		context.getCommandletManager().getCommandlet(commandlet).run();

		LOG.info("[GUI] === {} ran successfully. ===", commandlet);

hohwille Aug 21, 2026

Member

Suggested change

      
                      LOG.info("[GUI] === {} ran successfully. ===", commandlet);
          
                      LOG.info("[GUI] === {} completed successfully. ===", commandlet);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GUI