refactor: navegacion consistente con HandleBack en todas las vistas
Cada vista expone HandleBack() bool que retrocede un nivel interno (logs -> lista) o indica que esta en estado base. El model principal intercepta q/0/esc y delega a HandleBack: si retorna true, sale de la TUI; si false, la vista retrocedio internamente. Elimina manejo duplicado de esc/q/0 en sub-estados.
This commit is contained in:
@@ -53,6 +53,12 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
|
|||||||
switch msg.String() {
|
switch msg.String() {
|
||||||
case views.KeyQuit:
|
case views.KeyQuit:
|
||||||
return m, tea.Quit
|
return m, tea.Quit
|
||||||
|
case "q", "0", "esc":
|
||||||
|
updated, atBase := m.handleBack()
|
||||||
|
if atBase {
|
||||||
|
return updated, tea.Quit
|
||||||
|
}
|
||||||
|
return updated, nil
|
||||||
case views.KeyTab:
|
case views.KeyTab:
|
||||||
m.activeTab = (m.activeTab + 1) % len(tabNames)
|
m.activeTab = (m.activeTab + 1) % len(tabNames)
|
||||||
return m, m.initActiveView()
|
return m, m.initActiveView()
|
||||||
@@ -129,6 +135,29 @@ func (m Model) renderTabs() string {
|
|||||||
return m.Styles.Header.Render("Docker TUI") + " " + row
|
return m.Styles.Header.Render("Docker TUI") + " " + row
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// handleBack asks the active view to go back one level.
|
||||||
|
// Returns the updated model and true if the view was already at base level (app should quit).
|
||||||
|
func (m Model) handleBack() (Model, bool) {
|
||||||
|
switch View(m.activeTab) {
|
||||||
|
case ViewContainers:
|
||||||
|
atBase := m.containers.HandleBack()
|
||||||
|
return m, atBase
|
||||||
|
case ViewImages:
|
||||||
|
atBase := m.images.HandleBack()
|
||||||
|
return m, atBase
|
||||||
|
case ViewVolumes:
|
||||||
|
atBase := m.volumes.HandleBack()
|
||||||
|
return m, atBase
|
||||||
|
case ViewNetworks:
|
||||||
|
atBase := m.networks.HandleBack()
|
||||||
|
return m, atBase
|
||||||
|
case ViewCompose:
|
||||||
|
atBase := m.compose.HandleBack()
|
||||||
|
return m, atBase
|
||||||
|
}
|
||||||
|
return m, true
|
||||||
|
}
|
||||||
|
|
||||||
func (m Model) initActiveView() tea.Cmd {
|
func (m Model) initActiveView() tea.Cmd {
|
||||||
switch View(m.activeTab) {
|
switch View(m.activeTab) {
|
||||||
case ViewContainers:
|
case ViewContainers:
|
||||||
|
|||||||
@@ -134,9 +134,6 @@ func (m ComposeModel) Update(msg tea.Msg) (ComposeModel, tea.Cmd) {
|
|||||||
if m.scrollOff > 0 {
|
if m.scrollOff > 0 {
|
||||||
m.scrollOff--
|
m.scrollOff--
|
||||||
}
|
}
|
||||||
case "esc", "q", "0":
|
|
||||||
m.state = composeList
|
|
||||||
return m, nil
|
|
||||||
}
|
}
|
||||||
return m, nil
|
return m, nil
|
||||||
}
|
}
|
||||||
@@ -156,6 +153,17 @@ func (m ComposeModel) Update(msg tea.Msg) (ComposeModel, tea.Cmd) {
|
|||||||
return m, cmd
|
return m, cmd
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// HandleBack retrocede un nivel. Retorna true si ya estaba en estado base.
|
||||||
|
func (m *ComposeModel) HandleBack() bool {
|
||||||
|
switch m.state {
|
||||||
|
case composeLogs:
|
||||||
|
m.state = composeList
|
||||||
|
return false
|
||||||
|
default:
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func (m ComposeModel) View() string {
|
func (m ComposeModel) View() string {
|
||||||
switch m.state {
|
switch m.state {
|
||||||
case composeLoading, composeAction:
|
case composeLoading, composeAction:
|
||||||
|
|||||||
@@ -132,9 +132,6 @@ func (m ContainersModel) Update(msg tea.Msg) (ContainersModel, tea.Cmd) {
|
|||||||
if m.scrollOff > 0 {
|
if m.scrollOff > 0 {
|
||||||
m.scrollOff--
|
m.scrollOff--
|
||||||
}
|
}
|
||||||
case "esc", "q", "0":
|
|
||||||
m.state = containersList
|
|
||||||
return m, nil
|
|
||||||
}
|
}
|
||||||
return m, nil
|
return m, nil
|
||||||
}
|
}
|
||||||
@@ -155,6 +152,17 @@ func (m ContainersModel) Update(msg tea.Msg) (ContainersModel, tea.Cmd) {
|
|||||||
return m, cmd
|
return m, cmd
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// HandleBack retrocede un nivel. Retorna true si ya estaba en estado base (el caller debe salir).
|
||||||
|
func (m *ContainersModel) HandleBack() bool {
|
||||||
|
switch m.state {
|
||||||
|
case containersLogs:
|
||||||
|
m.state = containersList
|
||||||
|
return false
|
||||||
|
default:
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func (m ContainersModel) View() string {
|
func (m ContainersModel) View() string {
|
||||||
switch m.state {
|
switch m.state {
|
||||||
case containersLoading:
|
case containersLoading:
|
||||||
|
|||||||
@@ -108,6 +108,11 @@ func (m ImagesModel) Update(msg tea.Msg) (ImagesModel, tea.Cmd) {
|
|||||||
return m, cmd
|
return m, cmd
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// HandleBack retrocede un nivel. Retorna true si ya estaba en estado base.
|
||||||
|
func (m *ImagesModel) HandleBack() bool {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
|
||||||
func (m ImagesModel) View() string {
|
func (m ImagesModel) View() string {
|
||||||
switch m.state {
|
switch m.state {
|
||||||
case imagesLoading, imagesAction:
|
case imagesLoading, imagesAction:
|
||||||
|
|||||||
@@ -104,6 +104,11 @@ func (m NetworksModel) Update(msg tea.Msg) (NetworksModel, tea.Cmd) {
|
|||||||
return m, cmd
|
return m, cmd
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// HandleBack retrocede un nivel. Retorna true si ya estaba en estado base.
|
||||||
|
func (m *NetworksModel) HandleBack() bool {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
|
||||||
func (m NetworksModel) View() string {
|
func (m NetworksModel) View() string {
|
||||||
switch m.state {
|
switch m.state {
|
||||||
case networksLoading, networksAction:
|
case networksLoading, networksAction:
|
||||||
|
|||||||
@@ -104,6 +104,11 @@ func (m VolumesModel) Update(msg tea.Msg) (VolumesModel, tea.Cmd) {
|
|||||||
return m, cmd
|
return m, cmd
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// HandleBack retrocede un nivel. Retorna true si ya estaba en estado base.
|
||||||
|
func (m *VolumesModel) HandleBack() bool {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
|
||||||
func (m VolumesModel) View() string {
|
func (m VolumesModel) View() string {
|
||||||
switch m.state {
|
switch m.state {
|
||||||
case volumesLoading, volumesAction:
|
case volumesLoading, volumesAction:
|
||||||
|
|||||||
Reference in New Issue
Block a user