Skip to content

Release v1.0.8: Merge develop into main - #11

Merged
Kiolk merged 7 commits into
mainfrom
develop
Feb 16, 2026
Merged

Release v1.0.8: Merge develop into main#11
Kiolk merged 7 commits into
mainfrom
develop

Conversation

@Kiolk

@Kiolk Kiolk commented Feb 16, 2026

Copy link
Copy Markdown
Owner

This PR merges all changes from the develop branch into main for release.

Key changes included:

Closes #9

Summary by CodeRabbit

  • New Features

    • Added font size resizing using Ctrl+mouse wheel for improved text readability
    • Font size preferences are now saved and restored per project
    • Dialog window automatically resizes based on content and cursor visibility
  • Documentation

    • Added workflow documentation for issue processing and implementation planning

@coderabbitai

coderabbitai Bot commented Feb 16, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@Kiolk has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 22 minutes and 35 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

Implements Ctrl+mouse wheel font resizing with persistent storage. Adds fontSize property to TypingService for per-project persistence, updates TypingDialog to capture mouse wheel events with Ctrl modifier, dynamically adjust font size and window dimensions while maintaining cursor visibility and centering.

Changes

Cohort / File(s) Summary
Configuration & Documentation
.gitignore, AGENTS.md, plans/issue/issue-6-plan.md
Updated gitignore pattern, added AI agent workflow guidelines, and stored implementation plan for font resizing feature.
Service Layer
src/main/kotlin/com/github/kiolk/typingplugin/service/TypingService.kt
Added fontSize property to State data class with default value 14, plus getter/setter methods for font size persistence across sessions.
UI Implementation
src/main/kotlin/com/github/kiolk/typingplugin/ui/TypingDialog.kt
Introduced centerPanel for layout control, integrated TypingService for font persistence, added Ctrl+wheel zoom event handling, implemented dynamic window resizing with screen bounds clamping, and enhanced cursor management with auto-scrolling and padding.

Sequence Diagram

sequenceDiagram
    participant User
    participant TypingDialog
    participant TypingService
    participant UI as TextPane/Window

    User->>TypingDialog: Ctrl + Mouse Wheel
    TypingDialog->>TypingDialog: Capture MouseWheelEvent
    TypingDialog->>TypingDialog: Calculate new fontSize
    TypingDialog->>TypingService: setFontSize(newSize)
    TypingService->>TypingService: Update state.fontSize
    TypingDialog->>UI: Apply new Font to textPane
    TypingDialog->>TypingDialog: updateWindowSize()
    TypingDialog->>TypingDialog: Clamp to screen bounds
    TypingDialog->>UI: Update window preferred size
    TypingDialog->>UI: Reposition & center window
    UI-->>User: Display resized text & window
Loading

Estimated Code Review Effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly Related PRs

  • PR #8: Directly related — modifies the same files (TypingService, TypingDialog) with identical fontSize persistence APIs and Ctrl+wheel zoom functionality, suggesting this may be a continuation or related implementation effort.

Poem

🐰 Ctrl and wheel, a zoom so grand,
Text grows large across the land,
Window dances, bounds respected,
Cursor keeps itself protected,
Persistence blooms, sessions bright,
Fonts remember day and night!

🚥 Pre-merge checks | ✅ 1 | ❌ 5

❌ Failed checks (4 warnings, 1 inconclusive)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR adds font size persistence (TypingService) and zoom support, but the linked issue #9 about statistics persistence shows no matching implementation in the code changes. The code changes address #5 and #6 (font resizing/zoom) but lack implementation for #9 (statistics persistence). Either add statistics persistence or remove #9 from the PR objectives.
Out of Scope Changes check ⚠️ Warning Changes include font resizing implementation, .gitignore updates, and AGENTS.md workflow documentation that align with PR objectives but extend beyond the stated linked issue #9. AGENTS.md and plans/issue/issue-6-plan.md appear to be workflow documentation unrelated to the release. Consider removing these from the merge or clarifying their necessity for the release.
Docstring Coverage ⚠️ Warning Docstring coverage is 7.14% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (5 files):

⚔️ .gitignore (content)
⚔️ build.gradle.kts (content)
⚔️ src/main/kotlin/com/github/kiolk/typingplugin/service/TypingService.kt (content)
⚔️ src/main/kotlin/com/github/kiolk/typingplugin/ui/TypingDialog.kt (content)
⚔️ src/main/resources/META-INF/plugin.xml (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
Title check ❓ Inconclusive The title describes a release merge operation but doesn't reflect the actual scope of changes including font resizing, statistics persistence, and auto-scrolling improvements. Consider a more descriptive title like 'v1.0.8: Add font resize/zoom, fix statistics persistence, and improve window auto-resizing' to clarify the actual technical changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Kiolk
Kiolk merged commit 3cd3a52 into main Feb 16, 2026
3 checks passed
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.

Statistic doesn't store between sessions

1 participant