Sign in to follow this  
Followers 0
livewire

Strange GUICtrlSetBkColor behavior

5 posts in this topic

Running this script and clicking the button twice continuouly fires button click events.

Tried with Autoit3 versions 3.2.8.1 and 3.2.9.0

Windows XP SP2

#include <GUIConstants.au3>

Global $count = 0

GUICreate("My GUI Button")
$Button_1 = GUICtrlCreateButton("Click", 10, 20, 150)
$Input_1 = GUICtrlCreateInput("", 10,50, 150)

GUISetState()      ; will display an  dialog box with 2 button
While 1
    $msg = GUIGetMsg()
    Select
        Case $msg = $GUI_EVENT_CLOSE
            ExitLoop
        Case $msg = $Button_1
            $count += 1
            GUICtrlSetData($Input_1,"Button clicks: " & $count)
            GUICtrlSetBkColor($Button_1,0x00FF00)
    EndSelect
Wend

-Livewire

Share this post


Link to post
Share on other sites



#2 ·  Posted (edited)

Code looks OK.

I can confirm the same behaviour on my WIN98SE

with 3.2.8.0 and also 3.2.9.0

Probably some bug.

EDIT:

- source of problem is GUICtrlSetBkColor(), when you comment it no bad behaviour

- GUICtrlSetBkColor() return always 1 (OK) and @error = 0 (OK)

Here is improved testing script:

#include <GUIConstants.au3>

Global $count = 0

GUICreate("My GUI Button")
$Button_1 = GUICtrlCreateButton("Click", 10, 20, 150)
$Button_2 = GUICtrlCreateButton("Click2", 200, 20, 150)
$Input_1 = GUICtrlCreateInput("", 10,50, 150)
$Input_2 = GUICtrlCreateInput("", 200,50, 150)

GUISetState()      ; will display an  dialog box with 2 button
While 1
    $msg = GUIGetMsg()
    Select
        Case $msg = $GUI_EVENT_CLOSE
            ExitLoop
        Case $msg = $Button_1
            $count += 1
            GUICtrlSetData($Input_1,"Button clicks: " & $count)
            $ret = GUICtrlSetBkColor($Button_1,0x00FF00)
            GUICtrlSetData($Input_2,"@error - return: " & @error & ' ' & $ret)
        Case $msg = $Button_2
            GUICtrlSetData($Input_1,"")
            GUICtrlSetData($Input_2,"")
    EndSelect
Wend

- Button2 stops autolooping

- with GUICtrlSetData($Input_2,"@error - return: " & @error & ' ' & $ret) autolooping starts after first click

Edited by Zedna

Share this post


Link to post
Share on other sites

And as "ugly" workaround...

#include <GUIConstants.au3>

Global $count = 0, $lastcount = 1

GUICreate("My GUI Button")
$Button_1 = GUICtrlCreateButton("Click", 10, 20, 150)
$Input_1 = GUICtrlCreateInput("", 10,50, 150)

GUISetState()      ; will display an  dialog box with 2 button
While 1
    $msg = GUIGetMsg()
    $CurInfo = GUIGetCursorInfo()
    
    Select
        Case $msg = $GUI_EVENT_CLOSE
            ExitLoop
        Case $msg = $Button_1 Or ($CurInfo[2] = 1 And $CurInfo[4] = $Button_1 And $lastcount <> $count)
            $count += 1
            If $CurInfo[2] = 1 And $CurInfo[4] = $Button_1 Then
                $lastcount = $count
            Else
                $lastcount = 1
            EndIf
            
            GUICtrlSetData($Input_1,"Button clicks: " & $count)
            GUICtrlSetBkColor($Button_1,0x00FF00)
    EndSelect
Wend

:)

I think the problem is in the colored buttons, because with other controls this is not happened.


 

Spoiler

Using OS: Win 7 Professional, Using AutoIt Ver(s): 3.3.6.1 / 3.3.8.1

AutoIt_Rus_Community.png AutoIt Russian Community

My Work...

Spoiler

