Updates to the dynamic UI system after code review.
Refactor some actor and component names.
Author checklist
Tutorial for #7
Has the new tutorial been integrated into the main level? Yes
Has a new section of the readme been added for this tutorial? Yes
Have you done a full self-review of the code and blueprints? Yes
Have you tagged a primary reviewer? Yes
Are there any remaining tasks to do or blocking questions to answer before we can merge this? No
Testing plan
Verify that the line trace for the metadata component shows (can remove the teleport component to check). This should be blue if trace is successful, and orange if it fails.
Reminders for reviewers
Thank you for taking the time to review this PR. By approving a PR you are taking as much responsibility for these changes as the author. Please keep these points in mind throughout the review process:
Review and run all parts of the test plan on this branch and verify it matches expectations.
Review the code and make sure you do not have any remaining questions or concerns. You should understand why this code change fixes the issue and agree that it's the best approach to doing so.
Review the automated tests and make sure there are no missing tests or edge cases.
When you feel this is ready bump it to a second reviewer or merge.
@xuelongmu Looks great! I tested it and it's working as expected. Yellow makes sense for the sake of contrast as well and as an action color, aligns with teleportation as a heavier action seldom taken.
Summary
Updates to the dynamic UI system after code review.
Refactor some actor and component names.
Author checklist
Testing plan
Reminders for reviewers
Thank you for taking the time to review this PR. By approving a PR you are taking as much responsibility for these changes as the author. Please keep these points in mind throughout the review process: