Skip to content

fix(windows): align title bar controls for RTL#21

Merged
alaa-alsalehi merged 1 commit into
ruknsoftware:masterfrom
alaa-alsalehi:fix-windows-mirror-issue
May 12, 2026
Merged

fix(windows): align title bar controls for RTL#21
alaa-alsalehi merged 1 commit into
ruknsoftware:masterfrom
alaa-alsalehi:fix-windows-mirror-issue

Conversation

@alaa-alsalehi

@alaa-alsalehi alaa-alsalehi commented May 12, 2026

Copy link
Copy Markdown

https://tasko.rukn.sh/tasko/task/TASK-2026-00490
Position the caption button strip with logical inset (end-0) so it sits on the opposite edge from the flex-start logo when the app uses RTL. Set dir=auto on the centered title line for correct mixed Arabic text. Bump package version to 0.49.1.

Summary by CodeRabbit

  • Style

    • Updated Windows title bar layout and text rendering properties for improved display consistency.
  • Chores

    • Version bumped to 0.49.1.

Review Change Stack

Position the caption button strip with logical inset (end-0) so it
sits on the opposite edge from the flex-start logo when the app
uses RTL. Set dir=auto on the centered title line for correct
mixed Arabic text. Bump package version to 0.49.1.
@coderabbitai

coderabbitai Bot commented May 12, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This patch release updates the package version to 0.49.1 and refactors the Windows title bar layout to add directional text support and adjust window control positioning using updated Tailwind classes.

Changes

Patch Release 0.49.1

Layer / File(s) Summary
Version bump and title bar layout polish
package.json, src/components/WindowsTitleBar.vue
Package version incremented from 0.49.0 to 0.49.1. Title bar center text block gains dir="auto" for directional text handling, and controls wrapper repositions using absolute inset-y-0 end-0 Tailwind classes instead of prior right-0/h-full styling.

🎯 1 (Trivial) | ⏱️ ~3 minutes

🐰 A patch hops through the code so spry,
Version bumps up—0.49.1, we fly!
Title bars shimmer with auto-direction,
Layout refined to pixel perfection! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix(windows): align title bar controls for RTL' directly matches the main change in the PR: using logical CSS properties (end-0) and dir="auto" for RTL layout support.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
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

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.

@alaa-alsalehi alaa-alsalehi merged commit be9f6b9 into ruknsoftware:master May 12, 2026
1 of 5 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.

1 participant