Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autodesk: Android Proposal #17

Merged

Conversation

erikaharrison-adsk
Copy link
Contributor

Description of Proposal

This is a proposal on supporting Android within OpenUSD.

Supporting Materials

N/A

Contributing

Copy link
Member

@meshula meshula left a comment

Choose a reason for hiding this comment

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

I have a few questions ~ I was wondering if perhaps the PR could be updated to make those aspects more clear?

proposals/android/README.md Show resolved Hide resolved
proposals/android/README.md Show resolved Hide resolved
proposals/android/README.md Show resolved Hide resolved
proposals/android/README.md Outdated Show resolved Hide resolved
proposals/android/README.md Outdated Show resolved Hide resolved
@erikaharrison-adsk
Copy link
Contributor Author

Changes are reflected in the proposal. @meshula Please advise what other questions you have.

@erikaharrison-adsk
Copy link
Contributor Author

@meshula - any update on this? Thanks

@meshula
Copy link
Member

meshula commented Feb 8, 2024

Hi Erika, the changes look good. Unfortunately one of the updated commits didn't have signing, so merging is blocked. Would it be possible to rewrite the commits with signing, and then we can land it?

@erikaharrison-adsk
Copy link
Contributor Author

Updated to verified commits.

@meshula meshula merged commit 3da51a0 into PixarAnimationStudios:main Feb 9, 2024
erikaharrison-adsk pushed a commit to autodesk-forks/USD-proposals that referenced this pull request Apr 30, 2024
)

Update the proposal according to the comments.

We will focus on the schema in the proposal, not the implementation. So I removed the part about the Rprims, and also the plugins.
Add more detail about the SimpleText and MarkupText schemas. Add how the extents should be implemented.
Add the language tag for the textData in SimpleText and markup in MarkupText, and refer to the language proposal.
Add the "plain" tag for markup which is an alternate string when the markup can not be identified.
Remove the renderer property in SimpleText and MarkupText.
Change the name of the properties in TextStyle. Add altTypeface property which is used when the main typeface can not support some characters.
Add details about the font weight.
The TextLayout and TextLayoutAPI are merged into one schema. The new TextLayoutAPI not only contains the character directions, but also contains the direction how the lines are stacked.
Modify the name of the properties in ColumnStyle. Remove the direction and linesFlowDirection property in ColumnStyle. Instead, you can apply a TextLayoutAPI to a ColumnStyle to specify the directions.
In the part of "Some implementation details", discuss the extrusion, the missing characters, the layout for complex script and the billboard.
Finally there is a big part of four examples.

<!--
Put screenshots or other summary information here
-->

<!--
Please review the  [Contributing](https://graphics.pixar.com/usd/release/contributing_to_usd.html) page in the
documentation for the Supplemental Terms that apply to this repository.
Place an X in the box when you have reviewed and agree to the Supplemental Terms.
-->
- [ ] I agree to and accept the [Supplemental Terms](https://graphics.pixar.com/usd/release/contributing_supplemental.html).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Draft
Development

Successfully merging this pull request may close these issues.

4 participants