Skip to content

Conversation

ym
Copy link
Contributor

@ym ym commented May 22, 2025

No description provided.

@ym ym requested review from adamshiervani and Copilot May 22, 2025 22:56
@ym ym marked this pull request as ready for review May 22, 2025 22:57
Copy link
Contributor

@Copilot 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

This PR adds functionality to allow users to disable keyboard LED synchronization. It introduces new state and methods to manage LED sync availability, updates various UI components to reflect and control LED sync preferences, and integrates these changes into key application areas such as device routes, settings, and status displays.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
ui/src/routes/devices.$id.tsx Integrates LED sync availability setting on keyboard state retrieval
ui/src/routes/devices.$id.settings.keyboard.tsx Adds a new settings item for controlling LED state synchronization
ui/src/hooks/stores.ts Introduces new state and actions for LED sync and LED state management
ui/src/components/WebRTCVideo.tsx Incorporates LED sync state in RTC-related event handling
ui/src/components/VirtualKeyboard.tsx Adjusts virtual keyboard behavior based on LED sync status
ui/src/components/InfoBar.tsx Displays the LED sync status to users

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ym ym merged commit c1d771c into jetkvm:dev May 22, 2025
2 checks passed
ym added a commit to ym/jetkvm-kvm that referenced this pull request Sep 26, 2025
* feat: allow user to disable keyboard LED synchronization

* Update ui/src/hooks/stores.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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