Skip to content
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

🐛(frontend) add default toolbar buttons #355

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AntoLC
Copy link
Collaborator

@AntoLC AntoLC commented Oct 18, 2024

Purpose

We are overriding the default toolbar to add the markdown and ai buttons.
By doing that we were missing some default buttons that are useful depend on the block type.

Proposal

  • 🐛(frontend) add default toolbar buttons

Demo

image

@AntoLC AntoLC added bug Something isn't working frontend editor labels Oct 18, 2024
@AntoLC AntoLC linked an issue Oct 18, 2024 that may be closed by this pull request
We are overriding the default toolbar to add the
markdown and ai buttons. By doing that we were
missing some default buttons that are useful depend
on the block type. This commit adds the default
buttons to the toolbar.
@AntoLC AntoLC marked this pull request as ready for review October 18, 2024 09:11
@AntoLC AntoLC self-assigned this Oct 18, 2024
).toBeVisible();
await expect(page.locator('button[data-test="colors"]')).toBeVisible();
await expect(page.locator('button[data-test="unnestBlock"]')).toBeVisible();
await expect(page.locator('button[data-test="createLink"]')).toBeVisible();
Copy link
Collaborator

@PanchoutNathan PanchoutNathan Oct 18, 2024

Choose a reason for hiding this comment

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

Just the add the BlockTypeSelect inside tests and good to me !

Copy link
Collaborator

@PanchoutNathan PanchoutNathan left a comment

Choose a reason for hiding this comment

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

Just one minor thing and let's go :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working editor frontend
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

✨Add menu on editor attachments
2 participants