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

[BUG] Fix prop size to Text component #590

Closed
oharaju opened this issue Dec 28, 2022 · 3 comments
Closed

[BUG] Fix prop size to Text component #590

oharaju opened this issue Dec 28, 2022 · 3 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@oharaju
Copy link
Contributor

oharaju commented Dec 28, 2022

Hey Guys!

Using Yoga in my personal project, I noticed a problem regarding the <Text> component.

In the documentation of the Text props I verified that to change the font-size it is necessary to use the size, however when declaring it I verified that the prop does not work very well.
image

The Text component has compatibility with the Yoga System as described in the documentation "Text component also has support for system props as you can see here", this means that, using the typography resources, it enters into duplicity with regard to the properties to change the font-size.

Props:
size - only available in the component;
fontSize - available on the Yoga System.

When testing using fontSize instead of size it worked perfectly, below is evidence:
image

Conclusion:

The size property has been deprecated by Yoga System (typography), properties but the documentation has not been updated. Just as the variant prop is deprecated by color, the size prop should also be deprecated by fontSize.

Proposal:

  • Update of usage examples;
  • Add the prop size as deprecated (example variant);
    image
  • Change in properties documentation.
    image
@oharaju oharaju added the bug Something isn't working label Dec 28, 2022
@caiotracera caiotracera added the good first issue Good for newcomers label Dec 29, 2022
@caiotracera
Copy link
Contributor

Hey, @oharaju! How are you?
Firstly, congratulations on opening your first issue on Yoga! It's very good to know that you are using our design system in your personal projects and actively contributing with the project.

Since you already came with a solution, are you interested in working on this issue?

Cheers,
Caio.

@oharaju
Copy link
Contributor Author

oharaju commented Dec 29, 2022

Hi @caiotracera! I'm fine and you?

Yes, I'm interested. I will make the changes and soon I will open the pull request.

@caiotracera
Copy link
Contributor

@oharaju everything is fine 😁

Okay, so we will be waiting for your pull request. But don't need to rush, the next deploy will probably occur only next week.
Feel free to reach us if you need any help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants