-
Notifications
You must be signed in to change notification settings - Fork 19
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
489 link editing #490
489 link editing #490
Conversation
🦋 Changeset detectedLatest commit: cedf6f0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 24 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs comment about currentValues addressed or will break
packages/content-editor-plugin/src/components/Toolbar/InsertLink.tsx
Outdated
Show resolved
Hide resolved
c3d7107
to
cedf6f0
Compare
Pull Request Test Coverage Report for Build 6275528180
💛 - Coveralls |
</div> | ||
</DialogContent> | ||
<DialogActions> | ||
<ButtonBar> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for ButtonBar in actions right now. Although the actions don't have the full functionality of the old button bar. But if this works well then maybe leave it. I haven't tried it myself.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noted...will address in a future PR
No description provided.