- Issue created by @gkffzs
- 🇳🇿New Zealand quietone
Changes are made on on 11.x (our main development branch) first, and are then back ported as needed according to the Core change policies → .
- 🇮🇳India Ishani Patel
Hello @gkffzs,
I've checked and followed steps as you mentioned regarding delete action for menu.
So I've two solution for that,
1) Hide the "Delete" action for Views-generated links using small hook form alter
2) Improve the appearance instead of removing the "Delete" action, we can create a template override to display it as a disabled button or a tooltip indicating it cannot be deleted fromPlease let me know your thoughts for which solution is the best.
Thank you!
- 🇬🇷Greece gkffzs
Hello @ishani-patel, thank you for your answer and sorry for my delay!
I would suggest going with the first solution you proposed (hiding the "Delete" action) since it decreases the information load of the user. The second solution doesn't offer any benefit to the user, I think.
- Merge request !12132Issue #3513823: "Delete" action is shown as text and not link for Views-generated link in a menu → (Open) created by Unnamed author
- 🇮🇳India Ishani Patel
Hello @gkffzs,
I've created MR for the first solution as per your confirmation.
Kindly review.Thank you!
The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- Status changed to Needs review
14 days ago 10:16am 19 May 2025 - 🇮🇳India Ishani Patel
Hello @gkffzs,
I've fixed PHPCS and phpstan errors.Kindly review it.
Thank you!