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

Return 0 for wxSYS_CARET_ON_MSEC in wxGTK if appropriate too

Browse Source 
For some reason, 7f10d1fa8a handled
wxSYS_CARET_ON_MSEC and wxSYS_CARET_OFF_MSEC differently in wxGTK, even
though it looks like they should be handled in exactly the same way.

Change the code to do this, which results in returning 0 for the former
setting if the caret doesn't blink at all, just as was already the case
for the latter one.

See https://github.com/wxWidgets/wxWidgets/pull/1285

See #17629.
This commit is contained in:
Vadim Zeitlin
2019-04-18 18:01:47 +02:00
parent e5d850d76c
commit c65647130b
2 changed files with 6 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
interface/wx/settings.h
View File

@@ -196,8 +196,10 @@ enum wxSystemMetric
/**
Time, in milliseconds, for how long a blinking caret should
stay visible during a single blink cycle before it disappears.
If this value is negative, the platform does not support the
user setting.
If this value is zero, caret should be visible all the time
instead of blinking. If the value is negative, the platform
does not support the user setting.
@since 3.1.1
*/
@@ -206,7 +208,8 @@ enum wxSystemMetric
/**
Time, in milliseconds, for how long a blinking caret should
stay invisible during a single blink cycle before it reappears.
If this value is zero, carets should be visible all the time
If this value is zero, caret should be visible all the time
instead of blinking. If the value is negative, the platform
does not support the user setting.

10
src/gtk/settings.cpp
View File

@@ -975,16 +975,6 @@ int wxSystemSettingsNative::GetMetric( wxSystemMetric index, wxWindow* win )
return dclick;
case wxSYS_CARET_ON_MSEC:
{
gint blink_time = -1;
g_object_get(GetSettingsForWindowScreen(window),
"gtk-cursor-blink-time", &blink_time, NULL);
if (blink_time > 0)
return blink_time / 2;
return -1;
}
case wxSYS_CARET_OFF_MSEC:
{
gboolean should_blink = true;