Skip to content

ExprName - combine elements and cleanup docs#7370

Merged
Moderocky merged 3 commits into
SkriptLang:dev/featurefrom
ShaneBeee:shane/expr-name-cleanup
Jan 3, 2025
Merged

ExprName - combine elements and cleanup docs#7370
Moderocky merged 3 commits into
SkriptLang:dev/featurefrom
ShaneBeee:shane/expr-name-cleanup

Conversation

@ShaneBeee
Copy link
Copy Markdown
Contributor

Description

This PR aims to do a big cleanup to ExprName, 2 major changes:

  1. Register 1 element instead of 3. This prevents 3 separate expressions showing up in the docs, making it super confusing when people open the docs, find the first ExprName, and think thats it.
    (all 3 elements have the same examples, description, version... only diff is the pattern. Super confusing)
    Before:
    . Screenshot 2025-01-02 at 5 37 03 PM
    After: (just 1)

  2. Cleanup the weird html list issues in the description.
    (removing all the "ul" list things.)
    Before:
    see docs:
    https://docs.skriptlang.org/expressions.html#ExprName-4
    (look at description)
    After:

Screenshot 2025-01-02 at 5 38 16 PM

Target Minecraft Versions: any
Requirements: none
Related Issues: none

Comment thread src/main/java/ch/njol/skript/expressions/ExprName.java Outdated
@cheeezburga cheeezburga added enhancement Feature request, an issue about something that could be improved, or a PR improving something. feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. labels Jan 3, 2025
@Moderocky Moderocky merged commit 3e8c8ba into SkriptLang:dev/feature Jan 3, 2025
@ShaneBeee ShaneBeee deleted the shane/expr-name-cleanup branch January 3, 2025 18:59
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
* ExprName - combine 3 registrations

* ExprName - cleanup docs

* ExprName - suggestion from sovde
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Feature request, an issue about something that could be improved, or a PR improving something. feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants