icub-tech-iit / creo2urdf

Generate URDF models from CREO mechanisms
https://icub-tech-iit.github.io/creo2urdf/
BSD 3-Clause "New" or "Revised" License
26 stars 3 forks source link

Rename ribbon from "iCubTech" to creo2urdf ? #101

Closed traversaro closed 1 day ago

traversaro commented 1 month ago

While working with @DavideBarbone, I realized it is a bit confusing for non-IIT users for creo2urdf to understand why they need to navigate to the "iCubTech" ribbon to actually find the "creo2urdf" button. Could it make sense to rename the "iCubTech" ribbon to creo2urdf ?

Nicogene commented 1 month ago

You are right, maybe it is better to place it in the Tools ribbon as the Mathworks one

mfussi66 commented 2 weeks ago

I created a new ribbon file that addresses the issue. However, it seems impossible to "inject" the creo2urdf button in the Mathworks group, called TOOLKIT, so I did the following:

immagine

Let me know if it's ok.

The validation button can be removed as it's not used at the moment.

traversaro commented 2 weeks ago

I like it! The only minor comment is that URDF is an acronym, so in general it does not make a lot of sense to capitalize it, so I would either write URDF or urdf, while Urdf looks a bit strange (see for example how UDF Library is capitalized in the Utilities Menu).

mfussi66 commented 2 weeks ago

Coming right up:

immagine

Nicogene commented 1 day ago

Fixed by #108