Jump to content
Sign in to follow this  
NewVersionTester

"Bug"-Report for AutoIt3Wrapper: GUI-Design

Recommended Posts

NewVersionTester

Versions: AutoIt3Wrapper v.2.1.4.0 SciTE v.3.3.7.0

I would change this:

Local $H_INP_ICON = GUICtrlCreateEdit($INP_Icon, 100, 205, 440, 20)

-->

Local $H_INP_ICON = GUICtrlCreateINPUT($INP_Icon, 100, 205, 440, 20)

This way you have no useless scrollbar.

I hope this small bug-report was helpful.


P.S.: I also found this lines quite useless, but this is only my opinion:

If @error = 1 Then
        $rc = MsgBox(4100, "AutoIt3 Compile", "do you want to stop the process?")
        If $rc = 6 Then Exit
    EndIf

I think, if somebody clicks on "cancel" he also wants to "cancel" and stop the process.

Edited by NewVersionTester

Debugging is fun, because it always begins with a mystery.

Share this post


Link to post
Share on other sites
Jos

Versions: AutoIt3Wrapper v.2.1.4.0 SciTE v.3.3.7.0

I would change this:

Local $H_INP_ICON = GUICtrlCreateEdit($INP_Icon, 100, 205, 440, 20)

-->

Local $H_INP_ICON = GUICtrlCreateINPUT($INP_Icon, 100, 205, 440, 20)

This way you have no useless scrollbar.

I hope this small bug-report was helpful.

 

I've used an Edit control to be able to handle long paths.

Jos


SciTE4AutoIt3 Full installer Download page   - Beta files       Read before posting     How to post scriptsource        Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites
NewVersionTester

Thanks for your answer.

But the problem is that you cannot enter the path to the icon manually. (tested on windows xp SP3)

And why should the paths for the icon such long and the others not?

Edited by NewVersionTester

Debugging is fun, because it always begins with a mystery.

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  

×