Uno.Themes
Uno.Themes copied to clipboard
Include word is used Two times.
On which page?
https://platform.uno/docs/articles/external/uno.themes/doc/themes-overview.html
What's wrong?
The "Include" word is used Twice
Any feedback?
I think that changing the line to "Both libraries help you style your application with just a few lines of code, including:" will make more sense
Hi! I might be able to find where the double word is coming from and delete it. I can take a look. I am a first-timer contributor still learning.
Hi! I might be able to find where the double word is coming from and delete it. I can take a look. I am a first-timer contributor still learning.
Hi @sfazli96 welcome! We will happily accept your contribution to this issue. Here is our Contribution Guide for all the details. As it is detailed in it, your commit must be in the Conventional Commits format please.
Here is the updated sentence that we should have in this case instead:
Both libraries help you style your application with a few lines of code, including:
And here is the current related code: https://github.com/unoplatform/Uno.Themes/blob/52d545d39ed086c10151fd80c95518a776171abb/doc/themes-overview.md?plain=1#L21
Please let us know if you have any additional questions or blockers and we will be happy to help you!
Ok I made a pull request and fixed the wording.
Ok I made a pull request and fixed the wording.
Thank you a lot for your PR @sfazli96, it is really appreciated! I left you a comment https://github.com/unoplatform/Uno.Themes/pull/1279#issuecomment-1802713245
yeah I saw the comment and updated the title.
CC @agneszitte, @Darsh0307 Closing this issue as it was fixed by https://github.com/unoplatform/Uno.Themes/pull/1308