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 306646 - digital camera option should be in a separate panel
digital camera option should be in a separate panel
Status: RESOLVED OBSOLETE
Product: gnome-volume-manager
Classification: Deprecated
Component: general
1.2.x
Other All
: Normal minor
: ---
Assigned To: Gnome volume manager maintainers
Gnome volume manager maintainers
Depends on:
Blocks:
 
 
Reported: 2005-06-06 12:13 UTC by Hubert Figuiere (:hub)
Modified: 2006-11-02 18:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to implement the change (41.48 KB, patch)
2005-06-08 01:13 UTC, Hubert Figuiere (:hub)
none Details | Review
reworked patch (45.19 KB, patch)
2005-06-08 17:57 UTC, Hubert Figuiere (:hub)
none Details | Review

Description Hubert Figuiere (:hub) 2005-06-06 12:13:09 UTC
digital camera option should be in a separate panel. It is confusing to have
that option in a control panel labelled "Removable Drives and Media"

There should be a control panel named "Digital Camera"

Other information:
Comment 1 Hubert Figuiere (:hub) 2005-06-08 01:13:55 UTC
Created attachment 47410 [details] [review]
patch to implement the change

Attached is a patch that implements a gnome-camera-properties CApplet, against
CVS.

Please let me know what you think. I have CVS access so I can commit it, but I
want to make sure it is OK before.
Comment 2 Robert Love 2005-06-08 15:11:26 UTC
Hubert, looks good to me.

Fejj, whatcha think, duder?
Comment 3 Jeffrey Stedfast 2005-06-08 15:22:41 UTC
well, the patch looks fine...

I'm just wondering if it really makes sense to split it into 2 programs. I do
agree with Hubert that users won't necessarily associate digital cameras with
"removable drives and media", but what if instead we did something more like the
following solution:

create another .desktop file (or whatever is needed to add another icon thingy
in the control panel) for Cameras and made that run `gnome-volume-properties
--page=camera` or some such type command-line option that would open the
properties dialog to a particular page?

could this work?

otherwise I see this turning into gnome-ipod-properties,
gnome-cellphone-properties, gnome-volume-properties,
gnome-cd-and-dvd-properties, gnome-whatsit-properties, etc

I think there's already discussion (here at Novell at least) about changing the
name of g-v-m to something more appropriate (at least that's the feeling I got
talking to JP and from that desktop summit a month or so back), so perhaps a new
name can also be chosen such that it would be more intuitive to suers (but even
if not, I think the command-line option would be cool?)

you guys have any thoughts on that solution?
Comment 4 Robert Love 2005-06-08 15:29:20 UTC
Ugh, I thought you were making it another tab.  Why not just rename the capplet
and/or the tab?  Worse case, make it another tab, but I'd prefer to not make
additional capplets.
Comment 5 Hubert Figuiere (:hub) 2005-06-08 15:33:33 UTC
I somewhat thought about doing it, one binary and either trick with ARGV[0] or
pass an argument.

I'll rework the patch to do that and post a new one.
Comment 6 Hubert Figuiere (:hub) 2005-06-08 17:57:48 UTC
Created attachment 47464 [details] [review]
reworked patch 

This new patch implement it with a --page= command line argument. It also keep
compatibility by providing volume-properties if no argument is passed.

Let me know if I can commit it.

Thx
Comment 7 Jeffrey Stedfast 2005-06-09 17:17:36 UTC
what Rob and I meant was that we want it to be the same program with all user
interface elements visible, but just flip to the requested properties notebook tab.

anyways, maybe what we should do is just ask Anna to try to come up with a good
solution for us.
Comment 8 Hubert Figuiere (:hub) 2005-06-09 19:49:16 UTC
I somewhat don't agree. It is confusing for a user to have to open "Removable
disk" to set the preferences for action when you connect a camera to the PC.

Actually one use case missing is "what to do if we insert a picture CD", as they
are following the DCImage standard.

Looks at MacOS X. On 10.2, there is a control panel for digital cameras. For
10.3 they moved it to Image Capture, but I don't agree.
And in the "CD & DVD" control panel, there is the action for picture CD. 
Not that MacOS X is the best UI ever, but they have some good ideas, and that is
what I was trying to push with my patch. 

Nevermind, I'll push that into Ubuntu and we'll see have it comes out from a
user point of view...
Comment 9 Hubert Figuiere (:hub) 2005-06-09 22:02:38 UTC
Actually I start to see the point after talking with fejj.

The current solution I implement is still better than nothing and would do the
transition between current g-v-m and more general purpose "Device" capplet.
Comment 10 Hubert Figuiere (:hub) 2005-06-15 15:01:36 UTC
if you don't rewrite the applet for the GNOME 2.12 feature freeze, please at
least consider having this patch
Comment 11 Thomas M. Hinkle 2005-06-15 21:16:56 UTC
Not sure if this is relevant of not, but when I plug in my camera, it just looks
like a USB drive and is opened as such. Same goes for my mp3player.
Haven't tested lately, but in the past when I've tried to use the "import
photos" dialog provided in gphoto, I had no luck finding the camera.
Anyway, IMO, it's good behavior for these devices to just look like flash drives
to computers since that simplifies everything. That said, it's confusing when
there's a "get my pictures" or "get my music" button that doesn't get you
picturse/music/etc. because they look like a flash drive. So whatever these do,
if there's a separate "camera" or whatever button, make sure it still sees
devices that look like flash drives, because of course your average user won't
know the difference.
So this is somewhat uninformed, but H.F. requested comments, so there's my 2 c.
Comment 12 Hubert Figuiere (:hub) 2006-11-02 18:22:29 UTC
The current version completely change how this is handled making this bug report obsolete.

Closing as OBSOLETE