HISE Logo Forum
    • Categories
    • Register
    • Login

    Please do a pass of all LAF function objects and add the id property

    Scheduled Pinned Locked Moved Bug Reports
    1 Posts 1 Posters 72 Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • A
      aaronventure
      last edited by

      @Christoph-Hart can you please do a pass and make sure all LAF function objects have the id property? I already posted an issue on GitHub about the analyzer functions not having it, and now it seems that combobox doesn't have it either.

      This seems to be the only way to get a reference to the component actually triggering the function. It removes the need to hardcode references.

      For the combobox specifically, it prevents the relative usage of its actual text property (or any properties outside of what's included in obj, for that matter), which is different from the combobox items. Its obj.text incorrectly refers to the currently selected item. I imagine this is how it's been forever and how it'll stay for backwards compatibility.

      But I could just pull in the text property using get... if I had the id.

      1 Reply Last reply Reply Quote 2
      • First post
        Last post

      29

      Online

      1.8k

      Users

      12.0k

      Topics

      104.5k

      Posts