#15130 closed defect (fixed)

docs fixes

Reported by: catalin Owned by: vadz
Priority: normal Milestone:
Component: documentation Version: stable-latest
Keywords: Cc:
Blocked By: Blocking:
Patch: yes

Description

A few changes to the docs.

Attachments (1)

15130_docs.patch download (4.0 KB) - added by catalin 16 months ago.

Download all attachments as: .zip

Change History (9)

Changed 16 months ago by catalin

comment:1 follow-up: Changed 16 months ago by vadz

  • Owner set to vadz
  • Status changed from new to accepted

Thanks, I'll apply those soon except the changes to the filepicker.h because:

  1. CheckPath() doesn't even exist (it's declared but not defined, I'll remove the declaration too).
  2. I'm not sure if GetTextCtrlValue() is supposed to be public as I'm not sure when exactly is it useful and what for.

comment:2 in reply to: ↑ 1 Changed 16 months ago by catalin

Replying to vadz:

  1. CheckPath() doesn't even exist (it's declared but not defined, I'll remove the declaration too).
  2. I'm not sure if GetTextCtrlValue() is supposed to be public as I'm not sure when exactly is it useful and what for.

I got CheckPath() from the header, didn't realize it doesn't exist.
But GetTextCtrlValue() is the reason I changed this file. I couldn't find another way of getting the value of the text field when it was filled in manually and not by coosing something in the dir picker.

comment:3 Changed 16 months ago by vadz

Hmm, so it looks like this one actually should always be used instead of GetPath() for the pickers with text control part? Because otherwise we could not be getting the latest value... IOW maybe we should modify GetPath() instead?

comment:4 Changed 16 months ago by catalin

I realize now that I tried that in 2.8.12. I just checked with 29-svn and GetPath() works as expected and there's no need for GetTextCtrlValue(), so it can be left out too.

comment:5 Changed 16 months ago by VZ

(In [73750]) Document wxDataView{List,Tree}Ctrl as only available since 2.9.0.

See #15130.

comment:6 Changed 16 months ago by VZ

(In [73751]) Document wxDateTime arithmetic operators.

wxDateTime::operator+() and operator-() were not documented, do it now.

Also fix the return value of wxDateTime::Add().

See #15130.

comment:7 Changed 16 months ago by VZ

(In [73752]) Fix the header to be included in wxStringTokenizer documentation.

See #15130.

comment:8 Changed 16 months ago by VZ

  • Resolution set to fixed
  • Status changed from accepted to closed

(In [73753]) Fix the style value in wxWizard::Create() documentation.

Somehow the raw value was used instead of the symbolic constant.

Closes #15130.

Note: See TracTickets for help on using tickets.