Skip to content

Add mipmaps attribute to texture reference and remove unused reference in inspector#1694

Merged
willeastcott merged 2 commits intoplaycanvas:mainfrom
YeonSeong-Lee:main
Jan 19, 2026
Merged

Add mipmaps attribute to texture reference and remove unused reference in inspector#1694
willeastcott merged 2 commits intoplaycanvas:mainfrom
YeonSeong-Lee:main

Conversation

@YeonSeong-Lee
Copy link
Contributor

@YeonSeong-Lee YeonSeong-Lee commented Jan 19, 2026

What's Changed

Expose mipmaps in the Texture inspector tooltip/help: Added the asset:texture:mipmaps attribute reference (title/description/API link) so the Mipmaps field can display proper tooltip content.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds documentation reference support for the texture mipmaps attribute by removing an explicit null reference assignment and adding the corresponding reference definition.

Changes:

  • Removed reference: null from the mipmaps attribute in texture inspector to allow automatic reference assignment
  • Added mipmaps attribute reference definition with documentation and API link

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/editor/inspector/assets/texture.ts Removed explicit reference: null to enable auto-assignment of the mipmaps reference
src/editor/attributes/reference/assets/texture.ts Added mipmaps reference entry with documentation and API URL

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@willeastcott willeastcott left a comment

Choose a reason for hiding this comment

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

Great! Thanks for the contribution. 🙏

@willeastcott willeastcott merged commit 7814e6c into playcanvas:main Jan 19, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: editor interface bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants