Skip to content

Conversation

@capdevon
Copy link
Contributor

@capdevon capdevon commented May 24, 2025

This PR significantly improves the Javadoc for the Material class, making its functionality clearer and easier to understand for developers. I've created this initial PR specifically for the Javadoc as it's a prerequisite for my next PR.

That upcoming PR will introduce a feature that's already present in the codebase but currently disabled. To facilitate an easier review process, I've opted to split these changes into two separate PRs.

Additionally, I've repositioned the toString() method before the read/write methods for improved code readability, and I've also performed some minor code formatting in various places.

@yaRnMcDonuts yaRnMcDonuts added this to the v3.9.0 milestone May 28, 2025
@capdevon
Copy link
Contributor Author

Hi @yaRnMcDonuts ,
Could you please merge this PR when you get a chance? I need it in to start on the next feature. As mentioned in the description, there aren't any critical changes here.

@yaRnMcDonuts yaRnMcDonuts merged commit 9a10591 into jMonkeyEngine:master Jun 10, 2025
16 checks passed
@capdevon capdevon deleted the capdevon-Material branch June 15, 2025 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants