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 322514 - mouse capplet doesn't list the 'default' cursor theme
mouse capplet doesn't list the 'default' cursor theme
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Mouse
2.15.x
Other All
: Normal minor
: 2.16
Assigned To: Sven Herzberg
Control-Center Maintainers
: 329436 350594 351069 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-11-26 18:53 UTC by Chris Rebert
Modified: 2007-02-13 16:57 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
ugly mockup of the "reset"-button approach (25.39 KB, image/png)
2006-08-16 16:05 UTC, Christian Lohmaier
  Details
Proposed patch (3.34 KB, patch)
2006-08-16 17:07 UTC, Sven Herzberg
committed Details | Review

Description Chris Rebert 2005-11-26 18:53:15 UTC
Please describe the problem:
The Mouse pref panel doesn't list the 'default' cursor theme
(i.e. the xorg default, th one you would have on a fresh install before using
this control panel). As a result, if you change the cursor to one you dislike
and you preferred the default, you can't switch back to the default, even though
it's in the same directory (/usr/share/cursors/xorg-x11/) as the other themes.

Steps to reproduce:
1. On a fresh install of gnome/xorg, Go to G->Desktop->Prefs->Mouse
2. Go to the cursors tab
3. note your current pointer
4. change to one of the themes listed
5. try to revert to the previous theme


Actual results:
The 'default' (i.e. xorg default cursor) theme isn't listed, and thus this is
impossible.

Expected results:
There should have been a listing for the 'default' theme.

Does this happen every time?
Yes

Other information:
workaround:
use the configuration editor (G->System Tools->Configuration Editor) to change
the value of
/desktop/gnome/peripherals/mouse/cursor_theme
to
'default' (sans quotes)
Comment 1 Chris Rebert 2005-11-26 18:54:06 UTC
See also http://bugs.gentoo.org/show_bug.cgi?id=111930
Comment 2 Sebastien Bacher 2005-12-30 15:28:05 UTC
Thanks for your bug
Comment 3 Sven Herzberg 2006-07-07 15:14:10 UTC
*** Bug 329436 has been marked as a duplicate of this bug. ***
Comment 4 Sven Herzberg 2006-07-07 15:20:03 UTC
Please execute 'locate cursor.theme' and tell us the location of the cursor theme that gets loaded by default. Thank you.
Comment 5 Chris Rebert 2006-07-07 21:40:55 UTC
"locate cursor.theme" yields no results on my box.
Comment 6 Jens Granseuer 2006-07-12 11:15:24 UTC
Neither does it for me. I'm not sure if the default X.org cursor theme is even a file (i.e. I don't even have a /usr/share/cursors or similar).
Comment 7 Sven Herzberg 2006-07-24 09:10:04 UTC
I see two ways to fix it:

1. Always add the default theme without preview (this will look ugly and maybe users will claim that its broken).

2. Remove the preview code to make the dialog look more like the theme capplet. As we have instant-apply wihtin this dialog I think the preview is less important as users can see the results of their actions immediately. This way not having a preview for the default theme would suck less.
Comment 8 Jens Granseuer 2006-07-24 13:20:37 UTC
The problem with removing the preview (in contrast to the theme capplet) is that it's much more work to see what your pointer's going to look like. You will see the standard pointer, of course, but then you'll have to start dragging stuff around, resizing windows, etc. That's pretty bad.

If it's not possible to create a preview of the built-in theme (no idea what the cursor API looks like) I'd still vote for option 1. Since you'd have to special-case the default theme anyway, you could just display a message explaining why there is no preview for it.
Comment 9 Chris Rebert 2006-07-25 02:38:33 UTC
If it helps, the Cursor tab of XFCE's Mouse Preferences lists the "default" theme, but doesn't show a preview for it like it does for the others.
Comment 10 Christian Kirbach 2006-08-10 13:37:37 UTC
*** Bug 350594 has been marked as a duplicate of this bug. ***
Comment 11 Christian Kirbach 2006-08-10 13:38:35 UTC
from last duplicate

  • #8 IA__g_log
    at gmessages.c line 517
  • #9 IA__g_return_if_fail_warning
  • #10 cursor_theme_changed
    at gnome-mouse-properties.c line 484
  • #11 main
    at gnome-mouse-properties.c line 956

Comment 12 Sven Herzberg 2006-08-10 14:38:41 UTC
The backtrace doesn't bring us forward since I still think that this g_return_*() statement is correct.

Not correct is the fact that the default cursor theme is not displayed (which would - if fixed - also fix the runtime warning you guys are triggering).

Patch welcome
Comment 13 Fabio Bonelli 2006-08-13 08:18:18 UTC
*** Bug 351069 has been marked as a duplicate of this bug. ***
Comment 14 Christian Lohmaier 2006-08-15 23:14:23 UTC
a mee-too style of comment, 

but a remark to the solutions stated in comment #7

Why not make it a button "reset to default" instead of adding it as an entry in the preview box or disabling preview altogether?
Comment 15 Sven Herzberg 2006-08-16 11:04:56 UTC
> Why not make it a button "reset to default" instead of adding it as an entry in
> the preview box or disabling preview altogether?

I actually have several problems with this proposal:

1. You're just changing the cursor theme, so why not do it the same way as we do
   for all the other themes?
2. How would you display "default theme selected" in your UI proposal?
3. Don't you think users would be confused by an empty list to choose from, plus
   a button that says "reset to default". I'd not take this dialog serious, being
   a user.
Comment 16 Christian Lohmaier 2006-08-16 16:03:47 UTC
> 1. You're just changing the cursor theme, so why not do it the same way as we
>    do for all the other themes?

