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

File block: Fix editing of empty file name #50607

Conversation

n2erjo00
Copy link
Contributor

@n2erjo00 n2erjo00 commented May 12, 2023

What?

Fixes: #50421

Added wrapper element "wp-block-file__button-richtext-wrapper" for the RichText component.

Why?

Issue was pointed out by #50421

How?

If File block was added and then file name was erased from RichText user was unable to re-edit the RichText content.

Testing Instructions

  1. Open post or page and File block
  2. Once added name of the file is added as the name of file in the editor
  3. Erase that to see placeholder
  4. Click on some other block
  5. Re-edit the name of file in the File block.
    • If checkout trunk and do same steps you are unable to re-edit the name of the file

@n2erjo00 n2erjo00 requested a review from ajitbohra as a code owner May 12, 2023 20:44
@skorasaurus skorasaurus added [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Package] Rich text /packages/rich-text [Block] File Affects the File Block and removed [Package] Rich text /packages/rich-text labels May 16, 2023
Copy link
Contributor

@ntsekouras ntsekouras left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @n2erjo00 !

@ntsekouras ntsekouras merged commit ce29b86 into WordPress:trunk May 19, 2023
@github-actions github-actions bot added this to the Gutenberg 15.9 milestone May 19, 2023
@ntsekouras ntsekouras changed the title Added wrapper element for RichText in File block File block: Fix editing of empty file name May 19, 2023
@ntsekouras ntsekouras added the [Type] Bug An existing feature does not function as intended label May 19, 2023
westonruter added a commit that referenced this pull request May 19, 2023
…dd/static-closures

* 'trunk' of https://github.com/WordPress/gutenberg: (26 commits)
  Add transparent outline to input control BackdropUI focus style. (#50772)
  Added wrapper element for RichText in File block (#50607)
  Remove the experimental flag of the command center (#50781)
  Update the document title in the site editor to open the command center (#50369)
  Remove `unwrap` from transforms and add `ungroup` to more blocks (#50385)
  Add new experimental version of DropdownMenu (#49473)
  Force display of in custom css input boxes to LTR (#50768)
  Polish experimental navigation block (#50670)
  Support negation operator in selectors in the Interactivity API (#50732)
  Minor updates to theme.json schema pages (#50742)
  $revisions_controller is not used. Let's delete it. (#50763)
  Remove OffCanvasEditor (#50705)
  Mobile - E2E test - Update code to use the new navigateUp helper (#50736)
  Try: Smaller external link icon (#50728)
  Block Editor: Remove unused 'useIsDimensionsSupportValid' method (#50735)
  Fix flaky media inserter drag-and-dropping e2e test (#50740)
  docs: Fix change log typo (#50737)
  Edit Site: Fix `useEditedEntityRecord()` loading state (#50730)
  Fix labelling, description, and focus style of the block transform to pattern previews (#50577)
  Fix Global Styles sidebar block selection on zoom out mode (#50708)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] File Affects the File Block [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File block: Can not select and edit the link text
3 participants