Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change "Show info" in Removable Media plugin of lxqt-panel? #1823

Open
glu8716 opened this issue Oct 12, 2022 · 16 comments
Open

Change "Show info" in Removable Media plugin of lxqt-panel? #1823

glu8716 opened this issue Oct 12, 2022 · 16 comments

Comments

@glu8716
Copy link

glu8716 commented Oct 12, 2022

Expected Behavior

When I eject a removable device through the "Removable Media" panel widget or through PCManFM I should get a notification saying it is now safe to eject it.

Current Behavior

I don't get any notification.

Possible Solution
Steps to Reproduce (for bugs)
  1. Plug a removable drive, such as a USB stick
  2. Mount it
  3. Eject it through PCManFM or through the "Removable Media" panel widget
Context

No particular context. I just remember I was receiving notifications some weeks ago. Not they suddenly doesn't show up anymore.

System Information
  • Distribution & Version: Artix
  • Kernel: 5.19.13-zen
  • Qt Version: 5.15.6
  • liblxqt Version: 1.1.0
  • Package version: 3
@tsujan
Copy link
Member

tsujan commented Oct 12, 2022

Not reproducible with the latest git.

@tsujan tsujan closed this as completed Oct 12, 2022
@stefonarch
Copy link
Member

stefonarch commented Oct 12, 2022

On my side it's several months I don't see any notification anymore, with latest git on my laptop. Have to check the debian PC with LXQt 1.1 where I think I still see those notification about successful removed devices.

I never saw any settings about omitting this notification.

@stefonarch stefonarch reopened this Oct 12, 2022
@stefonarch
Copy link
Member

stefonarch commented Oct 13, 2022

Checked on debian LXQt 1.1.0 and also there I see no notification anymore.

@glu8716
Copy link
Author

glu8716 commented Oct 13, 2022

I looked everywhere and apparently the culprit is an option in the "Removable Media" widget configuration. There's a "When a device is connected" > "Show info" option there. Looks like that notifications are sent only if that option is set to "Show info".
Since there's also a "When a eject button is pressed" option, which doesn't have a "Show info" option, I haven't thought the problem could be there. I think this is a little bit unintuitive and should be redesigned if it's the root of the "problem".
I also think the possibility to see notifications about removable devices shouldn't be tied to a panel widget, but should be a global option.

@stefonarch
Copy link
Member

stefonarch commented Oct 13, 2022

Yep, confirmed, I played around with all combinations of options in pcmanfm-qt and some in the panel plugin but that was the only one which I didn't check.

Let's here others, but maybe only the text needs to be changed to "Notify about connect/remove" or similar, the actual text makes think about an popup on the panel. Here it shows menu with actions also when "show info" is set.

@tsujan
Copy link
Member

tsujan commented Oct 13, 2022

I don't get what the problem is. There are three options, whose texts seem self-explanatory: Popup menu, Show info, Do nothing. It has always been so, while the report pretends that something has changed (hence closing).

@stefonarch
Copy link
Member

stefonarch commented Oct 13, 2022

I happened to have the same experience as @glu8716: notifications stopped working, I didn't bother about.

IMHO "Show info" is not clear about notifications, it could refer to a panel popup.
"Show notifications and popup-menu" is what is does.

@tsujan
Copy link
Member

tsujan commented Oct 13, 2022

notifications stopped working

That's what I don't understand. If notifications stopped working, something should have been changed, while you say that the option in question works.

@glu8716
Copy link
Author

glu8716 commented Oct 13, 2022

The option in question works indeed. But we thought it wasn't working because it isn't clear (at least to me, since I'm relatively new to LXQt) what makes the notification appears or not. The reason I opened this issue is that I didn't know devices notifications were tied to the panel's widget, which I had removed at a later time. And if I read "Show info when a device is connected" I assume it's some kind of info that appears when a device is connected, and not disconnected.

The fact that it has always been so doesn't mean it can't be better. Maybe something like a checkbox with "Show devices notifications" or something like that could be clearer. And I think it should be something that doesn't depend on a panel widget. What if, for example, I want notifications but don't use the LXQt panel?

@tsujan
Copy link
Member

tsujan commented Oct 13, 2022

Quoiting: "I just remember I was receiving notifications some weeks ago...."

@glu8716
Copy link
Author

glu8716 commented Oct 13, 2022

The reason I opened this issue is that I didn't know devices notifications were tied to the panel's widget, which I had removed at a later time.

Initially I had the widget on the panel with that setting I unconsciously set to "show info", therefore I was receiving notifications. At that time I didn't think notifications were tied to the widget, I thought showing notifications was a normal behavior for LXQt.
When at a later time I removed the widget from the panel and haven't received notifications anymore I opened this issue. Only after opening it I realized the widget, if not present on the panel, is the reason for not having notifications.

So, yeah, this is not really a "issue" anymore, but rather a suggestion now, as I think this setting could be clearer :)

@tsujan
Copy link
Member

tsujan commented Oct 13, 2022

OK.

I have no problem with changing of "Show info", although it seems clear enough to me (and to many other users)..

Showing notifications for removable media needs a program running in the background. A panel plugin called "Removable Media" seems like an obvious choice.

I change the title because it was very misleading (I was going to check the changelog of solid when I first saw it).

@tsujan tsujan changed the title No notifications when I safely eject USB drives Change "Show info" in Removable Media plugin of lxqt-panel? Oct 13, 2022
@stefonarch
Copy link
Member

For me notifications stopped when I changed months ago this setting by case.
It looks also exclusive: Show info >< Popup menu while the popup menu is shown also with "Show info".

I see 2 options to improve clearness:

  • Change text to something "Popup menu and notifications"
  • Replace this option with a checkbox "Show device notifications"

@glu8716
Copy link
Author

glu8716 commented Oct 13, 2022

Yeah. I'm sorry for the misunderstanding.

To be honest to me the problem isn't really the "Show info" text (although I admit that changing it to "notifications" seems to be more straightforward), but the fact that the notifications when ejecting are set by a field called "When a device is connected" instead of the "When eject button is pressed" one. In this case my suggestion would be to add the same "Show info" (or whatever) to the "When eject button is pressed" field too for more consistency. Or making a global "Show notifications" checkbox.

@stefonarch
Copy link
Member

That's a good point. So a checkbox "Notify about devices"?

On a side note I just saw that whith a drive containing 2 partitions (one is encrypted) only the "connected" notifications do appear, while no info at all on unmount is shown. Probably related to #1639 , I have no drive with 2 normal partitions to test atm.

@tsujan
Copy link
Member

tsujan commented Oct 15, 2022

This report might not be seen here. Transferring it to lxqt-panel's issue tracker....

@tsujan tsujan transferred this issue from lxqt/lxqt Oct 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants