#4554 - Anchor references to jump inside tabs

Identifier #4554
Issue type Feature request or suggestion
Title Anchor references to jump inside tabs
Status Completed
Handling member Chris Graham
Addon core_rich_media
Description It would be nice to have a standard way to open a specific tab, by default with no effort.

Here is a diff I made for it in v10...

diff --git a/themes/default/templates/COMCODE_TAB_HEAD.tpl b/themes/default/templates/COMCODE_TAB_HEAD.tpl
index a1fbc69b..b01ce074 100644
--- a/themes/default/templates/COMCODE_TAB_HEAD.tpl
+++ b/themes/default/templates/COMCODE_TAB_HEAD.tpl
@@ -1 +1 @@
-<a aria-controls="g_{$GET|*,tab_sets}_{TITLE|*}" role="tab" href="#" id="t_{$GET|*,tab_sets}_{TITLE|*}" class="tab{+START,IF,{FIRST}} tab_active tab_first{+END}{+START,IF,{LAST}} tab_last{+END}" onclick="select_tab('g','{$GET|*,tab_sets}_{TITLE|*}'); return false;"><span>{TITLE*}</span></a>
+<a id="{$GET|*,tab_sets}_{TITLE|*}__jump"></a><a aria-controls="g_{$GET|*,tab_sets}_{TITLE|*}" role="tab" href="#" id="t_{$GET|*,tab_sets}_{TITLE|*}" class="tab{+START,IF,{FIRST}} tab_active tab_first{+END}{+START,IF,{LAST}} tab_last{+END}" onclick="select_tab('g','{$GET|*,tab_sets}_{TITLE|*}'); return false;"><span>{TITLE*}</span></a>

However, this could be cleaner. We should review the automatic tab navigation code, which was mainly created for member profile tabs. Ideally it might be possible with something shorter than the "<something>__jump" anchor here, probably just "<something>".

It should also be documented.
Steps to reproduce

Funded? No
The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".

Rating

Unrated