ForsakenGod Posted January 27, 2010 Share Posted January 27, 2010 Hello , I ve created a script that previews a color into an text label thrught Guictrlsetbkcolor but i have a problem this is in the 1st while 1 loop While 1 Case $OldSelect <> GUICtrlRead($colorinp) Farba() Wend Function Farba is following Func Farba() Global $color = GuiCtrlRead($colorinp) GUICtrlSetBkColor($preview, $color) Global $color2 = GuiCtrlRead($colorinp2) GUICtrlSetBkColor($preview2, $color2) Global $OldSelect = GUICtrlRead($colorinp) EndFunc The problem is that the color is still refreshing which makes it a bit ugly Actually it looks like it ignores my $OldSelect <> GUICtrlRead($colorinp) And i dont know why Does any one have idea why the color in my text labels is still looping even when my Case is does not occour ? (changinf the value of inputbox) Thanks in advance Unreal Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted January 27, 2010 Moderators Share Posted January 27, 2010 ForsakenGod,You do not show how you trigger the Case, so it might be that you just never test it. Try using an If statement rather than a Case:#include <GUIConstantsEx.au3> Global $OldSelect = "", $color $hGUI = GUICreate("Test", 500, 500) $colorinp = GUICtrlCreateInput("", 10, 10, 200, 20) $preview = GUICtrlCreateLabel("", 0, 100, 500, 200) GUISetState() While 1 Switch GUIGetMsg() Case $GUI_EVENT_CLOSE Exit EndSwitch If $OldSelect <> GUICtrlRead($colorinp) Then $OldSelect = GUICtrlRead($colorinp) Farba() EndIf WEnd Func Farba() $color = GuiCtrlRead($colorinp) GUICtrlSetBkColor($preview, $color) EndFuncDoes that help?M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
ForsakenGod Posted January 27, 2010 Author Share Posted January 27, 2010 ForsakenGod, You do not show how you trigger the Case, so it might be that you just never test it. Try using an If statement rather than a Case: #include <GUIConstantsEx.au3> Global $OldSelect = "", $color $hGUI = GUICreate("Test", 500, 500) $colorinp = GUICtrlCreateInput("", 10, 10, 200, 20) $preview = GUICtrlCreateLabel("", 0, 100, 500, 200) GUISetState() While 1 Switch GUIGetMsg() Case $GUI_EVENT_CLOSE Exit EndSwitch If $OldSelect <> GUICtrlRead($colorinp) Then $OldSelect = GUICtrlRead($colorinp) Farba() EndIf WEnd Func Farba() $color = GuiCtrlRead($colorinp) GUICtrlSetBkColor($preview, $color) EndFunc Does that help? M23 Ohh thanks It definelly showed me back my way thanks Link to comment Share on other sites More sharing options...
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now