Skip to content

[OGUI-1216] Overrise browser-zoom to increase log size only#3465

Open
isaachilly wants to merge 26 commits into
devfrom
feature/ILG/OGUI-1216/Override-browser-zoom-to-increase-log-size-only
Open

[OGUI-1216] Overrise browser-zoom to increase log size only#3465
isaachilly wants to merge 26 commits into
devfrom
feature/ILG/OGUI-1216/Override-browser-zoom-to-increase-log-size-only

Conversation

@isaachilly
Copy link
Copy Markdown
Collaborator

@isaachilly isaachilly commented May 21, 2026

I have JIRA issue created

  • branch and/or PR name(s) includes JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected
  • FLP integration tests were ran successful
  • Intercept Ctrl/CMD plus scroll and Ctrl/CMD plus +/- to zoom the log table font and row height independently of the browser, keeping the rest of the UI at normal scale
  • Zoom state has own model imported into Model
  • CSS custom properties -log-font-size and --row-height derive from the Zoom model
  • Added a minus, reset and add zoom buttons in the toolbar
  • Added a test suite covering keyboard zoom, mouse wheel zoom, button zoom, min/max clamping, reset button state, and constant row-height-to-font-size ratio

Intercept Ctrl+scroll and Ctrl+/- to zoom the log table independently of the browser.

Zoom state is tracked in the model, with font size and row height both being derived in rem from a multiplier.

Added a reset button in the toolbar.
Tests cover:
- Keyboard zoom
- Mouse wheel zoom
- Min/max clamping
- Reset button
- Preserve zoom ratio
@isaachilly isaachilly self-assigned this May 21, 2026
@isaachilly isaachilly added Frontend InfoLogger javascript Pull requests that update javascript code labels May 21, 2026
@isaachilly isaachilly marked this pull request as ready for review May 21, 2026 06:50
@isaachilly isaachilly requested a review from graduta as a code owner May 21, 2026 06:50
Comment thread InfoLogger/public/log/commandLogs.js Outdated
Comment thread InfoLogger/public/view.js Outdated
isaachilly and others added 6 commits May 22, 2026 14:55
Added a global wheel listener and handleWheel method to Model.
Replace the single reset-zoom button with a .btn-group.

Make space by taking away Reset text and not using icons.

Adds new tests covering clicking +/− changes the zoom level, and that the +/- buttons become disabled at min/max and enabled at the default zoom.
Co-Authored-By: George R. <9214854+graduta@users.noreply.github.com>
Co-Authored-By: George R. <9214854+graduta@users.noreply.github.com>
Comment thread InfoLogger/test/public/zoom.mocha.js Fixed
Comment thread InfoLogger/test/public/zoom.mocha.js Fixed
Co-Authored-By: George R. <9214854+graduta@users.noreply.github.com>
@isaachilly isaachilly requested a review from graduta May 22, 2026 15:35
Co-Authored-By: George R. <9214854+graduta@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend InfoLogger javascript Pull requests that update javascript code

Development

Successfully merging this pull request may close these issues.

3 participants