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 342921 - Cursor blinking preference should follow system defaults -- see bug 342921 for the correct solution
Cursor blinking preference should follow system defaults -- see bug 342921 fo...
Status: RESOLVED FIXED
Product: gnome-terminal
Classification: Core
Component: general
unspecified
Other Linux
: Normal enhancement
: gnome-2-24
Assigned To: Unknown User
Unknown User
Patch appreciated (see comment 27 for...
: 350105 524851 (view as bug list)
Depends on: 510903
Blocks:
 
 
Reported: 2006-05-25 13:44 UTC by Wouter Bolsterlee (uws)
Modified: 2008-11-09 12:34 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
Make the terminal follow the global cursor_blinks pref (30.10 KB, patch)
2006-08-26 22:20 UTC, Mariano Suárez-Alvarez
none Details | Review
Make the terminal follow the global cursor_blinks pref (30.10 KB, patch)
2006-08-26 22:22 UTC, Mariano Suárez-Alvarez
none Details | Review
Same as before, minus the unneeded glade induced changes to the .glade file (23.77 KB, patch)
2006-08-27 18:36 UTC, Mariano Suárez-Alvarez
none Details | Review

Description Wouter Bolsterlee (uws) 2006-05-25 13:44:33 UTC
The cursor blinking checkbox in the gnome-terminal preferences should be removed. Instead the Gnome-wide cursor blinking setting should be used.
Comment 1 Guilherme de Siqueira Pastore 2006-05-25 17:18:09 UTC
I tend to agree with this one, because it makes sense and because we would be making room for one of the several requested options. Comments?
Comment 2 Wouter Bolsterlee (uws) 2006-05-25 17:33:26 UTC
I'm on a quest to remove all annoying tiny little bits in GNOME's ui ;) any setting that cannot be justified should be nuked. By the way, why is are "background color" and "background" on different tabs?
Comment 3 Olav Vitters 2006-08-06 00:38:54 UTC
*** Bug 350105 has been marked as a duplicate of this bug. ***
Comment 4 Mariano Suárez-Alvarez 2006-08-26 22:20:55 UTC
Created attachment 71685 [details] [review]
Make the terminal follow the global cursor_blinks pref

This removes the checkbox from the profile editor, along with all the code that deals with it and the corresponding the schema. It installs a gconf notification listener to react to changes of the global GNOME key, /desktop/gnome/interface/cursor_blinks. (Along with some changes to the .glade2 file, due to the fact that glade loves to change stuff...)

This will not make the people that have profiles with different settings for blinking very happy, although I wonder if such people actually exist. Maybe there are a11y implications that we are not aware of? Maybe Billh knows.

In any case, since this changes a string (though it is just a warning) and the UI, freezes force to hold the patch.
Comment 5 Mariano Suárez-Alvarez 2006-08-26 22:22:44 UTC
Created attachment 71686 [details] [review]
Make the terminal follow the global cursor_blinks pref

This removes the checkbox from the profile editor, along with all the code that deals with it and the corresponding the schema. It installs a gconf notification listener to react to changes of the global GNOME key, /desktop/gnome/interface/cursor_blinks. (Along with some changes to the .glade2 file, due to the fact that glade loves to change stuff...)

This will not make the people that have profiles with different settings for blinking very happy, although I wonder if such people actually exist. Maybe there are a11y implications that we are not aware of? Maybe Billh knows.

In any case, since this changes a string (though it is just a warning) and the UI, freezes force to hold the patch.
Comment 6 Olav Vitters 2006-08-26 22:28:10 UTC
glade part contains some extra changes (usually happens)
Comment 7 Mariano Suárez-Alvarez 2006-08-27 18:36:46 UTC
Created attachment 71713 [details] [review]
Same as before, minus the unneeded glade induced changes to the .glade file

At some point, one should open the .glade file in glade, save it and commit the changes, in order to avoid this patch-editing thing. 

