-
-
Notifications
You must be signed in to change notification settings - Fork 253
fix Label render and add test case #2625
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@aismann im just add test, I think we should add test first, then fix bug |
Ok I see. |
agree |
|
@NgVThangBz |
|
does all bugs releated to SDF ? |
|
@halx99 some SDF and and some both |
|
@NgVThangBz I think the outline slider should only be enabled/shown if the Outline checkbox "checked" |
|
@aismann im update it |
|
@halx99 |
I can't control the check order |
|
@NgVThangBz |
|
if Outline enable -> glow will disable (if enable) , |
1 similar comment
|
if Outline enable -> glow will disable (if enable) , |
I thick 2 radiobuttons be better as 2 checkboxes |
|
If you wish, the “title” of the test can also be more self-explanatory: |
Sounds good. /Edit: |
|
@halx99 |
|
new UI Screen.Recording.2025-08-08.at.14.10.21.mov |
Yes, there is no old API in axmol, so just |
|
@NgVThangBz @halx99 |
I think it's ok, it just test case, so no need take much time to optimize ui layout |
Correct. But we do it one time forever. |
|
I have pushed the code |
|
Resizing the font is working now @NgVThangBz @halx99 |
|
Merged, thanks @NgVThangBz |











fix Label render
Add test Label render:
Normal, SDF
Glow, Outline, Shadow, Bold
Change Size font, Scale, size outline