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 756500 - vim: Add BOL and EOL bindings in C mode
vim: Add BOL and EOL bindings in C mode
Status: RESOLVED FIXED
Product: gnome-builder
Classification: Other
Component: editor
unspecified
Other All
: Normal normal
: ---
Assigned To: GNOME Builder Maintainers
GNOME Builder Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-10-13 13:30 UTC by simonsonc
Modified: 2015-10-13 17:54 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
vim: Add BOL and EOL bindings in C mode (1.53 KB, patch)
2015-10-13 13:30 UTC, simonsonc 		committed Details | Review

Description simonsonc 2015-10-13 13:30:16 UTC 
Using the commands "c0", "c^" and "c$" one can change the up to the beginning
or ends of lines.
Comment 1 simonsonc 2015-10-13 13:30:20 UTC 
Created attachment 313187 [details] [review]
vim: Add BOL and EOL bindings in C mode
Comment 2 Ray Strode [halfline] 2015-10-13 13:50:26 UTC 
Review of attachment 313187 [details] [review]:

oh cool

::: data/keybindings/vim.css
@@ +534,3 @@
+             "delete-selection" ()
+             "end-macro" () };
+  bind "<shift>asciicircum" { "begin-macro" ()

is the <shift> needed here? I don't think it's necessary

@@ +547,3 @@
+                  "end-macro" () };
+
+

extra space
Comment 3 simonsonc 2015-10-13 14:32:10 UTC 
(In reply to Ray Strode [halfline] from comment #2)
> Review of attachment 313187 [details] [review] [review]:
> 
> oh cool
> 
> ::: data/keybindings/vim.css
> @@ +534,3 @@
> +             "delete-selection" ()
> +             "end-macro" () };
> +  bind "<shift>asciicircum" { "begin-macro" ()
> 
> is the <shift> needed here? I don't think it's necessary
> 
> @@ +547,3 @@
> +                  "end-macro" () };
> +
> +
> 
> extra space

Not sure. I copied it from another location in the file.
Comment 4 Christian Hergert 2015-10-13 17:52:27 UTC 
Review of attachment 313187 [details] [review]:

LGTM. Dropped shift in both places as part of the push.
Comment 5 Christian Hergert 2015-10-13 17:54:19 UTC 
This problem has been fixed in the unstable development version. The fix will be available in the next major software release. You may need to upgrade your Linux distribution to obtain that newer version.

I also anticipate this landing in 3.18.1.