jedarden/FABRIC
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: add Escape key to close WorkerDetail modal

Browse source 
Previously Escape only handled view mode changes. Now it first checks
if the WorkerDetail modal is visible and closes it before returning
to the default view.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
jeda 2026-03-05 04:36:29 +00:00
parent a1856400df
commit b7ca7ab818
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/tui/app.ts
View file

@ -448,8 +448,14 @@ export class FabricTuiApp {
this.toggleAnalyticsView();
});
// Escape to return to default view
// Escape to close modals or return to default view
this.screen.key(['escape'], () => {
// First, hide worker detail if visible
if (this.workerDetail.isVisible()) {
this.workerDetail.hide();
return;
}
// Then, return to default view if in another view
if (this.viewMode !== 'default') {
this.setViewMode('default');
}