There is an unused include file, mainmenu.inc, and related CSS which can be removed.
A similar file, additem-menu.inc should be removed also.
Created attachment 12764 [details] [review] Bug 8898 - Remove unused mainmenu include and related styles in the staff client This patch removes the mainmenu and additem-menu includes and #navigation references in staff-global.css which were used by those includes.
Created attachment 12784 [details] [review] [SIGNED-OFF] Bug 8898 - Remove unused mainmenu include and related styles in the staff client This patch removes the mainmenu and additem-menu includes and #navigation references in staff-global.css which were used by those includes. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Applied patch, checked many different pages, no ill effects were percieved.
QA comment: * remove things only (but some string, thus QAing before String Freeze) * grep mainmenu.inc and additem-menu show it's not used anymore passed QA
Patch pushed to master
Pushed to 3.8.x, will be in 3.8.7