Skip to content

Added Unit Attribute and Sample Usage#181

Merged
vanifatovvlad merged 9 commits into
codewriter-packages:mainfrom
Kinggrass:main
Mar 19, 2025
Merged

Added Unit Attribute and Sample Usage#181
vanifatovvlad merged 9 commits into
codewriter-packages:mainfrom
Kinggrass:main

Conversation

@Kinggrass
Copy link
Copy Markdown
Contributor

This PR adds a new UnitAttribute to Tri-Inspector along with sample usage demonstrating various unit types.

Changes:
Added: Unit(Drawer/Attribute) class for rendering unit suffix in the inspector.
Added: Decorators_UnitSample scriptable object with various unit types.
Updated: README.md with example usage of the new Unit attribute.
Reorder: Various attribute files to better organize attributes into appropriate categorized folders.

Impact:
Enhances the inspector with unit display capabilities.
Provides examples for using units in different contexts.

Testing:
Verified the UnitDrawer renders correctly with different units.
Confirmed the example in README.md works as expected.

Preview:
Unit

    [Unit("My custom Unit")]
    public float freeTextUnit;

    [Unit(UnitAttribute.Meter)]
    public float lengthInMeters;

    [Unit(UnitAttribute.Centimeter)]
    public float lengthInCentimeters;

- Introduced a new UnitAttribute for specifying measurement units.
- Created a UnitDrawer to display the unit next to input fields in the editor.
- Added sample script showcasing various units like meters, kilograms, and seconds.
Because even though I want to contribute, I don't have permission to advertise sponsoring on the repository of the maintainer for myself.
I thought at first I could keep this exclusive on my fork.
@vanifatovvlad vanifatovvlad merged commit 6c041ac into codewriter-packages:main Mar 19, 2025
@retroluxfilm
Copy link
Copy Markdown

retroluxfilm commented Jul 10, 2025

Great contribution. Hope to see a new version soon to use this feature. @vanifatovvlad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants