Revert "Fix for '(null)' display on AirPod source selection"

Merge pull request #1845 from djordje/fix-omarchy-tui-install
Fix omarchy-tui-install
2026-02-17 15:25:37 +00:00 · 2025-09-28 15:18:25 -04:00 · 2025-09-28 15:01:17 -04:00 · 2025-09-28 14:56:47 -04:00 · 2025-09-28 14:55:55 -04:00 · 2025-09-25 19:37:18 -05:00
3 changed files with 21 additions and 3 deletions
--- a/bin/omarchy-cmd-screenrecord
+++ b/bin/omarchy-cmd-screenrecord
@@ -15,7 +15,7 @@ SCOPE="$1"
 AUDIO=$([[ $2 == "audio" ]] && echo "--audio")

 start_screenrecording() {
-  filename="$OUTPUT_DIR/screenrecording-$(date +'%Y-%m-%d_%H-%M-%S').mp4"
+  local filename="$OUTPUT_DIR/screenrecording-$(date +'%Y-%m-%d_%H-%M-%S').mp4"

  if lspci | grep -qi 'nvidia'; then
    wf-recorder $AUDIO -f "$filename" -c libx264 -p crf=23 -p preset=medium -p movflags=+faststart "$@" &
@@ -47,7 +47,8 @@ screenrecording_active() {
 if screenrecording_active; then
  stop_screenrecording
 elif [[ "$SCOPE" == "output" ]]; then
-  start_screenrecording
+  output=$(slurp -o) || exit 1
+  start_screenrecording -g "$output"
 else
  region=$(slurp) || exit 1
  start_screenrecording -g "$region"
--- a/bin/omarchy-tui-install
+++ b/bin/omarchy-tui-install
@@ -43,7 +43,7 @@ cat >"$DESKTOP_FILE" <<EOF
 Version=1.0
 Name=$APP_NAME
 Comment=$APP_NAME
-Exec=$TERMINAL --class $APP_CLASS -e $APP_EXEC
+Exec=\$TERMINAL --class=$APP_CLASS -e $APP_EXEC
 Terminal=false
 Type=Application
 Icon=$ICON_PATH
--- a/migrations/1758436991.sh
+++ b/migrations/1758436991.sh
@@ -0,0 +1,17 @@
+echo "Fix Disk Usage and Docker TUIs"
+
+APP_DIR="$HOME/.local/share/applications"
+ICON_DIR="$APP_DIR/icons"
+
+# Don't use omarchy-tui-remove to preserve icons
+
+if [[ -f "$APP_DIR/Docker.desktop" ]]; then
+  rm "$APP_DIR/Docker.desktop"
+  omarchy-tui-install "Docker" "lazydocker" tile "$ICON_DIR/Docker.png"
+fi
+
+if [[ -f "$APP_DIR/Disk Usage.desktop" ]]; then
+  rm "$APP_DIR/Disk Usage.desktop" 
+  omarchy-tui-install "Disk Usage" "bash -c 'dust -r; read -n 1 -s'" float "$ICON_DIR/Disk Usage.png"
+fi
+
Author	SHA1	Message	Date
Ryan Hughes	796ef67ede	Revert "Fix for '(null)' display on AirPod source selection"	2025-09-28 15:18:25 -04:00
Ryan Hughes	076da663f5	Merge pull request #1845 from djordje/fix-omarchy-tui-install Fix omarchy-tui-install	2025-09-28 15:01:17 -04:00
Ryan Hughes	b92ebff29b	Merge pull request #1859 from shawnyeager/audio-switch-fix Fix for '(null)' display on AirPod source selection	2025-09-28 14:56:47 -04:00
Ryan Hughes	8e5b59995a	Merge pull request #1920 from iamobservable/iamobservable/bug/screen-recording-no-longer-works-after-updating Use output display or region geometry to record	2025-09-28 14:55:55 -04:00
observable	d4b0ba5d94	Use output display or region geometry to record The previous code did not identify the output display correctly, leading to wf-recorder asking the user to select the region when the user requested the display output to be selected. With this change the bash script will use slurp with the -o flag to identify the display output. This has not been tested with wl-screenrec as no change has been made to its call signature in omarchy-cmd-screenrecord and the wl-screenrec binary is not included in the default omarchy installation.	2025-09-25 19:37:18 -05:00
Djordje Kovacevic	4d9f932e6c	Replace "--class" with "--class="	2025-09-23 22:52:58 +02:00
Shawn Yeager	38d536ba9d	Fix for '(null)' display on AirPod source selection	2025-09-21 16:18:12 -05:00
Djordje Kovacevic	441ee10d70	Add migration to fix Docker and Disk Usage TUIs	2025-09-21 09:29:44 +02:00
Djordje Kovacevic	45465d3e7c	Fix TUI install script - $TERMINAL should remain variable	2025-09-21 08:40:48 +02:00