[patch] tag-jump usability improvement in the help.

[h_east]

Hi Bram and list,

Sometimes tag-jump failed in the help.
Because, the help's 'iskeyword' option contains `(`.

Failed case #1:
:h E29
/textwidth
<C-]>

E149: Sorry, no help for ('textwidth'

Failed case #2:
:h viminfo-%
/buflisted
<C-]>

E149: Sorry, no help for ('buflisted'),

I wrote a patch.
When tag starts `('` skip `(` at ":help".
It is very useful :-)
Please include an attached patch.

--
Best regards,
Hirohito Higashi (a.k.a. h_east)

[h_east]

Hi Bram,

2016-3-14(Mon) 18:28:44 UTC+9 h_east:

> Hi Bram and list,
>
> Sometimes tag-jump failed in the help.
> Because, the help's 'iskeyword' option contains `(`.
>
> Failed case #1:
> :h E29
> /textwidth
> <C-]>
>
> E149: Sorry, no help for ('textwidth'
>
>
> Failed case #2:
> :h viminfo-%
> /buflisted
> <C-]>
>
> E149: Sorry, no help for ('buflisted'),
>
>
> I wrote a patch.
> When tag starts `('` skip `(` at ":help".
> It is very useful :-)
> Please include an attached patch.

If you include the above patch, please include this patch too.
(Workaround inserted space is deleted)

thanks

[Bram Moolenaar]

Hirohito Higashi wrote:

> Hi Bram and list,
>
> Sometimes tag-jump failed in the help.
> Because, the help's 'iskeyword' option contains `(`.
>
> Failed case #1:
> :h E29
> /textwidth
> <C-]>
>
> E149: Sorry, no help for ('textwidth'
>
>
> Failed case #2:
> :h viminfo-%
> /buflisted
> <C-]>
>
> E149: Sorry, no help for ('buflisted'),
>
>
> I wrote a patch.
> When tag starts `('` skip `(` at ":help".
> It is very useful :-)
> Please include an attached patch.

That is useful, thanks.

We should have a test for this complicated stuff...

--
George: "I just got a new set of golf clubs for my wife!"
John: "Great trade!"

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///

[h_east]

Hi Bram,

2016-3-15(Tue) 21:44:36 UTC+9 Bram Moolenaar:

> Hirohito Higashi wrote:
>
> > Hi Bram and list,
> >
> > Sometimes tag-jump failed in the help.
> > Because, the help's 'iskeyword' option contains `(`.
> >
> > Failed case #1:
> > :h E29
> > /textwidth
> > <C-]>
> >
> > E149: Sorry, no help for ('textwidth'
> >
> >
> > Failed case #2:
> > :h viminfo-%
> > /buflisted
> > <C-]>
> >
> > E149: Sorry, no help for ('buflisted'),
> >
> >
> > I wrote a patch.
> > When tag starts `('` skip `(` at ":help".
> > It is very useful :-)
> > Please include an attached patch.
>
> That is useful, thanks.
>
> We should have a test for this complicated stuff...

Thanks for including my patch!
I begin to write a test from now.
Please wait one hour.

[h_east]

Hi Bram,

2016-3-15(Tue) 21:54:57 UTC+9 h_east:

Here is a test.

P.S.
Hopefully, following patch also want to include.
https://groups.google.com/d/msg/vim_dev/fH5gsRE-EjE/PtJD1gL-FwAJ

[Bram Moolenaar]

Thanks. I assume you used "exec" to work around a highlighting problem.
We don't need it for "helpclose".

> P.S.
> Hopefully, following patch also want to include.
> https://groups.google.com/d/msg/vim_dev/fH5gsRE-EjE/PtJD1gL-FwAJ

Already did, will go out with runtime file updates.


--
hundred-and-one symptoms of being an internet addict:
42. Your virtual girlfriend finds a new net sweetheart with a larger bandwidth.

[h_east]

Hi Bram and list,

I wrote another improved patch for the help tagjump.

If tag contains ({ or ([, Tag is set to terminate the (

It will not require a space after the "function_name(" in the help.
Please include this.

[h_east]

Hi Bram and list,

2016-3-26(Sat) 11:40:45 UTC+9 h_east:

> Hi Bram and list,
>
> I wrote another improved patch for the help tagjump.
>
> If tag contains ({ or ([, Tag is set to terminate the (
>
> It will not require a space after the "function_name(" in the help.
> Please include this.

PS
Current official runtime/doc/tags does not contained tag (' and ({ and ([.
When this patch is included, we can no longer tag-jump above tags in official/unofficial help.
Original `(` command is one of the Text object motions. And it ends the command in the `(`.
So that, `({` and `([` with a user-defined command is defined, we believe that almost no.

Someone, Do you have any opinion?

Thanks

[Bram Moolenaar]

Hirohito Higashi wrote:

> Hi Bram and list,
>
> I wrote another improved patch for the help tagjump.
>
> If tag contains ({ or ([, Tag is set to terminate the (
>
> It will not require a space after the "function_name(" in the help.
> Please include this.

Thanks, I'll look into it.

--
hundred-and-one symptoms of being an internet addict:

127. You bring your laptop and cellular phone to church.