Skip to content

Add font zoom functionality to console view#2579

Open
raghucssit wants to merge 1 commit intoeclipse-platform:masterfrom
raghucssit:console_zoom_support
Open

Add font zoom functionality to console view#2579
raghucssit wants to merge 1 commit intoeclipse-platform:masterfrom
raghucssit:console_zoom_support

Conversation

@raghucssit
Copy link
Contributor

-Add key listener on text widget of console view which listens ctrl plus and control minus(including numpad +/-)
-Zoom in/out of the console view text in steps based on the keys pressed.

see #2578

Copilot AI review requested due to automatic review settings March 18, 2026 21:40
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds Ctrl +/- (including numpad +/-) zoom support to the Eclipse Console view to align behavior with text editors (Issue #2578).

Changes:

  • Installs a key listener on the console’s StyledText to detect Ctrl+Plus / Ctrl+Minus.
  • Implements font resizing logic with min/max bounds and a step size, creating a derived SWT Font.
  • Disposes the created font on view disposal.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 18, 2026

Test Results

    54 files  ±0      54 suites  ±0   34m 43s ⏱️ -25s
 4 546 tests ±0   4 523 ✅ +1   23 💤 ±0  0 ❌  - 1 
12 234 runs  ±0  12 075 ✅ +1  159 💤 ±0  0 ❌  - 1 

Results for commit 21d419f. ± Comparison against base commit 519a3b0.

♻️ This comment has been updated with latest results.

-Add key listener on text widget of console view which listens ctrl plus
and control minus(including numpad +/-)
-Zoom in/out of the console view text in steps based on the keys
pressed.

see eclipse-platform#2578
@raghucssit raghucssit force-pushed the console_zoom_support branch from 74720eb to 21d419f Compare March 19, 2026 15:57

// Install font zoom key listener on the StyledText widget
StyledText textWidget = null;
if (page instanceof TextConsolePage textPage) {
Copy link
Member

Choose a reason for hiding this comment

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

I did not do full review, but looking at this palce here wonder if the code change should be done in TextConsolePage and all init in createControl() there? Because all other pages are irrelevant for the new code.

@iloveeclipse
Copy link
Member

Have you thought about persisting the current zoom over the current session? That will be likely next immediate customer request after using this feature for the first time & IDE restart.

Assuming the code moved to the page (from the console), we would always know "where we are" and should be able to persist the zoom in the preference store by using the IConsole.getType() as part of the key.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants