Skip to content

feat(DoDontBlock): extracted the toolbar - #1393

Merged
benfrontify merged 12 commits into
mainfrom
gcm-203-add-file-dosdontstoolbartsx-dosdontsitemstsx
Jul 23, 2026
Merged

feat(DoDontBlock): extracted the toolbar#1393
benfrontify merged 12 commits into
mainfrom
gcm-203-add-file-dosdontstoolbartsx-dosdontsitemstsx

Conversation

@benfrontify

Copy link
Copy Markdown
Contributor

extracted the toolbar from DoDontItem

@benfrontify
benfrontify requested a review from a team as a code owner July 14, 2026 06:12
@benfrontify benfrontify changed the title Feat(DoDontBlock): extracted the toolbar feat(DoDontBlock): extracted the toolbar Jul 14, 2026
@benfrontify
benfrontify requested review from ragi96 and removed request for hochreutenerl July 14, 2026 06:15
Comment thread packages/dos-donts-block/src/DoDontItem.tsx Outdated
Comment thread packages/dos-donts-block/src/DoDontToolbar.tsx Outdated
Comment thread packages/dos-donts-block/src/DoDontToolbar.tsx Outdated
@benfrontify

Copy link
Copy Markdown
Contributor Author

I’m not quite sure which naming convention I should use. I followed your latest review because DoDont seems more reasonable than DosDont, which was suggested in the first review. Please let me know if I should change it.

@benfrontify
benfrontify requested a review from ragi96 July 14, 2026 12:04
@benfrontify

Copy link
Copy Markdown
Contributor Author

i wold have done DosDonts but idk

Comment thread packages/dos-donts-block/src/DoDontToolbar.tsx Outdated
Comment thread packages/dos-donts-block/src/DoDontToolbar.tsx
@benfrontify
benfrontify requested a review from ragi96 July 15, 2026 09:37
Comment thread packages/dos-donts-block/src/DoDontToolbar.tsx Outdated
Comment thread packages/dos-donts-block/src/components/DoDontItemWrapper.tsx
Comment thread packages/dos-donts-block/src/components/DoDontItemWrapper.tsx
Comment thread packages/dos-donts-block/src/components/DoDontItemWrapper.tsx
Comment thread packages/dos-donts-block/src/DoDontItem.tsx Outdated
@benfrontify
benfrontify requested review from ragi96 July 21, 2026 13:46
Comment thread packages/dos-donts-block/src/DoDontItemWrapper.spec.tsx Outdated
Comment thread packages/dos-donts-block/src/components/DoDontItemWrapper.spec.tsx
Comment thread packages/dos-donts-block/src/DoDontItem.tsx Outdated

@ragi96 ragi96 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Github error?

Comment thread packages/dos-donts-block/src/components/DoDontItemWrapper.tsx
@benfrontify
benfrontify merged commit e0a024c into main Jul 23, 2026
13 checks passed
@benfrontify
benfrontify deleted the gcm-203-add-file-dosdontstoolbartsx-dosdontsitemstsx branch July 23, 2026 13:15
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.

2 participants