rgolangh / gnome-shell-extension-containers

containers is a gnome-shell extension to manage linux container, run by podman
Apache License 2.0
45 stars 11 forks source link

[Feature Request] There are options for Change targeted terminal #57

Open benyaminl opened 4 days ago

benyaminl commented 4 days ago

Hello, I just want to ask, is it possible to change the

https://github.com/rgolangh/gnome-shell-extension-containers/blob/0d76598003c269e1f7ff4fae387c351e59611ab2/src/modules/podman.js#L266-L268

With other terminal command?

I'm on gnome but I'm not using gnome-terminal (sorry, haha...)

Or I need to change the code myself for that?

Thank you

EDIT:

For now I do have workaround by making gnome-terminal shell script to get the command using xfce4-terminal /usr/local/bin/gnome-terminal

#!/bin/bash
if [ $1 == '--' ]; then
  shift
  xfce4-terminal -x $@
else
  xfce4-terminal $@
fi  
rgolangh commented 4 days ago

Thanks for reporting this. :+1: +1 for clever script , but I would hate to force you do that. I think it should be easy to put that in a setting, and fallback to gnome terminal otherwise.

rgolangh commented 2 days ago

@benyaminl can you please checkout the branch from #58 and configure your terminal? I don't have an installation with xfce4 at the moment. I did test few other terminals, some run as flatpaks.

benyaminl commented 1 day ago

@benyaminl can you please checkout the branch from #58 and configure your terminal? I don't have an installation with xfce4 at the moment. I did test few other terminals, some run as flatpaks.

Yeah, thanks man it works

image

Just the process of running from the alpha/test version is quite strange I guess, I need to logout and relogin to make the new version works.

the command line for xfce terminal is

xfce4-terminal -x , ~with extra space in the end~. like gnome-terminal --. ~I think this need to be conveyed to the user? *or based on the extension, you can assume, most user is developer so.. they know it. haha..~

image

EDIT: without extra space, terminal works

rgolangh commented 1 day ago

what do you see in the journalctl and what happens if you don't add the space? I tested here with a script, similar to what you did, and I don't see the space is needed.

benyaminl commented 9 hours ago

@rgolangh sorry, seems it works as it's without space, I just retest it.