Re: [rt.cpan.org #103235] Support for TreeListCtrl->EnsureVisible(item)

[email protected] ("Steve Cookson - gmail via RT")
Newsgroups perl.wxperl.users
Message-ID <[email protected]>
Sun Apr 12 10:31:10 2015: Request 103235 was acted upon.
Transaction: Correspondence added by [email protected]
       Queue: Wx
     Subject: Re: [rt.cpan.org #103235] Support for TreeListCtrl->EnsureVisible(item)
   Broken in: (no value)
    Severity: (no value)
       Owner: Nobody
  Requestors: [email protected]
      Status: new
 Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=103235 >


Hi Robert,

The corresponding change in wxPerl would be to the file:

TreeCtrl.xs in the wxPerl/XS directory.

The syntax looks a little painful, maybe because of the history of 
wxTreeCtrl, but if you took the existing void function IsExpanded and 
copied it to create this:

    #if WXPERL_W_VERSION_GE(3, 0, 2 )

    void
    wxTreeCtrl::EnsureVisible( item )
         wxTreeItemId* item
       C_ARGS: *item

    #endif

then did a sudo make install from the wxPerl directory, I imagine it 
would work.  I did add the patch you sent to my wxWidgets install and 
got a clean compile, but I'm afraid that for other reasons I don't get a 
clean compile on wxPerl! So I'm sorry that I can't check it for you.

If it does work, I'm sure  Mark would appreciate a patch.

Good luck,

Steve.

On 31/03/15 16:29, Rowe, Thomas via RT wrote:
> Tue Mar 31 15:29:42 2015: Request 103235 was acted upon.
> Transaction: Ticket created by [email protected]
>         Queue: Wx
>       Subject: Support for TreeListCtrl->EnsureVisible(item)
>     Broken in: (no value)
>      Severity: (no value)
>         Owner: Nobody
>    Requestors: [email protected]
>        Status: new
>   Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=103235 >
>
>
> Hi, I have a Wx pull request pending that adds support for wxTreeListCtrl::EnsureVisible.
> https://github.com/wxWidgets/wxWidgets/pull/22
>
>
> Perhaps it's premature to request the corresponding wxperl change, but I'm using it locally and hope to find it incorporated upstream in a future release.
>
> Thanks.
>
>
> diff --git a/interface/wx/treelist.h b/interface/wx/treelist.h
> index 96678c6..d8cb794 100644
> --- a/interface/wx/treelist.h
> +++ b/interface/wx/treelist.h
> @@ -211,6 +211,8 @@ wxTreeListCtrl::GetImageList()
>
>       bool IsExpanded(wxTreeListItem item) const;
>
> +    void EnsureVisible(wxTreeListItem item);
> +
>       wxTreeListItem GetSelection() const;
>
>   %{
>
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.