Validation of custom menu link

[dvirhazout]

1. go to menus page
2. try to add custom menu link with label only. without a URL.

expected and actual result: validation failed and link cannot be added.

1. go to menus page
2. add custom menu link with label and URL: '#'
3. save menu
4. edit the custom link from the menu and remove the URL.
5. click save

expected result: validation failed and menu won't be saved.
actual result: menu saved. no validation.

[audrasjb]

Replying to dvirhazout:

Hello,
I think that is the desired behavior.
Indeed, a user might need to use an empty anchor (preferably using "#").

For example, this allows a multilevel menu to add some "virtual" (non-clickable) levels that only serve to unfold the child levels.

[dvirhazout]

If that's the case, I can't understand the motive to this validation from the first place.

Replying to audrasjb:

Replying to dvirhazout:

Hello,
I think that is the desired behavior.
Indeed, a user might need to use an empty anchor (preferably using "#").

For example, this allows a multilevel menu to add some "virtual" (non-clickable) levels that only serve to unfold the child levels.

[mukesh27]

@dvirhazout you are right WordPress core team need to add patch for validation or they have to remove validation when anyone add custom menu without link.

Replying to dvirhazout:

If that's the case, I can't understand the motive to this validation from the first place.

Replying to audrasjb:

Replying to dvirhazout:

Hello,
I think that is the desired behavior.
Indeed, a user might need to use an empty anchor (preferably using "#").

For example, this allows a multilevel menu to add some "virtual" (non-clickable) levels that only serve to unfold the child levels.

[jukkarau]

Adds URL validation for custom links in customizer

[jukkarau]

Attached a diff, which adds validation for the custom link URLs in the Customizer menus. Using the exact same regex, which is used on the frontend code. The validation now checks the URL format for the submitted strings, and if the URL is empty it invalidates that too. (​https://github.com/jukra/wordpress-develop/pull/1)