After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 602951 - share bar finetuning
share bar finetuning
Status: RESOLVED OBSOLETE
Product: gnome-user-share
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-user-share maintainers
gnome-user-share maintainers
Depends on:
Blocks:
 
 
Reported: 2009-11-25 16:07 UTC by Matthias Clasen
Modified: 2019-03-20 10:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gnome-user-share-share-bar-check.patch (8.30 KB, patch)
2010-04-03 18:41 UTC, Michael Cronenworth
rejected Details | Review
gnome-user-share-share-extension.patch (1.63 KB, patch)
2010-04-03 18:41 UTC, Michael Cronenworth
rejected Details | Review

Description Matthias Clasen 2009-11-25 16:07:12 UTC
Previously reported here: 

- Currently the share bar doesn't differentiate between enabled and disabled.
Imo, the messages should be slightly different. If sharing is disabled, the
message should be as it is currently:

"You can share files from this folder..."


But if sharing is enabled, the message should be something like:

"Files in this folder are shared..."


And you had some concerns:

We can't fix the second part. It's easy if you have 3 different folders for
ObexPush, ObexFTP and DAVS sharing. But it gets more complicated when you need
to tell the user that:
You can share files from this folder using HTTP, this folder is already shared
using ObexFTP over Bluetooth.

And what do we show when the public directory is the same as the downloads
directory? Ideas welcome, in a separate bug...



Here are some ideas:

- I don't think details about the individual transports need to be shown in the share bar message. 

- If sharing is disabled, you want to tell the user that it can be turned on, mostly:

You can receive files from other users in this folder  [ Launch Preferences ]

You can share files from this folder with other users  [ Launch Preferences ]

seem both fine to me.


- If sharing is enabled, we need to tell users that the files they see in Public can be seen by others.

Files received from other users will be placed here  [ Launch Preferences ]
Files in this folder are visible to others           [ Launch Preferences ]
Comment 1 Matthias Clasen 2009-11-25 16:07:45 UTC
I meant to say

Previously reported here: https://bugzilla.gnome.org/show_bug.cgi?id=600499
Comment 2 Matthias Clasen 2009-11-25 20:57:40 UTC
Also worth pointing out that the annoyance factor of the message is probably to a large part due to the bar and thus the button being slightly too large, due to two-line text. Another small annoyance: the first line is regular, the second line italic. 

Removing the 'Personal File Sharing' in the first line would solve both of these.
Comment 3 mads 2009-12-04 22:45:42 UTC
I think the current amount of text and the annoyance is acceptable as a notification, but that requires that there is a simple way to turn it off.

I would assume that this file sharing preferences menu is available somewhere in the menu too, but I can't find it? "Launch Preferences" in this context hints that it is Edit/Preferences. "Launch Personal File Sharing Preferences" would be more descriptive ... but probably too verbose.
Comment 4 Michael Cronenworth 2010-04-03 18:41:12 UTC
Created attachment 157833 [details] [review]
gnome-user-share-share-bar-check.patch

This is a patch against Gnome 2.28 in Fedora 12 from the Red Hat bugzilla report: https://bugzilla.redhat.com/show_bug.cgi?id=539586

It adds a preference toggle to the Gnome User Share settings window to turn on/off the share bar text.

The first patch creates a checkbox in the preferences and a boolean in the
schema.
Comment 5 Michael Cronenworth 2010-04-03 18:41:47 UTC
Created attachment 157834 [details] [review]
gnome-user-share-share-extension.patch

The second patch adds the boolean check to the share-extension.c module.

The other folder bars (Public, etc.) could be added using my patches as a
template.
Comment 6 Bastien Nocera 2010-04-14 16:12:44 UTC
I don't agree at all with the solution of adding a configuration option.
Comment 7 Bastien Nocera 2010-04-14 16:47:32 UTC
See also bug 613584 for the specific case of always showing the bar in the Downloads folder.
Comment 8 Michael Cronenworth 2010-05-18 20:27:15 UTC
(In reply to comment #6)
> I don't agree at all with the solution of adding a configuration option.

The bar takes up valuable folder display space. Why must I be *constantly* reminded that the Downloads folder is where Downloads go? Redundant crap if I've ever seen it. How did this feature get added by some random person's suggestion, yet to add a configurable option (user toggled or otherwise) is dismissed immediately? What will be appropriate to satisfy everyone? I'm ready and willing to write a patch. I just missed getting on the CC list on this bug somehow.
Comment 9 GNOME Infrastructure Team 2019-03-20 10:39:01 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/gnome-user-share/issues/17.