miketheprogrammer / go-thrust

Cross Platform UI Kit powered by Blink/V8/Chromium Content Lib
MIT License
445 stars 34 forks source link

Resize not work #43

Closed Nyarum closed 9 years ago

Nyarum commented 9 years ago

Any call to this function - changes the size of the application on the ~ 200 x 1500.

OS: Windows 8.1 Compile Go version: 1.4

miketheprogrammer commented 9 years ago

@spolu Can you look into this in thrust core.

kdar commented 9 years ago

This is an issue with go-thrust.

Changing

func (w *Window) Resize(width, height uint) {
    command := Command{
        Method: "resize",
        Args: CommandArguments{
            Size: SizeHW{
                Width:  width,
                Height: height,
            },
        },
    }

    w.CallWhenReady(&command)
}

to

func (w *Window) Resize(width, height uint) {
    command := Command{
        Method: "resize",
        Args: CommandArguments{
            Width:  width,
            Height: height,
        },
    }

    w.CallWhenReady(&command)
}

at https://github.com/miketheprogrammer/go-thrust/blob/master/lib/bindings/window/window.go#L314 fixes it.

Nyarum commented 9 years ago

Good had worked! Pull. r.?

miketheprogrammer commented 9 years ago

I'll fix today thanks

miketheprogrammer commented 9 years ago

Fixed