Skip to content
This repository has been archived by the owner on Oct 3, 2022. It is now read-only.

The DSL text component builder is inconvenient #311

Closed
TheFruxz opened this issue Aug 13, 2022 · 0 comments · Fixed by #315
Closed

The DSL text component builder is inconvenient #311

TheFruxz opened this issue Aug 13, 2022 · 0 comments · Fixed by #315
Assignees
@TheFruxz
Labels
From: user Created / Reported from user In: MoltenKT-Unfold Affecting the MoltenKT-Unfold Target Relates: v1.0 Relates to the v1.0 Version/Release State: verified Verified, checked if true / real
Milestone
Stable Release | ...

Comments

@TheFruxz
Copy link
Owner

The new text component builder introduced with #205 is a little bit confusing to use because the name of the functions, used inside the DSL is the same as the standalone text component functions.

The result is easy misuse of functions inside the builder
@TheFruxz TheFruxz added State: verified Verified, checked if true / real Relates: v1.0 Relates to the v1.0 Version/Release From: user Created / Reported from user In: MoltenKT-Unfold Affecting the MoltenKT-Unfold Target labels Aug 13, 2022
@TheFruxz TheFruxz added this to the Stable Release | v1.0 milestone Aug 13, 2022
@TheFruxz TheFruxz self-assigned this Aug 13, 2022
@TheFruxz TheFruxz moved this to 🔍Idea in Project JET Aug 13, 2022
@TheFruxz TheFruxz moved this from 🔍Idea to 🗒️Todo in Project JET Aug 13, 2022
@TheFruxz TheFruxz linked a pull request Aug 14, 2022 that will close this issue
feature: upgrade unfold to new context based structure #315
Merged
Repository owner moved this from 🗒️Todo to 🍩Done in Project JET Aug 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@TheFruxz TheFruxz

Labels
From: user Created / Reported from user In: MoltenKT-Unfold Affecting the MoltenKT-Unfold Target Relates: v1.0 Relates to the v1.0 Version/Release State: verified Verified, checked if true / real
Projects
No open projects
Project JET
Status: 🍩Done
Milestone
Stable Release | v1.0
Development

Successfully merging a pull request may close this issue.

feature: upgrade unfold to new context based structure TheFruxz/MoltenKT
1 participant
@TheFruxz