dc73d7f5d4
Fix confusion of what cell activation is and inconsistence with native handling in GTK+. Document the distinction between activating (~ editing) a cell and activating (double-clicking) the whole item. Deprecate wxDataViewCustomRenderer::LeftClick() and Activate() methods, replace them with single ActivateCell() that is called for both kinds of activation. Fix implementations so that ActivateCell() is not called on double-click, when it shouldn't, and vice versa: don't send wxEVT_COMMAND_DATAVIEW_ITEM_ACTIVATED for cell activation. Partially reverts r67099 -- restores old 2.9 signatures of compatibility LeftClick() and Activate() methods. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@69473 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
191 lines
7.9 KiB
C++
191 lines
7.9 KiB
C++
/////////////////////////////////////////////////////////////////////////////
|
|
// Name: changes_since28.h
|
|
// Purpose: topic overview
|
|
// Author: Vadim Zeitlin
|
|
// Created: 2008-05-08
|
|
// RCS-ID: $Id$
|
|
// Licence: wxWindows licence
|
|
/////////////////////////////////////////////////////////////////////////////
|
|
|
|
/**
|
|
|
|
@page overview_changes_since28 Changes Since wxWidgets 2.8
|
|
|
|
This topic describes backwards-incompatible changes in wxWidgets 3.0 compared
|
|
to the last stable release.
|
|
|
|
The incompatible changes can be grouped into the following categories:
|
|
|
|
@li @ref overview_changes_unicode
|
|
@li @ref overview_changes_other
|
|
|
|
|
|
<hr>
|
|
|
|
@section overview_changes_unicode Unicode-related Changes
|
|
|
|
If you used Unicode build of wxWidgets 2.8 or previous version, please read
|
|
@ref overview_unicode for the details about how the API changed in 3.0 as a lot
|
|
of the information which was correct before doesn't apply any longer.
|
|
|
|
For example, the notorious (due to the confusion they created) macros @c wxT()
|
|
and @c _T() are not needed at all any longer. Basically, you can remove them
|
|
from any code which used them. On the other hand, there is no particular harm
|
|
in leaving them neither as the code will still compile and work correctly --
|
|
you only need to remove them if you think that your code looks tidier without
|
|
them. You also don't need to use @c wxChar any longer but can directly use the
|
|
standard @c wchar_t type even if, again, @c wxChar continues to work.
|
|
|
|
The most serious backwards-incompatible change is related to the change of
|
|
return type of wxString::c_str() method: it returns a special proxy object
|
|
instead of a simple @c char* or @c wchar_t* now. Because of this, you cannot
|
|
pass its result to any standard vararg functions such as @c printf() any more
|
|
as described in @ref overview_unicode_compilation_errors. All wxWidgets
|
|
functions, such as wxPrintf(), wxLogMessage() &c still work with it, but
|
|
passing it to @c printf() will now result in a crash. It is strongly advised to
|
|
recompile your code with a compiler warning about passing non-POD objects to
|
|
vararg functions, such as g++.
|
|
|
|
The change of the type of wxString::c_str() can also result in compilation
|
|
errors when passing its result to a function overloaded to take both narrow and
|
|
wide strings and in this case you must select the version which you really want
|
|
to use, e.g.:
|
|
@code
|
|
void OpenLogFile(const char *filename);
|
|
void OpenLogFile(const wchar_t *filename);
|
|
|
|
wxString s;
|
|
OpenLogFile(s); // ERROR: ambiguity
|
|
OpenLogFile(s.c_str()); // ERROR: ambiguity
|
|
OpenLogFile(s.wx_str()); // OK: function called depends on the build
|
|
OpenLogFile(s.mb_str()); // OK: always calls narrow string overload
|
|
OpenLogFile(s.wc_str()); // OK: always calls wide string overload
|
|
@endcode
|
|
A common example of such problem arises with @c std::fstream class constructor
|
|
in Microsoft Visual C++ standard library implementation. In addition to a
|
|
constructor from @c const @c char * which this class must have, it also
|
|
provides a constructor taking a wide character file name. Because of this, code
|
|
like the following
|
|
@code
|
|
#include <fstream>
|
|
|
|
void MyFunc(const wxString& filename)
|
|
{
|
|
std::ifstream ifs(filename.c_str());
|
|
...
|
|
}
|
|
@endcode
|
|
does not compile when using Microsoft Visual C++ and needs to be changed to use
|
|
mb_str() (which will not work for file names containing Unicode characters,
|
|
consider using wxWidgets classes and functions to work with such file names as
|
|
they are not supported by standard C++ library).
|
|
|
|
The other class of incompatible changes is due to modifying some virtual
|
|
methods to use @c wxString parameters instead of @c const @c wxChar* ones to
|
|
make them accept both narrow and wide strings. This is not a problem if you
|
|
simply call these functions but you need to change the signature of the derived
|
|
class versions if you override them as otherwise they wouldn't be called any
|
|
more. Again, the best way to ensure that this problem doesn't arise is to
|
|
rebuild your code using a compiler which warns about function signature
|
|
mismatch (you can use @c -Woverloaded-virtual g++ option).
|
|
|
|
Finally, a few structure fields, notable @c wxCmdLineEntryDesc::shortName,
|
|
@c longName and @c description fields have been changed to be of type @c const
|
|
@c char* instead of @c const @c wxChar* so you will need to remove @c wxT() or
|
|
@c _T() if you used it with their initializers.
|
|
|
|
@section overview_changes_other Miscellaneous Other Changes
|
|
|
|
- Default location of wxFileConfig files has changed under Windows, you will
|
|
need to update your code if you access these files directly.
|
|
|
|
- wxWindow::IsEnabled() now returns false if a window parent (and not
|
|
necessarily the window itself) is disabled, new function IsThisEnabled()
|
|
with the same behaviour as old IsEnabled() was added.
|
|
|
|
- Generating wxNavigationKeyEvent events doesn't work any more under wxGTK (and
|
|
other platforms in the future), use wxWindow::Navigate() or NavigateIn()
|
|
instead.
|
|
|
|
- Sizers distribute only the extra space between the stretchable items
|
|
according to their proportions and not all available space. We believe the
|
|
new behaviour corresponds better to user expectations but if you did rely
|
|
on the old behaviour you will have to update your code to set the minimal
|
|
sizes of the sizer items to be in the same proportion as the items
|
|
proportions to return to the old behaviour.
|
|
|
|
- wxWindow::Freeze/Thaw() are not virtual any more, if you overrode them in
|
|
your code you need to override DoFreeze/Thaw() instead now.
|
|
|
|
- wxCalendarCtrl has native implementation in wxGTK, but it has less features
|
|
than the generic one. The native implementation is used by default, but you
|
|
can still use wxGenericCalendarCtrl instead of wxCalendarCtrl in your code if
|
|
you need the extra features.
|
|
|
|
- wxDocument::FileHistoryLoad() and wxFileHistory::Load() now take const
|
|
reference to wxConfigBase argument and not just a reference, please update
|
|
your code if you overrode these functions and change the functions in the
|
|
derived classes to use const reference as well.
|
|
|
|
- Calling wxConfig::Write() with an enum value will fail to compile because
|
|
wxConfig now tries to convert all unknown types to wxString automatically
|
|
using wxToString() function.
|
|
|
|
The simplest solution is to cast the enum value to int, e.g.
|
|
@code
|
|
enum Colour { Red, Green, Blue };
|
|
|
|
wxConfig conf;
|
|
conf.Write("MyFavouriteColour", Red); // ERROR: no match
|
|
conf.Write("MyFavouriteColour", int(Red)); // OK
|
|
@endcode
|
|
|
|
Another possibility which exists now is to provide an overload of
|
|
wxToString() (and wxFromString()) for your own type, e.g.
|
|
|
|
@code
|
|
wxString wxToString(Colour col)
|
|
{
|
|
return col == Red ? "R" : col == Green ? "G" : "B";
|
|
}
|
|
|
|
bool wxFromString(const wxString& s, Colour* col)
|
|
{
|
|
if ( s.length() != 1 )
|
|
return false;
|
|
|
|
switch ( s[0].GetValue() )
|
|
{
|
|
case 'R': *col = Red; return true;
|
|
case 'G': *col = Green; return true;
|
|
case 'B': *col = Blue; return true;
|
|
}
|
|
|
|
return false;
|
|
}
|
|
@endcode
|
|
|
|
Of course, this will change the format of the wxConfig output which may be
|
|
undesirable.
|
|
|
|
- wxTE_AUTO_SCROLL style is deprecated as it's always on by default anyhow in
|
|
the ports which support it so you should simply remove any mentions of it
|
|
from your code.
|
|
|
|
- If you use wxScrolled<T>::SetTargetWindow() you must override
|
|
wxScrolled<T>::GetSizeAvailableForScrollTarget() method to compute the size
|
|
available for the scroll target as function of the main window size, please
|
|
see the documentation of this method for more details.
|
|
|
|
- Signature of wxDataViewCustomRenderer::StartDrag() virtual method changed.
|
|
You will need to change it in your derived renderer class too if you override
|
|
it.
|
|
|
|
- wxDataViewCustomRenderer::Activate() and
|
|
wxDataViewCustomRenderer::LeftClick() were replaced with the new
|
|
wxDataViewCustomRenderer::ActivateCell() method. You will need to change it
|
|
in your derived renderer class accordingly.
|
|
|
|
*/
|
|
|