Skip to content

Conversation

@WBbug
Copy link
Collaborator

@WBbug WBbug commented Aug 31, 2025

  • add stroke select state
image
  • add stroke type text
image

@pubuzhixing8
Copy link
Contributor

LGTM, thank @WBbug !

If you have time you could review it again @NaoCoding .

@NaoCoding
Copy link
Member

Hi @pubuzhixing8 ,
I can do the code review later.

Copy link
Member

@NaoCoding NaoCoding left a comment

Choose a reason for hiding this comment

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

chrome-capture-2025-9-1

Hello, it seems that the selected states aren't set properly. When the stroke style changed, the selected states will not changed to the target style.

@NaoCoding
Copy link
Member

The selected status should change the background color of the Toolbar button, but it seems that the background color will always shown at the first button

@cloudflare-workers-and-pages
Copy link

Deploying drawnix with  Cloudflare Pages  Cloudflare Pages

Latest commit: b328d44
Status: ✅  Deploy successful!
Preview URL: https://c8e35e64.drawnix.pages.dev
Branch Preview URL: https://optimize-stroke-text.drawnix.pages.dev

View logs

Copy link
Member

@NaoCoding NaoCoding left a comment

Choose a reason for hiding this comment

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

LGTM! Maybe @pubuzhixing8 you can check for the last time.

@pubuzhixing8
Copy link
Contributor

pubuzhixing8 commented Sep 1, 2025

Great @WBbug @NaoCoding , I will merge then.

@pubuzhixing8 pubuzhixing8 merged commit ae5c17d into develop Sep 1, 2025
2 checks passed
@pubuzhixing8 pubuzhixing8 deleted the optimize-stroke-text branch September 1, 2025 12:59
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.

4 participants