Skip to content

Conversation

@dannymcgee
Copy link
Contributor

@dannymcgee dannymcgee commented Feb 29, 2024

This PR cherry-picks some improvements from #63 that are general-purpose and not specifically related to IK.

  • Some logic has been added to AppElement to track the currently selected Scene Element
  • disabled property has been added to ButtonElement, with custom styling and accessibility features
  • Tagging entities with their SceneElement type has been made more robust and reliable (previously, it was possible for all entities to appear in the hierarchy as "Generic" elements depending on the order of system execution)

Additionally:

  • Forces the Netlify CI pipeline to always used the latest stable Rust version

@netlify
Copy link

netlify bot commented Feb 29, 2024

Deploy Preview for storyteller-studio ready!

Name Link
🔨 Latest commit 044a6ef
🔍 Latest deploy log https://app.netlify.com/sites/storyteller-studio/deploys/65e0e912985d100008bc9b44
😎 Deploy Preview https://deploy-preview-64--storyteller-studio.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Feb 29, 2024

Deploy Preview for storyteller-live-demo ready!

Name Link
🔨 Latest commit 044a6ef
🔍 Latest deploy log https://app.netlify.com/sites/storyteller-live-demo/deploys/65e0e9121970be0008ba0d49
😎 Deploy Preview https://deploy-preview-64--storyteller-live-demo.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dannymcgee dannymcgee marked this pull request as ready for review February 29, 2024 20:12
Copy link
Contributor

@MrVintage710 MrVintage710 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is short and sweet! Good improvements to have. LGTM!

@dannymcgee dannymcgee merged commit 3413810 into main Feb 29, 2024
@dannymcgee dannymcgee deleted the misc-updates branch February 29, 2024 20:42
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.

2 participants