Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP initial attempt at tag userdata, does not work #2265

Closed
wants to merge 1 commit into from

Conversation

sparr
Copy link
Contributor

@sparr sparr commented Feb 14, 2020

My goal here is to add userdata functionality to tags, as currently exists for cels, layers, and slices. The biggest hiccup was that the tag properties window isn't as fancifully defined as the others, but I think I got over that. Now everything works fine up until you actually change the userdata on a tag, at which point the change doesn't save. I would appreciate any insight on what bit of functionality I might have missed implementing.

FrameTagPropertiesCommand::onExecute returns unexpectedly. Attempts to debug with gdb have met with limited success and much confusion.
@sparr sparr requested a review from dacap as a code owner February 14, 2020 05:03
@dacap
Copy link
Member

dacap commented Feb 14, 2020

Hi @sparr, please check the CLA signing process.

@dacap
Copy link
Member

dacap commented May 25, 2020

I will try to review this PR soon and try to integrated it into master if it's possible and with a patch to the issues you're saying.

@jestarray
Copy link
Contributor

dose this also address the problems with tags and scripting as mentioned here: #2487 ?

@dacap
Copy link
Member

dacap commented Aug 19, 2020

About this PR we will merge some of these changes + fixes for the future release v1.3. Thanks @sparr

@dacap dacap closed this Aug 19, 2020
@sparr
Copy link
Contributor Author

sparr commented Aug 19, 2020

You're welcome. Thanks for taking a look. I am glad at least some of this will be of use. I wish I had been able to see this through to completion.

@dacap
Copy link
Member

dacap commented May 3, 2022

I'm reviewing some old PRs, this was finally merged with 55a37d0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants