Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed setting colours of wxPGProperty

Browse Source 
When wxPGProperty's text or background colours are modified with dedicated wxPropertyGridInterface utility functions (SetPropertyBackgroundColour, SetPropertyTextColour, SetPropertyColoursToDefault) then it is enough to redraw the property with new colours because its internal state remains unmodified and full refreshing is not necessary.

Closes #17588
This commit is contained in:
Artur Wieczorek
2016-07-08 19:44:32 +02:00
parent d2b3484e60
commit 4f1e3b7172
3 changed files with 83 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
interface/wx/propgrid/property.h
View File

@@ -1664,6 +1664,10 @@ public:
Default is wxPG_RECURSE which causes colour to be set recursively.
Omit this flag to only set colour for the property in question
and not any of its children.
@remarks
Unlike wxPropertyGridInterface::SetPropertyBackgroundColour(),
this does not automatically update the display.
*/
void SetBackgroundColour( const wxColour& colour,
int flags = wxPG_RECURSE );
@@ -1753,8 +1757,11 @@ public:
/**
Sets property's label.
@remarks Properties under same parent may have same labels. However,
property names must still remain unique.
@remarks
- Properties under same parent may have same labels. However,
property names must still remain unique.
- Unlike wxPropertyGridInterface::SetPropertyLabel(),
this does not automatically update the display.
*/
void SetLabel( const wxString& label );
@@ -1796,6 +1803,10 @@ public:
Default is wxPG_RECURSE which causes colour to be set recursively.
Omit this flag to only set colour for the property in question
and not any of its children.
@remarks
Unlike wxPropertyGridInterface::SetPropertyTextColour(),
this does not automatically update the display.
*/
void SetTextColour( const wxColour& colour,
int flags = wxPG_RECURSE );
@@ -1807,6 +1818,10 @@ public:
Default is wxPG_RECURSE which causes colours to be set recursively.
Omit this flag to only set colours for the property in question
and not any of its children.
@remarks
Unlike wxPropertyGridInterface::SetPropertyColoursToDefault(),
this does not automatically update the display.
*/
void SetDefaultColours(int flags = wxPG_RECURSE);

43
interface/wx/propgrid/propgridiface.h
View File

@@ -156,6 +156,9 @@ public:
/**
Disables a property.
@remarks
Property is refreshed with new settings.
@see EnableProperty(), wxPGProperty::Enable()
*/
bool DisableProperty( wxPGPropArg id );
@@ -176,6 +179,9 @@ public:
@param enable
If @false, property is disabled instead.
@remarks
Property is refreshed with new settings.
@see wxPGProperty::Enable()
*/
bool EnableProperty( wxPGPropArg id, bool enable = true );
@@ -603,6 +609,9 @@ public:
/**
Disables (limit = @true) or enables (limit = @false) wxTextCtrl editor
of a property, if it is not the sole mean to edit the value.
@remarks
Property is refreshed with new settings.
*/
void LimitPropertyEditing( wxPGPropArg id, bool limit = true );
@@ -768,8 +777,10 @@ public:
Optional.
Use wxPG_RECURSE to set the attribute to child properties recursively.
@remarks Setting attribute's value to wxNullVariant will simply remove it
from property's set of attributes.
@remarks
- Setting attribute's value to wxNullVariant will simply remove it
from property's set of attributes.
- Property is refreshed with new settings.
*/
void SetPropertyAttribute( wxPGPropArg id, const wxString& attrName,
wxVariant value, long argFlags = 0 );
@@ -778,11 +789,14 @@ public:
Sets property attribute for all applicapple properties.
Be sure to use this method only after all properties have been
added to the grid.
@remarks
Properties are refreshed with new settings.
*/
void SetPropertyAttributeAll( const wxString& attrName, wxVariant value );
/**
Sets background colour of a property.
Sets background colour of given property.
@param id
Property name or pointer.
@@ -794,6 +808,10 @@ public:
Default is wxPG_RECURSE which causes colour to be set recursively.
Omit this flag to only set colour for the property in question
and not any of its children.
@remarks
- If category is tried to set recursively, only its children are affected.
- Property is redrawn with new colour.
*/
void SetPropertyBackgroundColour( wxPGPropArg id,
const wxColour& colour,
@@ -823,12 +841,17 @@ public:
/**
Resets text and background colours of given property.
@param id
Property name or pointer.
@param flags
Default is wxPG_DONT_RECURSE which causes colour to be reset
only for the property in question (for backward compatibility).
@remarks
- If category is tried to set recursively, only its children are affected.
- Property is redrawn with new colours.
*/
void SetPropertyColoursToDefault(wxPGPropArg id, int flags = wxPG_DONT_RECURSE);
@@ -887,8 +910,10 @@ public:
By default changes are applied recursively. Set this parameter
to wxPG_DONT_RECURSE to prevent this.
@remarks This is mainly for use with textctrl editor. Only some other
editors fully support it.
@remarks
- This is mainly for use with textctrl editor. Only some other
editors fully support it.
- Property is refreshed with new settings.
*/
void SetPropertyReadOnly( wxPGPropArg id, bool set = true,
int flags = wxPG_RECURSE );
@@ -936,18 +961,22 @@ public:
/**
Sets text colour of a property.
Sets text colour of given property.
@param id
Property name or pointer.
@param colour
New background colour.
New text colour.
@param flags
Default is wxPG_RECURSE which causes colour to be set recursively.
Omit this flag to only set colour for the property in question
and not any of its children.
@remarks
- If category is tried to set recursively, only its children are affected.
- Property is redrawn with new colour.
*/
void SetPropertyTextColour( wxPGPropArg id,
const wxColour& colour,