https://linen.dev logo
Join Discord
Powered by
# haxe-ui
  • f

    full-journalist-82607

    11/27/2022, 3:52 PM
    Ok, after looking at my code, I think it's kind of logical that it shouldn't work with the change.
  • f

    full-journalist-82607

    11/27/2022, 3:53 PM
    so just overriding onComponentAdded works, what would be the advantage to override the create function instead ?
  • b

    bright-gpu-74537

    11/27/2022, 3:54 PM
    did onComponentAdded already exist?
  • f

    full-journalist-82607

    11/27/2022, 3:54 PM
    Yes πŸ™‚
  • b

    bright-gpu-74537

    11/27/2022, 3:54 PM
    and no advantage, just didnt think onComponentAdded existed
  • b

    bright-gpu-74537

    11/27/2022, 3:54 PM
    oh, then great... ignore me totally πŸ™‚
  • f

    full-journalist-82607

    11/27/2022, 3:55 PM
    Haha
  • b

    bright-gpu-74537

    11/27/2022, 3:55 PM
    ... ... sometimes i feel like there are people here who know haxeui better than i do πŸ˜„
  • f

    full-journalist-82607

    11/27/2022, 4:02 PM
    Haha, only because I saw there was a bug, and explored the code. I didn't know it's existence before.
  • b

    bright-gpu-74537

    11/27/2022, 4:03 PM
    ... ... ... neither did i... πŸ˜… ... thanks "past Ian" πŸ˜„
  • f

    full-journalist-82607

    11/27/2022, 4:07 PM
    BTW, on github, I already made a pull request for another bug. Normally,if I want to make another pull request, does it mean I have to create another branch ? Do I have to create a branch for each pull request ?
  • b

    bright-gpu-74537

    11/27/2022, 4:08 PM
    each PR is generally a separate branch, yeah... But it doenst have to be, i mean, i guess its up to you... i normally delete my branches once a PR has been merged
  • b

    bright-gpu-74537

    11/27/2022, 4:08 PM
    ... ... whats the other PR? Was it merged?
  • f

    full-journalist-82607

    11/27/2022, 4:10 PM
    Okay, I'll do like this. There is https://github.com/haxeui/haxeui-core/pull/494 not merged yet
  • b

    bright-gpu-74537

    11/27/2022, 4:15 PM
    oh, i thought i merged this tbh, must have just read it and forgot to actually merge
  • b

    bright-gpu-74537

    11/27/2022, 4:15 PM
    (merged)
  • f

    full-journalist-82607

    11/27/2022, 4:18 PM
    πŸ™‚ I need to leave now . I'll make pull request tomorrow .
  • b

    bright-gpu-74537

    11/27/2022, 4:19 PM
    thanks for the effort πŸ™‚ - very much appreciated
  • f

    full-journalist-82607

    11/27/2022, 4:19 PM
    You're welcome ☺️
  • f

    famous-alarm-22563

    11/27/2022, 6:53 PM
    the flixel stuff is a bit annoying though.. if you are gonna implement ui elements do them properly or use an external lib
  • f

    famous-alarm-22563

    11/27/2022, 6:53 PM
    ... Or is this residue from copying the flash flixel engine over to haxe?
  • e

    elegant-twilight-61392

    11/27/2022, 6:54 PM
    huh?
  • e

    elegant-twilight-61392

    11/27/2022, 6:54 PM
    are you talking about haxeui-flixel or flixel-ui?
  • b

    bright-gpu-74537

    11/27/2022, 7:11 PM
    ... ? ... HaxeUI is an external library (to flixel)... are you sure you are venting in the right channel? You are, of course, free to write your own library that "does it properly"... any particular gripe? Or just having a shitty sunday?
  • f

    famous-alarm-22563

    11/27/2022, 7:39 PM
    Hey, thanks forr the rebuke πŸ™‚ My complaint was a bit rude... I was running into lots of little things that were a bit annoying (like the lack of a function that sets graphic and size for the FlxButton... etc) I switched over to haxeui for my needs atm... I'll probably sit down to fix up the ui bits of haxeflixel maybe next week πŸ™‚ For now.. I want to get my project done.
  • e

    elegant-twilight-61392

    11/27/2022, 7:42 PM
    FlxButton isnt part of haxeui, and is a very simple button. the size of the button is based on the size of the graphic you use (and then you can optionally scale it). its not like an html button where it will automatically grow based on its content
  • f

    famous-alarm-22563

    11/27/2022, 7:43 PM
    ^ My main thing was scaling the text.... since the scale var didn't have a setter. I'll get around to cleaning up that stuff
  • a

    ambitious-knife-25690

    11/27/2022, 7:52 PM
    i think you posted in the wrong channel dude
  • a

    ambitious-knife-25690

    11/27/2022, 7:53 PM
    were you meant to say this in #165234904815239168 in reply to my response to you?
  • a

    ambitious-knife-25690

    11/27/2022, 7:53 PM
    but for context the flixel ui stuff is a very old lib that was maintained by some user that was later abandoned
1...130113021303...1687Latest