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

Edited link URL doesn't reflect new value until after refresh #15522

Closed
iandunn opened this issue May 9, 2019 · 2 comments · Fixed by #15573
Closed

Edited link URL doesn't reflect new value until after refresh #15522

iandunn opened this issue May 9, 2019 · 2 comments · Fixed by #15573
Labels
[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended

Comments

@iandunn
Copy link
Member

iandunn commented May 9, 2019

Describe the bug

When copying a page and then editing the value of a link, the change is saved, but not reflected in the UI until after refresh.

To reproduce

Steps to reproduce the behavior:

  1. Open a page that contains a Classic block and has links in it
  2. Go to code editor and copy the raw markup
  3. Create a new page and open the code editor, paste the raw markup, switch back to the code editor
  4. Convert the existing Classic block into regular blocks
  5. Click on a link inside one of the paragraph blocks, the toolbar and popover appear
  6. Click on the pencil icon in the popover
  7. Change the value of the URL
  8. Hit enter
  9. Notice that the value has not changed.
  10. Save page, refresh, notice that the value has changed.

Expected behavior

The new value should be reflected in the UI immediately.

@iandunn iandunn added [Type] Bug An existing feature does not function as intended [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable labels May 9, 2019
@iandunn
Copy link
Member Author

iandunn commented May 9, 2019

Actually, I'm seeing this happen with existing pages too, without steps 1-4. It's odd that that wasn't noticed before 5.2 was release, though, so maybe there are other conditions?

@youknowriad youknowriad added the [Priority] High Used to indicate top priority items that need quick attention label May 10, 2019
@aduth
Copy link
Member

aduth commented May 10, 2019

It's quite likely this and #15503 are of the same underlying cause.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants