[ticket/11989] Rationalize overall_header_navigation* events

[senky]

Checklist:

• Correct branch: master for new features; 3.2.x, 3.1.x for fixes
• Tests pass
• Code follows coding guidelines: master / 3.2.x, 3.1.x
• Commit follows commit message format

Tracker ticket (set the ticket ID to your ticket ID):

https://tracker.phpbb.com/browse/PHPBB3-11989

PHPBB3-11989

[iMattPro]

This is a bad change. It's too late for this (it was proposed while 3.1 was still in development).

You can't move template events around now.

Especially one's as widely used and important as these. This will affect a lot of extensions.

Instead you should add new template events in the locations where you moved the existing ones to.

[DavidIQ]

+1 agree it's too late to be moving events around. 3.3 should bring about a new default template so it'll be more appropriate there.