AutoIt_Icon_small.pngProjects: ATT - Application Translate Tool {new}| BlockIt - Block files & folders {new}| SIP - Selected Image Preview {new}| SISCABMAN - SciTE Abbreviations Manager {new}| AutoIt Path Switcher | AutoIt Menu for Opera! | YouTube Download Center! | Desktop Icons Restorator | Math Tasks | KeyBoard & Mouse Cleaner | CaptureIt - Capture Images Utility | CheckFileSize Program

AutoIt_Icon_small.pngUDFs: OnAutoItErrorRegister - Handle AutoIt critical errors {new}| AutoIt Syntax Highlight {new}| Opera Library! | Winamp Library | GetFolderToMenu | Custom_InputBox()! | _FileRun UDF | _CheckInput() UDF | _GUIInputSetOnlyNumbers() UDF | _FileGetValidName() UDF | _GUICtrlCreateRadioCBox UDF | _GuiCreateGrid() | _PathSplitByRegExp() | _GUICtrlListView_MoveItems - UDF | GUICtrlSetOnHover_UDF! | _ControlTab UDF! | _MouseSetOnEvent() UDF! | _ProcessListEx - UDF | GUICtrl_SetResizing - UDF! | Mod. for _IniString UDFs | _StringStripChars UDF | _ColorIsDarkShade UDF | _ColorConvertValue UDF | _GUICtrlTab_CoverBackground | CUI_App_UDF | _IncludeScripts UDF | _AutoIt3ExecuteCode | _DragList UDF | Mod. for _ListView_Progress | _ListView_SysLink | _GenerateRandomNumbers | _BlockInputEx | _IsPressedEx | OnAutoItExit Handler | _GUICtrlCreateTFLabel UDF | WinControlSetEvent UDF | Mod. for _DirGetSizeEx UDF
 
AutoIt_Icon_small.pngExamples: 
ScreenSaver Demo - Matrix included | Gui Drag Without pause the script | _WinAttach()! | Turn Off/On Monitor | ComboBox Handler Example | Mod. for "Thinking Box" | Cool "About" Box | TasksBar Imitation Demo

Like the Projects/UDFs/Examples? Please rate the topic (up-right corner of the post header: Rating AutoIt_Rating.gif)

* === My topics === *

==================================================
My_Userbar.gif
==================================================

 

 

 

AutoIt is simple, subtle, elegant. © AutoIt Team

Share this post


Link to post
Share on other sites

And as "ugly" workaround...

#include <GUIConstants.au3>

Global $count = 0, $lastcount = 1

GUICreate("My GUI Button")
$Button_1 = GUICtrlCreateButton("Click", 10, 20, 150)
$Input_1 = GUICtrlCreateInput("", 10,50, 150)

GUISetState()      ; will display an  dialog box with 2 button
While 1
    $msg = GUIGetMsg()
    $CurInfo = GUIGetCursorInfo()
    
    Select
        Case $msg = $GUI_EVENT_CLOSE
            ExitLoop
        Case $msg = $Button_1 Or ($CurInfo[2] = 1 And $CurInfo[4] = $Button_1 And $lastcount <> $count)
            $count += 1
            If $CurInfo[2] = 1 And $CurInfo[4] = $Button_1 Then
                $lastcount = $count
            Else
                $lastcount = 1
            EndIf
            
            GUICtrlSetData($Input_1,"Button clicks: " & $count)
            GUICtrlSetBkColor($Button_1,0x00FF00)
    EndSelect
Wend

:)

I think the problem is in the colored buttons, because with other controls this is not happened.

Presumably the setbkcolor is generating button messages, so another way to fix it would be to clear the message queue immediately after.

GUICtrlSetBkColor($Button_1,0x00FF00)

While GUIGetMsg() ;clear message queue

WEnd


Serial port communications UDF Includes functions for binary transmission and reception.printing UDF Useful for graphs, forms, labels, reports etc.Add User Call Tips to SciTE for functions in UDFs not included with AutoIt and for your own scripts.Functions with parameters in OnEvent mode and for Hot Keys One function replaces GuiSetOnEvent, GuiCtrlSetOnEvent and HotKeySet.UDF IsConnected2 for notification of status of connected state of many urls or IPs, without slowing the script.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0