WordPress.org

Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #38904, comment 8


Ignore:
Timestamp:
02/02/2017 04:24:47 PM (3 years ago)
Author:
adamsilverstein
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #38904, comment 8

    initial v1  
    11@timersys & @welcher - thanks for the bug report and patch.
    2 
    3 I found several related tickets although there seems to be some confusion on hooks for admin vs. front end.
    42
    53My concern with the current patch is it is so narrowly focused & application/need specific. While it solves the reported use case, what if plugins want to add something to a submenu or alter the classes added to menus? We might need many hooks to address every use case. I think this is the reason the general approach of replacing the walker has been used, and was the resolution for #13342.