??? You're changing the cursor theme. Right. But currently this is a one-way. You cannot unod the change (apart from hacking gconf-settings). That's why this bug was filed.
Other themes changed can be undone by simply choosing the previously used theme. This is not possible with the current dialog.

> 2. How would you display "default theme selected" in your UI proposal?

I would not display it. Why should it be displayed? It is plain ol' X.

If you acciently reset to default and don't like it, just use one of the offered cursors again. As you wrote: Only chaning the cursor theme, so playing with the different curors offered is not a big deal.

> 3. Don't you think users would be confused by an empty list to choose from,
>    plus a button that says "reset to default".

Not at all. I'm confused by your comment. 
Why should the list to choose from be empty?

>    I'd not take this dialog serious, being a user.

Being a user, I don't take dialogs serious that don't let me change back to default.
Comment 17 Christian Lohmaier 2006-08-16 16:05:33 UTC
Created attachment 71020 [details]
ugly mockup of the "reset"-button approach

The button in the middle should reset the theme to X's default. No fancy other stuff required.

Whether you put it in the bottom row or place it somewhere else is not relevant.
Comment 18 Jens Granseuer 2006-08-16 16:19:39 UTC
(In reply to comment #16)
> > 3. Don't you think users would be confused by an empty list to choose from,
> >    plus a button that says "reset to default".
> 
> Not at all. I'm confused by your comment. 
> Why should the list to choose from be empty?

Because some people (like me) don't have any cursor themes installed. I'd rather have a "default" theme in the list, even if that doesn't offer a preview.
Comment 19 Christian Lohmaier 2006-08-16 16:28:28 UTC
Well, the redglass, handheld and whiteglass I have come with XFree86 - so...

But anyway:

If there are no themes, then the dialog box should tell so: 

"No cursor themes installed"

Instead of offering something to choose from/instead of offering "empty" choices at all.

Whether you show or hide the reset button in that case or just disable it is again an implementation detail.

And again: The main thing is to be able to switch to the default theme again after playing around with the cursor themes.[1]

The Reset-button just was a proposal, since I really think that it is the wrong choice to disable the previews altogether.

[1] one of the main reasons being that the new cursor is not applied everywhere - e.g. inside epiphany's body the cursor still is the default, whereas in the menu/toolbar area the themed one is used - but that's another story.
Comment 20 Sven Herzberg 2006-08-16 16:56:10 UTC
In reply to comment #18
> Because some people (like me) don't have any cursor themes installed. I'd
> rather have a "default" theme in the list, even if that doesn't offer a
> preview.

Thanks Jens, I forgot that one.

Christian, to cut a long story short: I start fixing this bug right now. And my fix will not include ANY reset button.
Comment 21 Sven Herzberg 2006-08-16 17:01:14 UTC
In reply to comment #19
> If there are no themes, then the dialog box should tell so: 
>
> "No cursor themes installed"
>
> Instead of offering something to choose from/instead of offering "empty"
> choices at all.

Christian, this is wrong. As long as there is a default theme, there's no need to claim that no theme is installed.

> And again: The main thing is to be able to switch to the default theme again
> after playing around with the cursor themes.[1]

... which you can pretty easily do by just selecting your default cursor from a list.

> [1] one of the main reasons being that the new cursor is not applied everywhere
> - e.g. inside epiphany's body the cursor still is the default, whereas in the
> menu/toolbar area the themed one is used - but that's another story.

Right, IIRC this is a mozilla story that should be brought up at bugzilla.mozilla.com

Just one line more: the default theme IS a theme, so there's no reason to claim there's no installed theme (which I would consider being a lie). As the default theme is a theme, why not treat it like all others (except for the preview thing, which has got technical reasons)?
Comment 22 Sven Herzberg 2006-08-16 17:04:52 UTC
In reply to comment #17:
> Created an attachment (id=71020) [edit]
> ugly mockup of the "reset"-button approach

Putting the button into the button bar would make it available for the other tabs too. So this would actually confuse me if I modify the double click delay and "reset to defaults" doesn't seem to do anything but change my cursor theme.
Comment 23 Sven Herzberg 2006-08-16 17:07:44 UTC
Created attachment 71023 [details] [review]
Proposed patch

Proposed patch, created with -b so whitespace changes don't appear. Cleaning up the code a bit, added the default theme with a localized name (which is already used in the !XCursor case, so there's actually no new string)
Comment 24 Sebastien Bacher 2006-08-16 17:12:35 UTC
thank you for the patch Sven, from a quick look it seems to be fine to me, feel free to commit!
Comment 25 Sven Herzberg 2006-08-16 17:16:50 UTC
2006-08-16  Sven Herzberg  <herzi@gnome-de.org>

        * gnome-mouse-properties.c: also display the default theme (though
        without a preview for technical reasons); fixes bug #322514
Comment 26 Jens Granseuer 2006-08-16 17:58:23 UTC
Looks good here. Thanks Sven!
Comment 27 Christian Lohmaier 2006-08-17 23:45:17 UTC
I would have expected that the default entry would be at the top of the list, not at the bottom - but apart from that I can confirm that it works here as well.
Comment 28 Sven Herzberg 2006-08-18 10:57:13 UTC
In reply to comment #27
> I would have expected that the default entry would be at the top of the list,
> not at the bottom

It does not appear at the bottom. It appears in alphabetical order.
Comment 29 Christian Lohmaier 2006-08-18 11:22:57 UTC
Having it in alphabetical order is even less fortunate IMHO - but the main problem is fixed and that is what counts...
Comment 30 Sam Morris 2007-02-13 16:57:42 UTC
When I pick the Default-Pointer theme in gnome 2.16 I still have the industrial cursors. Does this work for anyone else?