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 345252 - Automatic re-enter not function correctly
Automatic re-enter not function correctly
Status: RESOLVED FIXED
Product: gtksourceview
Classification: Platform
Component: General
unspecified
Other All
: Normal trivial
: ---
Assigned To: GTK Sourceview maintainers
GTK Sourceview maintainers
Depends on:
Blocks:
 
 
Reported: 2006-06-18 17:13 UTC by Giovanni Roggio
Modified: 2006-06-23 08:24 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
patch (878 bytes, patch)
2006-06-20 21:18 UTC, Paolo Borelli
none Details | Review
patch against HEAD (1.41 KB, patch)
2006-06-22 13:10 UTC, Steve Frécinaux
committed Details | Review

Description Giovanni Roggio 2006-06-18 17:13:54 UTC
Please describe the problem:
If i press enter on the numeric-pad automatic re-enter not function

Steps to reproduce:
1. Press enter with numeric-pad


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Steve Frécinaux 2006-06-18 18:15:22 UTC
mh, and what is exactly “re-enter” ?
Comment 2 Paolo Borelli 2006-06-18 19:15:54 UTC
I guess Giovanni means auto-indent...
Comment 3 Giovanni Roggio 2006-06-18 22:43:46 UTC
Sorry i've the italian version of gnome ;D
Auto-ident it's correct
Comment 4 Paolo Borelli 2006-06-20 17:32:57 UTC
Does pressing the Numeric Keypad's Enter cause the cursor to go on a new line (without autoindentation) or it does nothing at all?

Do you by any chance have gtksourceview installed from source? It would be nice to  know if it happens with gtksourceview's test-widget...


Steve, Jesse: can someone replicate this? (I just have a laptop here so no keypad...)
If yes it would be nice to break into gtk_source_view_key_press_event() and take a look at the event->keyval and event->state values.
Comment 5 jessevdk@gmail.com 2006-06-20 18:19:48 UTC
I can confirm this. Pressing Enter on the numpad makes the cursor go to the next line without auto indentation. This also happens with the gtksourceview test app.
Comment 6 Paolo Borelli 2006-06-20 18:56:40 UTC
Ok, moving to sourceview... Jesse, can you take a look at the event->keyval value?

My wild guess it's just that we need to add another symbol beside GDK_Return, (though I don't see a GDK_NumPad_Return or something like that)
Comment 7 jessevdk@gmail.com 2006-06-20 19:47:41 UTC
Found it: GDK_KP_Enter
Comment 8 Paolo Borelli 2006-06-20 21:18:28 UTC
Created attachment 67738 [details] [review]
patch

trivial patch (should apply with some offsets because I have other local changes).


Can you guys test it?
Comment 9 Steve Frécinaux 2006-06-22 13:10:08 UTC
Created attachment 67844 [details] [review]
patch against HEAD

Same patch as the previous one, but against CVS HEAD (I didn't see there was already one patch attached)

I tested it quickly with the test widget, it works ok.
Comment 10 Paolo Maggi 2006-06-22 16:57:42 UTC
Please, commit in HEAD and (if we already branched) in branch.
Comment 11 Steve Frécinaux 2006-06-23 08:24:06 UTC
Comitted. I don't think we have branched yet.