gz-sim icon indicating copy to clipboard operation
gz-sim copied to clipboard

Adds a "Attach Plugin" context menu

Open arjo129 opened this issue 10 months ago β€’ 5 comments

πŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”Έ

πŸŽ‰ New feature

Closes #<NUMBER>

Summary

Adds a ui to attach a plugin from the Qt GUI itself to an object. add_plugin_ux

Test it

Right click on any entity, click "Attach Plugin" you should see the UI and plugin.

Checklist

  • [ ] Signed all commits for DCO
  • [ ] Added tests
  • [ ] Added example and/or tutorial
  • [ ] Updated documentation (as needed)
  • [ ] Updated migration guide (as needed)
  • [ ] Consider updating Python bindings (if the library has them)
  • [ ] codecheck passed (See contributing)
  • [ ] All tests passed (See test coverage)
  • [ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • [ ] Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Generated-by: Remove this if GenAI was not used.

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

πŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”ΈπŸ”Έ

arjo129 avatar Jun 18 '25 09:06 arjo129

note the component inspector also has a + on the top right corner that pops up a dialog for adding a plugin to an entity. It would be nice to consolidate / reuse the code if possible.

iche033 avatar Jun 20 '25 17:06 iche033

Oh wow I was not aware of this, will look into how we canreuse the code across both the "+" is incredibly small on my gazebosim which is probably why I missed it. Also the fact that its in the component inspector and not the component inspector editor where I'd expect it to be.

arjo129 avatar Jun 20 '25 19:06 arjo129

It would be great to get this in since it's a lot more discoverable than the one in the component inspector, but as @iche033 said, is there a way we could reuse the existing code?

azeey avatar Aug 18 '25 19:08 azeey

I don't think this will make jetty.

arjo129 avatar Aug 24 '25 08:08 arjo129

@arjo129, okay. thanks for the update.

azeey avatar Aug 25 '25 14:08 azeey