Skip to content

feat(richTextEditor): add stickytoolbaroffset as prop#1966

Open
Sam Ashley (sam-is-content) wants to merge 1 commit into
masterfrom
rich-text/add-sticky-toolbar-offset
Open

feat(richTextEditor): add stickytoolbaroffset as prop#1966
Sam Ashley (sam-is-content) wants to merge 1 commit into
masterfrom
rich-text/add-sticky-toolbar-offset

Conversation

@sam-is-content
Copy link
Copy Markdown

stickyToolbarOffset is currently a prop for the ConnectedRichTextEditor but not for the RichTextEditor which I understand as the one we should use for apps. I would like to add stickyToolbarOffset as a prop of the RichTextEditor so that when a rich text field is used in an app, we can optionally have a sticky toolbar.

Why am I doing this now?
I received a Zendesk ticket from a customer saying they like the new rich-text-versioning app but do not find it worth using because the toolbar is not sticky. Let's change that :)

@sam-is-content Sam Ashley (sam-is-content) requested a review from a team as a code owner October 29, 2025 23:00
@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 29, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
field-editors Ready Ready Preview Oct 29, 2025 11:01pm

@YvesRijckaert
Copy link
Copy Markdown
Contributor

hey Sam Ashley (@sam-is-content) 👋
were you able to try this out when it's rendered in an iframe? I thought this wouldn't work since sticky positioning only considers the scrolling container within the same document

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Before we merge let's see if it works within an iframe

@github-actions
Copy link
Copy Markdown

Marking pull request as stale since there was no activity for 30 days

@github-actions github-actions Bot added the stale Used to mark when there was no activity for a set period of time label Nov 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package:rich-text stale Used to mark when there was no activity for a set period of time

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants