Skip to content

feat(ui): add always-visible ATX LED status indicators#1099

Open
nitishagar wants to merge 1 commit intojetkvm:devfrom
nitishagar:fix-729
Open

feat(ui): add always-visible ATX LED status indicators#1099
nitishagar wants to merge 1 commit intojetkvm:devfrom
nitishagar:fix-729

Conversation

@nitishagar
Copy link
Contributor

  • Add ATXStateIndicator component to ActionBar
  • Show Power/HDD LED status when ATX extension is active
  • Add popover with Power/Reset buttons for quick access
  • Add confirmation dialogs for power actions to prevent accidents

Note: Long-press support for force power off can be added in a future iteration - the backend already supports "power-long" action.

Fixes #729

Copy link
Contributor

@IDisposable IDisposable left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The UI needs a localization pass

@nitishagar
Copy link
Contributor Author

Thanks for the thorough review @IDisposable! I've addressed all the comments. Please take a look when you get a chance.

@nitishagar
Copy link
Contributor Author

@IDisposable Thanks. Can you take another look?

@nitishagar
Copy link
Contributor Author

nitishagar commented Jan 22, 2026

@IDisposable Thanks for the review. Is this going to be merged automatically?

@IDisposable
Copy link
Contributor

IDisposable commented Jan 28, 2026

@IDisposable Thanks for the review. Is this going to be merged automatically?

@adamshiervani or @ym are actual code owners so they do the mainline merges.

After it's merged, we will need to do a localization pass for all the other languages (you can do so with cd ui && npm run i18n:machine-translate and include in this PR). More info here

- Add ATXStateIndicator component to ActionBar
- Show Power/HDD LED status when ATX extension is active
- Add popover with Power/Reset buttons for quick access
- Add confirmation dialogs for power actions to prevent accidents

Note: Long-press support for force power off can be added in a future
iteration - the backend already supports "power-long" action.

Fixes jetkvm#729
@nitishagar
Copy link
Contributor Author

@IDisposable Thanks. I've run the machine translation and added localizations for all supported languages (da, de, es, fr, it, nb, sv, zh).

@IDisposable
Copy link
Contributor

:shipit:

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.

Make ATX LEDs always visible

2 participants