https://linen.dev logo
Powered by
# haxe-ui
  • m
    Will do!
  • b
    Could not find haxelib "simple", does it need to be installed
  • b
    (and there is no haxelib called simple i could find)
  • m
    Dammit, I keep forgetting my dependencies
  • m
    It's just a couple of files really
  • m
    Just a sec
  • m
    no more shady dependencies 🙂
  • a
    there is, I've had similar confusion before
  • a
    lime depends on it
  • b
    oh right, fair enough
  • m
    Wait guys, "simple" lib in this case is my personal game framework built on top of openfl https://github.com/guifes/simple
  • a
    oh lol
  • b
    gotcha
  • m
    It's just not published to haxelib though, because it's not complete 🙂
  • m
    btw there's a couple of color math bugs related to brightness and saturation :P, I'll be looking into it later today
  • b
    i think this is your issue tbh, i mean, you are setting different components to different values... i think this is why you are getting loops
  • b
    like i dont really get all the / 100, * 100 stuff
  • m
    That is probably not needed. I could just change the scale of the sliders
  • b
    yeah, i did that also
  • m
    I just did the conversion on code and left the sliders all from 0 to 100
  • b
    Copy code
    haxe
    private function syncUI() {
        redTextfield.text = Std.string(_pickedColor.red);
        redSlider.pos = _pickedColor.red;

        greenTextfield.text = Std.string(_pickedColor.green);
        greenSlider.pos = _pickedColor.green;
        
        blueTextfield.text = Std.string(_pickedColor.blue);
        blueSlider.pos = _pickedColor.blue;

        updatePickedColorDisplay();
    }
    
    @:bind(redTextfield, UIEvent.CHANGE)
    private function onRedTextField(_) {
        redSlider.pos = Std.parseInt(redTextfield.text);
    }
    
    @:bind(redSlider, UIEvent.CHANGE)
    private function onRedSlider(_) {
        _pickedColor.red = Std.int(redSlider.pos);
        syncUI();
    }   
    
    @:bind(greenTextfield, UIEvent.CHANGE)
    private function onGreenTextField(_) {
        greenSlider.pos = Std.parseInt(greenTextfield.text);
    }
    
    @:bind(greenSlider, UIEvent.CHANGE)
    private function onGreenSlider(_) {
        _pickedColor.green = Std.int(greenSlider.pos);
        syncUI();
    }   
    
    @:bind(blueTextfield, UIEvent.CHANGE)
    private function onBlueTextField(_) {
        blueSlider.pos = Std.parseInt(blueTextfield.text);
    }
    
    @:bind(blueSlider, UIEvent.CHANGE)
    private function onBlueSlider(_) {
        _pickedColor.blue = Std.int(blueSlider.pos);
        syncUI();
    }
  • m
    Let me try and refactor the code to this approach, maybe my dumb way of doing is adding problems all over
  • b
    yeah, i think the main issue is your maths is making the values different
  • b
    its like trying to sync the colours except the change handler is actually ending up with a different value
  • m
    I see, the value being the same, makes the onChange not trigger twice, is that so?
  • b
    if the value is the same, there will be no trigger
  • m
    oh create a loop I mean
  • m
    If I code around that, I can make everything simpler... Just need to make sure I don't get rounding difference
  • m
    That will trigger infinite loop when updating the ui
  • b
    Copy code
    haxe
mySlider.pos = 10; // trigger (assuming its 0 to start with)
mySlider.pos = 20; // trigger
mySlider.pos = 20; // will not trigger
mySlider.pos = 20; // will not trigger
mySlider.pos = 20; // will not trigger
mySlider.pos = 20; // will not trigger
mySlider.pos = 20; // will not trigger
mySlider.pos = 20; // will not trigger
1...109410951096...1687Latest