fix: actually add the category if we succeed

TypicalAM · Jul 4, 2023 · cf17742 · cf17742
1 parent dab7a88
commit cf17742
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/internal/model/browser/browser.go b/internal/model/browser/browser.go
@@ -74,9 +74,14 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		return m, nil
 
 	case popup.ChosenCategoryMsg:
-		m.message = fmt.Sprintf("Chosen category: %s", msg.Name)
 		m.popupShown = false
-		return m, nil
+		if err := m.config.Backend.Rss.AddCategory(msg.Name, ""); err != nil {
+			m.message = fmt.Sprintf("Error adding category: %s", err.Error())
+		} else {
+			m.message = fmt.Sprintf("Added category %s", msg.Name)
+		}
+
+		return m, m.config.Backend.FetchCategories()
 
 	case tab.NewTabMessage:
 		// Create the new tab

diff --git a/internal/popup/popup.go b/internal/popup/popup.go
@@ -97,6 +97,7 @@ func (p Popup) Update(msg tea.Msg) (Popup, tea.Cmd) {
 				cmds = append(cmds, p.textInput.Focus())
 			case newCategoryField:
 				p.focused = allField
+				p.textInput.Blur()
 			}
 
 		case "up", "k":
@@ -108,6 +109,7 @@ func (p Popup) Update(msg tea.Msg) (Popup, tea.Cmd) {
 				p.focused = allField
 			case newCategoryField:
 				p.focused = downloadedField
+				p.textInput.Blur()
 			}
 
 		case "enter":