Allow configuring default sort column for the processviewer widget via meta key
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 76/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- typescript
- Área
- frontend
Línea de trabajo
Comienza en frontend/app/view/processviewer/processviewer.tsx, especialmente con el tipo SortCol y la inicialización predeterminada del átomo en las líneas 32 y 113-114. Sigue cómo se leen los metadatos del bloque para el widget processviewer y, a continuación, verifica que las dos claves de metadatos solicitadas establezcan la ordenación inicial, mientras que el comportamiento manual de volver a ordenar mediante clic permanece sin cambios.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
The processviewer widget's default sort is hardcoded to CPU descending. There's no way to configure a different default — every fresh processviewer block opens sorted by CPU, and the user has to manually click another column header (e.g. Memory) every time. The sort preference doesn't persist across block recreations either, since it's stored only in a runtime atom rather than block meta.
For users who primarily care about a non-CPU column (memory hogs being the obvious case), CPU is a poor default to be locked into.
Current behavior
In frontend/app/view/processviewer/processviewer.tsx:113-114:
this.sortByAtom = jotai.atom<SortCol>("cpu");
this.sortDescAtom = jotai.atom<boolean>(true);
Hardcoded. The SortCol type at line 32 already enumerates all valid columns:
type SortCol = "pid" | "command" | "user" | "cpu" | "mem" | "status" | "threads";
On Windows the visible columns are PID / Command / CPU% / Memory (status, user, threads are hidden via hideOnPlatform), so the meaningful defaults a Windows user would want are one of those four. Other platforms get all seven.
Request
Please add meta keys to control the initial sort, e.g.:
processviewer:sortby— one of theSortColvalues ("pid" | "command" | "user" | "cpu" | "mem" | "status" | "threads"), default"cpu"processviewer:sortdesc— bool, defaulttrue
So a user can put this in their widgets.json to override the default processviewer widget to sort by memory:
"defwidget@processviewer": {
"display:order": -1,
"icon": "list-tree",
"label": "processes",
"blockdef": {
"meta": {
"view": "processviewer",
"processviewer:sortby": "mem",
"processviewer:sortdesc": true
}
}
}
Manual click-to-resort behavior would still work as today; the meta keys only control the initial state of the atoms.
- Lenguaje dominante
- Go
- Estrellas
- 22.3k
- Forks
- 1.1k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de wavetermdev/waveterm
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
wavetermdev/waveterm#3481 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
wavetermdev/waveterm#3435 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
wavetermdev/waveterm#3432 ·
-
enhancement triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
wavetermdev/waveterm#3431 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
wavetermdev/waveterm#3428 ·
Todos los issues de wavetermdev/waveterm
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
bug group: validation priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
codecheckers/chekhov#51 ·
-
Creating worktree from an existing remote branch with a slash in it, has unexpected behaviour Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100