#14707: Ambiguous wxHelpControllerBase::SetFrameParameters parameter name

3 views
Skip to first unread message

wxTrac

unread,
Sep 28, 2012, 4:09:47 AM9/28/12
to wx-...@googlegroups.com
Ticket URL: <http://trac.wxwidgets.org/ticket/14707>

#14707: Ambiguous wxHelpControllerBase::SetFrameParameters parameter name
---------------------------------------+------------------------------------
Reporter: johnr | Owner:
Type: defect | Status: new
Priority: normal | Milestone:
Component: GUI-all | Version: 2.9-svn
Keywords: wxHelp SetFrameParameters | Blockedby:
Patch: 1 | Blocking:
---------------------------------------+------------------------------------
On reading the docs, I thought SetFrameParameters( wxString title,...) set
the label title for the help frame but it sets the format string for the
title. I note the docs have changed with an improved note but
wxExtHelpController is still unclear.

Patch changes "title" to "titleFormat" for anyone else who might read this
as literally as I did.


--
Ticket URL: <http://trac.wxwidgets.org/ticket/14707>

wxTrac

unread,
Sep 28, 2012, 7:48:59 PM9/28/12
to wx-...@googlegroups.com
Ticket URL: <http://trac.wxwidgets.org/ticket/14707#comment:1>

#14707: Ambiguous wxHelpControllerBase::SetFrameParameters parameter name
----------------------+-----------------------------------------------------
Reporter: johnr | Owner:
Type: defect | Status: closed
Priority: normal | Milestone:
Component: GUI-all | Version: 2.9-svn
Resolution: fixed | Keywords: wxHelp SetFrameParameters
Blockedby: | Patch: 1
Blocking: |
----------------------+-----------------------------------------------------
Changes (by VZ):

* status: new => closed
* resolution: => fixed


Comment:

(In [72573]) Make wxHelpControllerBase::SetFrameParameters() title more
clear.

Describe it as "title format string" and call it "titleFormat" and not
just
"title" because this is what it is.

Closes #14707.


--
Ticket URL: <http://trac.wxwidgets.org/ticket/14707#comment:1>
Reply all
Reply to author
Forward
0 new messages