Make WordPress Core

Opened 2 months ago

Closed 8 weeks ago

Last modified 8 weeks ago

#54506 closed task (blessed) (fixed)

Remove Navigation Areas

Reported by: noisysocks Owned by: spacedmonkey
Milestone: 5.9 Priority: normal
Severity: normal Version: trunk
Component: Editor Keywords: has-patch has-unit-tests commit
Focuses: rest-api Cc:

Description

In https://github.com/WordPress/gutenberg/issues/36524 it was decided to remove the Navigation Areas functionality for WP 5.9. We will need to remove the wp_navigation_area CPT and wp-includes/navigation-areas.php code from Core.

Change History (16)

#1 in reply to: ↑ description @noisysocks
2 months ago

Replying to noisysocks:

In https://github.com/WordPress/gutenberg/issues/36524 it was decided to remove the Navigation Areas functionality for WP 5.9. We will need to remove the navigation area endpoints and wp-includes/navigation-areas.php code from Core.

This ticket was mentioned in PR #1944 on WordPress/wordpress-develop by noisysocks.


2 months ago

  • Keywords has-patch has-unit-tests added; needs-patch removed

This removes the Navigation Area endpoints and utility functions from Core. In https://github.com/WordPress/gutenberg/issues/36524 it was decided to remove the Navigation Area functionality form WP 5.9.

Trac ticket: https://core.trac.wordpress.org/ticket/54506

#3 @prbot
2 months ago

noisysocks commented on PR #1944:

Not sure why GitHub isn't letting me request a review from @getdave 🤔

#4 @prbot
2 months ago

getdave commented on PR #1944:

Not sure why GitHub isn't letting me request a review from @getdave 🤔

I guess because I haven't committed here before or something?

#5 @prbot
2 months ago

getdave commented on PR #1944:

I tried to testing this but my local WordPress develop setup wouldn't start up. Adam's 👍 is good enough for me.

#6 @spacedmonkey
2 months ago

  • Focuses rest-api added
  • Version set to trunk

This ticket was mentioned in Slack in #core-restapi by spacedmonkey. View the logs.


2 months ago

#8 @prbot
2 months ago

spacedmonkey commented on PR #1944:

This looks good @noisysocks . Happy to commit?

#9 @spacedmonkey
2 months ago

  • Keywords commit added

#10 @prbot
2 months ago

getdave commented on PR #1944:

This looks good @noisysocks . Happy to commit?

If you wouldn't mind committing @spacedmonkey I think we're good to go.

#12 @spacedmonkey
8 weeks ago

  • Owner set to spacedmonkey
  • Resolution set to fixed
  • Status changed from new to closed

In 52272:

Editor: Remove Navigation Areas

Navigation area were merged as part of full site editing and the new navigation block. This functionality is experimental and not currently used in WordPress core, so should be removed.

Props noisysocks, spacedmonkey, get_dave, zieladam.
Fixes #54506.

This ticket was mentioned in Slack in #core-committers by hellofromtonya. View the logs.


8 weeks ago

#14 @noisysocks
8 weeks ago

In 52273:

Restore block PHP to canonical version in @wordpress/block-library

r52272 erroneously modified src/wp-includes/blocks/navigation.php. This file
is copied from @wordpress/block-library and so must be updated via npm. This
commit restores the file by running grunt build --dev.

Follows [52272].
See #54506.

#15 @peterwilsoncc
8 weeks ago

In 52274:

Build/Test Tools: Revert ignore prop change in [52272].

See #54506.

Note: See TracTickets for help on using tickets.