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

Navigation Block: Define allowedFormats option for NavigationLink #19507

Conversation

WunderBart
Copy link
Member

@WunderBart WunderBart commented Jan 8, 2020

Description

Defines the allowedFormats prop for the NavigationLink component. As discussed in #18318, the Inline Code option will no longer be available.

How has this been tested?

Check formatting options for a navigation item - they should include only:

  • Bold
  • Italic
  • Inline image
  • Strikethrough

Screenshots

image

Types of changes

Non-breaking change. If the Inline code formatting was used for nav item before it will still render as inline code in both editor and view, but the option will no longer be available.

Closes: #18318

@frontdevde frontdevde added [Block] Navigation Affects the Navigation Block [Feature] List View Menu item in the top toolbar to select blocks from a list of links. labels Jan 8, 2020
Copy link
Contributor

@frontdevde frontdevde left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Tested it locally and it worked as expected 👌

Screenshot 2020-01-08 at 15 53 50

@frontdevde frontdevde merged commit 808fa4e into WordPress:master Jan 8, 2020
@frontdevde
Copy link
Contributor

@MDWolinski
This PR is simply the implementation of this decision. Feel free to add further thoughts to #18318.

@MDWolinski
Copy link

Removed my comment and put it on #18318.

@ellatrix ellatrix added this to the Gutenberg 7.3 milestone Jan 20, 2020
@manmotive
Copy link

@WunderBart could you take another look at this please? The change prevents custom formats / buttons from being added to the menu item toolbar. I've reported it here:

#21649

..but I am not sure how to get that ticket 'triaged'.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Navigation Affects the Navigation Block [Feature] List View Menu item in the top toolbar to select blocks from a list of links.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigation Block: Reduce toolbar options for menu item
5 participants