Skip to content

Conversation

@TheLimeGlass
Copy link
Contributor

Problem

Minecraft internally in NMS uses ARGB frequently. Skript base Color class doesn't have a quick method for getting the ARGB of a color without being a SkriptColor

Solution

Adds a method to allow overriding ARGB in your custom implementation Color class.


Completes: none
Related: none

@TheLimeGlass TheLimeGlass requested a review from a team as a code owner August 10, 2025 17:27
@TheLimeGlass TheLimeGlass requested review from Absolutionism and Pesekjak and removed request for a team August 10, 2025 17:27
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Aug 10, 2025
Copy link
Contributor

@EquipableMC EquipableMC left a comment

Choose a reason for hiding this comment

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

Where is the check regarding if ARGB is even supported? Not all versions have this, and this will throw a NoClassDefFoundError exception.

@TheLimeGlass
Copy link
Contributor Author

Where is the check regarding if ARGB is even supported? Not all versions have this, and this will throw a NoClassDefFoundError exception.

It's existed before Skript's supported MC version 1.19.4

@skriptlang-automation skriptlang-automation bot added feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. and removed needs reviews A PR that needs additional reviews labels Aug 10, 2025
@Efnilite Efnilite moved this to Awaiting Merge in 2.13 Releases Aug 10, 2025
@Efnilite Efnilite added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Aug 10, 2025
Co-authored-by: Efnilite <35348263+Efnilite@users.noreply.github.com>
@Burbulinis Burbulinis merged commit 4b1a30c into SkriptLang:dev/feature Aug 13, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done - Awaiting Release in 2.13 Releases Aug 13, 2025
@skriptlang-automation skriptlang-automation bot added completed The issue has been fully resolved and the change will be in the next Skript update. and removed feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. labels Aug 13, 2025
@TheLimeGlass TheLimeGlass deleted the feature/color-argb branch October 2, 2025 10:51
@sovdeeth sovdeeth moved this from Done - Awaiting Release to Done - Released in 2.13 Releases Oct 15, 2025
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
* Adds an ARGB method to color api

* Update src/main/java/ch/njol/skript/util/Color.java
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
* Adds an ARGB method to color api

* Update src/main/java/ch/njol/skript/util/Color.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something.

Projects

No open projects
Status: Done - Released

Development

Successfully merging this pull request may close these issues.

6 participants