Of course, glade should avoid unnecessary changes: I rememeber there was a bug on glade about that, but I do not seem to be able to find it now.
Comment 8 Behdad Esfahbod 2006-09-18 17:21:25 UTC
Related bug 356552.
Comment 9 Wouter Bolsterlee (uws) 2007-05-13 17:07:17 UTC
Any progress on this one?
Comment 10 Wouter Bolsterlee (uws) 2008-01-17 11:04:46 UTC
Behdad, can we get this in?
Comment 11 Christian Persch 2008-01-20 18:33:35 UTC
I don't see why we should add gconf listeners when we can just connect to notify::gtk-cursor-{blink,time,timeout} on the GtkSettings object ? And even better would be to make vte automatically follow those settings.

The parts of the patch removing the UI are ok.
Comment 12 Behdad Esfahbod 2008-01-20 18:36:06 UTC
Right.  Go for it!
Comment 13 Christian Persch 2008-01-20 21:40:37 UTC
        * src/gnome-terminal.glade2:
        * src/gnome-terminal.schemas.in:
        * src/profile-editor.c: (profile_changed), (terminal_profile_edit),
        (profile_editor_update_sensitivity):
        * src/terminal-profile.c: (terminal_profile_init),
        (terminal_profile_update), (profile_change_notify),
        (terminal_profile_create):
        * src/terminal-profile.h:
        * src/terminal-screen.c: (terminal_screen_sync_settings),
        (terminal_screen_screen_changed), (terminal_screen_class_init),
        (terminal_screen_finalize), (terminal_screen_reread_profile): Use the
        gtk-cursor-blink setting, and remove the profile option. Bug #342921,
        based on a patch by Mariano Suárez-Alvarez.

Comment 14 Behdad Esfahbod 2008-01-20 22:53:50 UTC
Humm, thinking about this again, should make vte follow gtk+ cursor settings by default?
Comment 15 Christian Persch 2008-01-20 22:57:22 UTC
Definitely yes. Already filed bug 510903 for that; once that's in I'll remove the code from g-t (in the screen-changed handler) again.
Comment 16 Vincent Untz 2008-02-05 11:50:41 UTC
I actually disagree with this change for various reasons:

 + it breaks settings of users

 + I don't know of any easy way to change the gtk+ setting

 + it's a bit similar to the situation where the HIG recommends that the user can override the control center toolbar appearance (so even if we had a central setting for this, we should allow overriding it in the terminal since it's really essential in the interface)

In my personal case, the blinking cursor is probably one of the worst annoyances that can exist in a terminal.

Can we reopen the bug and change this in some way?
Comment 17 Wouter Bolsterlee (uws) 2008-02-05 12:47:12 UTC
Just get rid of the blinking by default ;-)
Comment 18 Wouter Bolsterlee (uws) 2008-02-05 12:49:24 UTC
(In reply to comment #16)
>  + I don't know of any easy way to change the gtk+ setting

http://uwstopia.nl/blog/2006/05/blink-blink
Comment 19 Allison Karlitskaya (desrt) 2008-02-26 08:52:33 UTC
a flashing gnome-terminal cursor block is about 10 times as annoying as a flashing | thin cursor in every other app.

this change has driven me so far to distraction that i've gone and disabled global cursor flashing for my entire desktop, even though i prefer to have the cursor flash in normal applications.

this really needs to be reconsidered.
Comment 20 Allison Karlitskaya (desrt) 2008-02-26 08:54:48 UTC
ps: everyone complaining about this thus far (and even those requesting it in the first place, including the duplicate) all want their terminal cursor not to blink.

maybe it makes more sense simply to hardwire the terminal cursor to never blink (without removing the choice for the rest of the desktop).
Comment 21 Behdad Esfahbod 2008-02-26 09:01:02 UTC
/me wants his terminal cursor blinking...
Comment 23 Forlong 2008-03-01 13:14:38 UTC
Although it's good that the Gnome-Terminal follows the system-wide option now, I don't see why the option has been removed from the terminal's profile options.

I like my blinking cursors *except* in a specific terminal profile (for a terminal that sits on my desktop).

Can the option brought back please?
Comment 24 Alex Shulgin 2008-03-24 09:30:50 UTC
Hey, were do I send hate mail for this change? :)

I'm quite accustomed to thick-non-blinking cursor in terminal _and_ thin-and-blinking cursor in other editable areas like gedit.

And this is for a reason:

1. The thick cursor in terminal just seems natural and, when it's thick, it's already easy to spot and blinking is just a big annoyance.
2. The thin cursor isn't that easy (at least for me) to spot when it doesn't blink.

With the current options I cannot have both thick-non-blinking and thin-and-blinking one...

So why don't we have _two_ options, like these?

1. Thin cursor blinks (default: yes).
2. Thick cursor blinks (default: no).

--
Cheers,
Alex
Comment 25 Anthony 2008-03-26 06:12:03 UTC
After having just installed The 8.04 beta1 of Ubuntu I fired up the default terminal and decided to play some nethack on nethack.alt.org.

So, I set up my nethack profile to black on white and telnetted away but to my horror my poor little @ just wouldn't stop blinking.  ARRRRGH.  Now, I see that I can turn off the cursor system wide but that is a poor solution.  PLEASE bring back the option to switch cursor options.

In the meantime, I'm off to install some other terminals that I can adjust settings on properly without changing the rest of my system.
Comment 26 Vincent Untz 2008-03-26 11:16:51 UTC
I thought I added a comment about this, but I think this setting, in the context of  gnome-terminal, could be overridable from the application, like http://library.gnome.org/devel/hig-book/stable/toolbars-controlling-display.html.en suggests for the display of toolbars.
Comment 27 Wouter Bolsterlee (uws) 2008-03-26 11:19:03 UTC
So you suggest a "System default", "Always", "Never" combo widget?
Comment 28 Vincent Untz 2008-03-26 11:53:28 UTC
System default, Blink, Do not blink. Or something like this. The checkbox looked "simpler", but if following the system default is important, then it's a way to solve this.
Comment 29 Christian Persch 2008-03-28 23:13:41 UTC
*** Bug 524851 has been marked as a duplicate of this bug. ***
Comment 30 Nikolai Weibull 2008-04-07 21:05:25 UTC
I got to say that I too was terribly disturbed to find my gnome-terminal wh^H^Hblinking at me.  Please bring back the option and set it to “System default” or “Off” by default.
Comment 31 Mikael Hakali 2008-04-08 17:05:25 UTC
I just registered an account to display my dislike of the fact that I cannot have blinking thin cursors and non-blinking terminal cursors at the same time. This is just bananas and breaks my working environment which I have used over the past 2 years :(

+1 to adding it as a profile option to override system defaults. 
Comment 32 Christian Persch 2008-04-08 17:12:31 UTC
Don't spam the bug with useless comment please. Provide a patch instead, as outlined in comment 27 ff.
Comment 33 Alex Shulgin 2008-04-08 18:42:18 UTC
> Don't spam the bug with useless comment please.

I personally don't think that these comments are completely useless.  Now at least you can see that there are people who need both options.

> Provide a patch instead, as outlined in comment 27 ff.

I doubt that an average gnome-terminal user (even if he's experienced programmer himself) can do such patch in a reasonable time.  You as a gnome-terminal developer are in much more comfortable position for this task. ;-)

--
Regards,
Alex
Comment 34 Olav Vitters 2008-04-08 19:00:12 UTC
(In reply to comment #33)
> I personally don't think that these comments are completely useless.  Now at
> least you can see that there are people who need both options.

The developer now mentioned that such a change would be accepted, so any additional comments (after that) would be noise. Although I wasn't aware until comment 32 that such a solution would be acceptable.
Comment 35 bjorn 2008-04-15 07:06:45 UTC
Any news on this one? Now Ubuntu Hardy will almost be out and I'd really hate to be forced to compile my own gnome-terminal just so I can turn off cursor blinking.

Maybe Mariano and Christian could pick this up again? They seem to have removed the option after all...
Comment 36 Vincent Untz 2008-04-15 11:21:56 UTC
(In reply to comment #35)
> Any news on this one? Now Ubuntu Hardy will almost be out and I'd really hate
> to be forced to compile my own gnome-terminal just so I can turn off cursor
> blinking.

Keep in mind that GNOME & Ubuntu are two different things ;-) It won't be fixed upstream in GNOME for Hardy because the changes implies a UI change and a UI change can't be done in GNOME 2.22. Also, Hardy will be out quite soon and it's more or less frozen too now...
Comment 37 Philip Belemezov 2008-04-27 00:20:15 UTC
I too was shocked to see the cursor blinking and the option removed after upgrading to gnome 2.22.

As previously suggested by others, the best approach is to provide three different options ("System default", "Always", "Never").

Comment 38 Ralph Corderoy 2008-05-04 15:23:45 UTC
Having just made the Ubuntu 7.10 to 8.04 upgrade.  This change is very short-sighted.  There's a big difference in appearance between the one-pixel-wide cursor used in text areas and the one-character-rectangle cursor used by gnome-terminal.  It's helpful for the thin one to blink because it can be hard to spot.  The block one is big, fat, and hairy enough on its own so can easily be spotted and blinking is a huge distration.  We block animated ads and blinking HTML but now have a blinking blinking rectangle in our terminal windows.  Something simply must be done.
Comment 39 Olav Vitters 2008-05-04 16:41:01 UTC
> Something simply must be done.

So go ahead and provide a patch (if able). Saying 'me too' while the developer agrees will just make this bugreport unreadable.
Comment 43 Olav Vitters 2008-05-05 16:48:48 UTC
Fixing typo in Bugzilla Tip. Please follow that suggestion (meaning: no discussion). Removing comments that didn't follow this suggestion.
Comment 44 Behdad Esfahbod 2008-05-05 17:01:08 UTC
Wouter, can I bribe you into submitting a patch?
Comment 45 Wouter Bolsterlee (uws) 2008-05-05 18:01:10 UTC
(In reply to comment #44)
> Wouter, can I bribe you into submitting a patch?

I'm sorry. I don't speak C. :) chpe might perhaps?
Comment 46 Ralph Corderoy 2008-05-07 11:09:18 UTC
Why have comments #40 and #42 from Jay L. T. Cornwall been removed?  I presume by Olav from #43.  Jay's comments were polite, helpful, and he was willing to provide a patch and trying to discuss the patch's content.  If I was Jay, I'd find it very discouraging and not bother to contribute further.

Unless a better offer comes along, a patch to revert to the old behaviour would seem the best route;  have gnome-terminal have its own cursor blink setting for its own big fat cursor.  That would satisfy most complaints.  If someone wants to tidy things up further in the future with a use-system-default/no-blink/blink combo then they can under a separate timescale but it's not worth holding up fixing this bug for that to happen.
Comment 47 bjorn 2008-05-07 13:13:42 UTC
I agree with Ralph and also found the comment by Jay L. T. Cornwall to be constructive and sensible.
Comment 48 Olav Vitters 2008-05-07 20:06:12 UTC
I polity asked not to post off topic comments to keep the bugreport readable. Seems this bug only gets such responses, making it unusable. Closing bug.
Comment 49 Olav Vitters 2008-05-07 20:12:05 UTC
PS: Every point has been raised before. Reverting also wouldn't do anything. #1 it wouldn't be available in stable (UI break aren't allowed) #2 nobody would implement the right thing anymore.
Comment 50 Olav Vitters 2008-05-17 07:22:37 UTC
I hoped with closing this that people would read the bugzilla tip, etc. Heard that it is misinterpreted to mean that GNOME doesn't listen or whatever. I think this bug states often enough that:


  Yes, we agree fully

  Yes, following system default is way from ideal

  Yes, something will be down about it


In short: I've opened a new bug to implement the correct thing. That is bug 533522. So please don't state we didn't listen; I'm just doing what the bugzilla tip stated up to know (file new bug -- no patch as I am not a dev).


Note: that bug would mean an UI freeze break if it would be implemented for 2.22 (developers cannot just commit such changes to a stable branch). However: exceptions can be asked (by devs) and made. Further, distributions are free to do what they want.