Skip to content

fix(ui): restore Windows capsule hitbox metrics#541

Merged
H-Chris233 merged 1 commit into
Open-Less:betafrom
H-Chris233:fix/windows-capsule-hitbox
May 28, 2026
Merged

fix(ui): restore Windows capsule hitbox metrics#541
H-Chris233 merged 1 commit into
Open-Less:betafrom
H-Chris233:fix/windows-capsule-hitbox

Conversation

@H-Chris233
Copy link
Copy Markdown
Collaborator

@H-Chris233 H-Chris233 commented May 28, 2026

User description

Summary

  • restore Windows capsule pill metrics to 196x52 with text width 104
  • keep frontend capsule layout aligned with the native hitbox contract in src-tauri/src/lib.rs

Test Plan

  • node "openless-all/app/src/lib/capsuleLayout.test.ts"
  • cargo test --manifest-path "openless-all/app/src-tauri/Cargo.toml" capsule
  • npm --prefix "openless-all/app" run build
  • git diff --check

PR Type

Bug fix


Description

  • Restore Windows capsule pill metrics

  • Match native hitbox contract

  • Update text width and outer dimensions


Diagram Walkthrough

flowchart LR
  A["Windows capsule layout"] -- "uses" --> B["Native hitbox contract"]
  B -- "requires" --> C["Updated pill metrics"]
  C -- "sets" --> D["196x52 size, 104 text width"]
Loading

File Walkthrough

Relevant files
Bug fix
capsuleLayout.ts
Restore Windows capsule sizing metrics                                     

openless-all/app/src/lib/capsuleLayout.ts

  • Updates Windows pill metrics to 196x52
  • Increases textWidth to 104
  • Keeps boxSizing as border-box
  • Aligns frontend layout with native contract
+2/-3     

@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@H-Chris233 H-Chris233 merged commit 5acb2df into Open-Less:beta May 28, 2026
4 checks passed
@H-Chris233 H-Chris233 deleted the fix/windows-capsule-hitbox branch May 29, 2026 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant