Skip to content

Commit

Permalink
Do not default to xterm
Browse files Browse the repository at this point in the history
It makes more sense to default to the terminals people are less likely
to have. If a user has `xterm` installed, then that is, most probably,
because it ships with X11. Whereas if a user has `st` or `alacritty`
instlled, then that is because he/she chose to install it.
  • Loading branch information
kovmir committed Sep 20, 2024
1 parent 07a972a commit 28c8504
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions plugins/preview-tabbed
Original file line number Diff line number Diff line change
Expand Up @@ -57,14 +57,14 @@ if [ -n "$WAYLAND_DISPLAY" ] ; then
exit 1
fi

if type xterm >/dev/null 2>&1 ; then
TERMINAL="xterm -into"
elif type urxvt >/dev/null 2>&1 ; then
TERMINAL="urxvt -embed"
if type alacritty >/dev/null 2>&1 ; then
TERMINAL="alacritty --embed"
elif type st >/dev/null 2>&1 ; then
TERMINAL="st -w"
elif type alacritty >/dev/null 2>&1 ; then
TERMINAL="alacritty --embed"
elif type urxvt >/dev/null 2>&1 ; then
TERMINAL="urxvt -embed"
elif type xterm >/dev/null 2>&1 ; then
TERMINAL="xterm -into"
else
echo "No xembed term found" >&2
fi
Expand Down

0 comments on commit 28c8504

Please sign in to comment.