[vim/vim] st (simple terminal from suckless.org) supports xterm-style mouse (#963)

448 views
Skip to first unread message

Manuel Schiller

unread,
Aug 2, 2016, 5:45:22 AM8/2/16
to vim/vim

Add an explicit match against $TERM: if $TERM is either exactly "st" or
starts with "st-", or with "stterm", xterm-like mouse support is
enabled. The slightly convoluted logic in the string match is there to
avoid false positives from terminal names like "st52", and whatever else
might exist out there.

You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/963

Commit Summary

  • st (simple terminal from suckless.org) supports xterm-style mouse

File Changes

Patch Links:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, .

Manuel Schiller

unread,
Aug 7, 2016, 7:41:06 AM8/7/16
to vim/vim, Push

@manuelschiller pushed 9 commits.

—
You are receiving this because you are subscribed to this thread.

View it on GitHub .

Anmol Sethi

unread,
Aug 7, 2016, 8:21:31 PM8/7/16
to vim/vim

Could you please squash the commits into the relevant two?

Manuel Schiller

unread,
Aug 8, 2016, 5:44:31 AM8/8/16
to vim/vim

Hi, I committed another patch without thinking (my bad - I'm kind of new to github). If it's okay, I'll create two separate branches with only the relevant fixes on, and will then open a new pull request containing only that one cherry-picked fix. Would that be acceptable?

Anmol Sethi

unread,
Aug 8, 2016, 5:49:24 AM8/8/16
to vim/vim

You might not even need to do anything because Bram does not actually merge, he will take only the relevant commits.

Either way, try and fix this branch, it'll be a nice way to learn git.

Manuel Schiller

unread,
Aug 8, 2016, 5:51:10 AM8/8/16
to vim/vim

Closed #963.

Manuel Schiller

unread,
Aug 8, 2016, 5:52:12 AM8/8/16
to vim/vim

Opening new separate pull requests with one patch per issue. Sorry for the noise.

Reply all
Reply to author
Forward
0 new messages