Nicer message for no network.

Author: ole108

cobracmd/root.go

@@ -1,7 +1,6 @@
 package cobracmd
 
 import (
-	"log"
 	"os"
 
 	cmd "github.com/can3p/kleiner/shared/cmd/cobra"
@@ -30,10 +29,6 @@ func init() {
 	info := buildinfo.Info()
 	cmd.Setup(info, rootCmd)
 
-	defer func() {
-		if r := recover(); r != nil {
-			log.Printf("WARNING: %v", r)
-		}
-	}()
+	defer recover() // error isn't valuable
 	published.MaybeNotifyAboutNewVersion(info)
 }

info.png

@@ -39,6 +39,28 @@ func createPasswordEntry(attrs ui.AttributesDescr, values map[string]any, output
 	return entry
 }
 
+func createSelect(attrs ui.AttributesDescr, values map[string]any, outputKey, _ string) fyne.CanvasObject {
+	sel := widget.NewSelect(ui.AnysToStrings(attrs["options"]), nil)
+	if ph, ok := attrs["placeHolder"].(string); ok {
+		sel.PlaceHolder = ph
+	}
+	if initial, ok := attrs["initiallySelected"].(string); ok {
+		sel.SetSelected(initial)
+	}
+	values[outputKey] = &sel.Selected
+	return sel
+}
+
+func createSelectEntry(attrs ui.AttributesDescr, values map[string]any, outputKey, fullName string) fyne.CanvasObject {
+	sel := widget.NewSelectEntry(ui.AnysToStrings(attrs["options"]))
+	if ph, _ := attrs["placeHolder"].(string); ph != "" {
+		sel.SetPlaceHolder(ph)
+	}
+	sel.Validator = StringValidator(attrs, fullName)
+	values[outputKey] = &sel.Text
+	return sel
+}
+
 func createCheckBox(attrs ui.AttributesDescr, values map[string]any, outputKey, _ string) fyne.CanvasObject {
 	subLabel, _ := attrs["subLabel"].(string)
 	box := widget.NewCheck(subLabel, nil)
@@ -58,16 +80,6 @@ func createCheckGroup(attrs ui.AttributesDescr, values map[string]any, outputKey
 	return cg
 }
 
-func createHyperlink(attrs ui.AttributesDescr, _ map[string]any, _, fullName string) fyne.CanvasObject {
-	text, _ := attrs["text"].(string)
-	surl, _ := attrs["url"].(string)
-	link, err := url.Parse(surl)
-	if err != nil {
-		log.Printf("ERROR: for %q: %v", fullName, err)
-	}
-	return widget.NewHyperlink(text, link)
-}
-
 func createRadioGroup(attrs ui.AttributesDescr, values map[string]any, outputKey, _ string) fyne.CanvasObject {
 	rg := widget.NewRadioGroup(ui.AnysToStrings(attrs["options"]), nil)
 	horizontal, _ := attrs["horizontal"].(bool)
@@ -81,6 +93,23 @@ func createRadioGroup(attrs ui.AttributesDescr, values map[string]any, outputKey
 	return rg
 }
 
+func createSlider(attrs ui.AttributesDescr, values map[string]any, outputKey, _ string) fyne.CanvasObject {
+	minv, _ := attrs["min"].(float64) // default min is 0
+	maxv, ok := attrs["max"].(float64)
+	if !ok {
+		maxv = 100.0 // default max is 100
+	}
+	slider := widget.NewSlider(minv, maxv)
+	if initial, ok := attrs["initialValue"].(float64); ok {
+		slider.SetValue(initial)
+	}
+	if step, ok := attrs["step"].(float64); ok { // default step is 1
+		slider.Step = step
+	}
+	values[outputKey] = &slider.Value
+	return slider
+}
+
 func createRichText(attrs ui.AttributesDescr, _ map[string]any, _, _ string) fyne.CanvasObject {
 	text, _ := attrs["text"].(string)
 	rt := widget.NewRichTextFromMarkdown(text)
@@ -99,45 +128,16 @@ func createRichText(attrs ui.AttributesDescr, _ map[string]any, _, _ string) fyn
 	return rt
 }
 
-func createSelect(attrs ui.AttributesDescr, values map[string]any, outputKey, _ string) fyne.CanvasObject {
-	sel := widget.NewSelect(ui.AnysToStrings(attrs["options"]), nil)
-	if ph, ok := attrs["placeHolder"].(string); ok {
-		sel.PlaceHolder = ph
-	}
-	if initial, ok := attrs["initiallySelected"].(string); ok {
-		sel.SetSelected(initial)
-	}
-	values[outputKey] = &sel.Selected
-	return sel
-}
-
-func createSelectEntry(attrs ui.AttributesDescr, values map[string]any, outputKey, fullName string) fyne.CanvasObject {
-	sel := widget.NewSelectEntry(ui.AnysToStrings(attrs["options"]))
-	if ph, _ := attrs["placeHolder"].(string); ph != "" {
-		sel.SetPlaceHolder(ph)
+func createHyperlink(attrs ui.AttributesDescr, _ map[string]any, _, fullName string) fyne.CanvasObject {
+	text, _ := attrs["text"].(string)
+	surl, _ := attrs["url"].(string)
+	link, err := url.Parse(surl)
+	if err != nil {
+		log.Printf("ERROR: for %q: %v", fullName, err)
 	}
-	sel.Validator = StringValidator(attrs, fullName)
-	values[outputKey] = &sel.Text
-	return sel
+	return widget.NewHyperlink(text, link)
 }
 
 func createSeparator(_ ui.AttributesDescr, _ map[string]any, _, _ string) fyne.CanvasObject {
 	return widget.NewSeparator()
 }
-
-func createSlider(attrs ui.AttributesDescr, values map[string]any, outputKey, _ string) fyne.CanvasObject {
-	minv, _ := attrs["min"].(float64) // default min is 0
-	maxv, ok := attrs["max"].(float64)
-	if !ok {
-		maxv = 100.0 // default max is 100
-	}
-	slider := widget.NewSlider(minv, maxv)
-	if initial, ok := attrs["initialValue"].(float64); ok {
-		slider.SetValue(initial)
-	}
-	if step, ok := attrs["step"].(float64); ok { // default step is 1
-		slider.Step = step
-	}
-	values[outputKey] = &slider.Value
-	return slider
-}