-
-
Notifications
You must be signed in to change notification settings - Fork 310
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
docs: improve texture compression doc #2231
Conversation
WalkthroughThe documentation updates focus on detailing the generation and handling of KTX2 texture files in both English and Chinese versions. They encompass editor configurations for project packaging, compatibility issues on iOS, and texture compression formats, providing clearer guidance for developers. Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Outside diff range, codebase verification and nitpick comments (1)
docs/graphics/texture/compression.md (1)
35-39
: Clarify the documentation on KTX2 generation in the editor.The section on configuring KTX2 generation during project packaging is clear but could benefit from a direct link to the 'Project Publishing' document for better user guidance.
- 可以参考『[项目发布](${docs}assets-build)』文档。 + 可以参考[项目发布文档](${docs}assets-build)。
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- docs/graphics/texture/compression.md (1 hunks)
Additional comments not posted (1)
docs/graphics/texture/compression.md (1)
41-42
: Well-documented texture compression format details.The details about ETC1S and UASTC are clearly explained, providing useful information on when to use each format based on texture type.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Outside diff range, codebase verification and nitpick comments (2)
docs/zh/graphics/texture/compression.md (2)
39-39
: Image usage and accessibility.The image included here is essential for understanding the editor's UI for texture compression settings. However, consider adding a more descriptive
alt
text for better accessibility and SEO.- <img src="https://mdn.alipayobjects.com/rms/afts/img/A*fmURSZ4HwKUAAAAAAAAAAAAAARQnAQ/original/image-20240705112419249.png" alt="image-20240705112419249" style="zoom:50%;" /> + <img src="https://mdn.alipayobjects.com/rms/afts/img/A*fmURSZ4HwKUAAAAAAAAAAAAAARQnAQ/original/image-20240705112419249.png" alt="Editor texture panel showing options to configure KTX2 compression" style="zoom:50%;" />
41-43
: Consider adding more details on texture formats.The brief descriptions of ETC1S and UASTC are helpful, but more detailed information or examples could enhance understanding, especially for those unfamiliar with these formats.
Consider expanding this section to include examples of when to use each format or visual comparisons to illustrate the differences in quality and size.
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- docs/zh/graphics/texture/compression.md (1 hunks)
Additional comments not posted (1)
docs/zh/graphics/texture/compression.md (1)
46-46
: Update compatibility information for iOS.The text mentions iOS 11.3+ compatibility but also notes issues with versions 11.0 to 11.2. This is crucial information for developers targeting iOS devices.
However, ensure that this information is up-to-date and consider adding a note about any recent iOS updates that might affect compatibility.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Please check if the PR fulfills these requirements
resolve: #2216
Summary by CodeRabbit