Merge pull request #241 from stloyd/issue_240

Fix for #240
KnpLabs · Sep 14, 2012 · c581724 · c581724
2 parents 81782e9 + 1700789
commit c581724
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/Knp/Bundle/KnpBundlesBundle/Resources/public/js/main.js b/src/Knp/Bundle/KnpBundlesBundle/Resources/public/js/main.js
@@ -1,4 +1,8 @@
 (function($) {
+    if (window.location.hash.length > 0) {
+        $('ul.nav-tabs li > a[data-target="' + window.location.hash.replace('#', '') + '"]').tab('show');
+    }
+
     $(document).bind('keydown.s keydown.f keydown./', function() {
         $('#search-query').focus();
         return false;

diff --git a/src/Knp/Bundle/KnpBundlesBundle/Resources/views/Bundle/show.html.twig b/src/Knp/Bundle/KnpBundlesBundle/Resources/views/Bundle/show.html.twig
@@ -209,14 +209,14 @@
 {% block left %}
 <div class="content-box tabbable">
     <ul class="nav nav-tabs">
-        <li class="active"><a href="#readme" data-toggle="tab"><i class="icon-info"></i>README</a></li>
+        <li class="active"><a href="#readme" data-toggle="tab" data-target="readme"><i class="icon-info"></i>README</a></li>
         {%- if bundle.license -%}
-        <li><a href="#license" data-toggle="tab"><i class="icon-bag"></i>LICENSE</a></li>
+        <li><a href="#license" data-toggle="tab" data-target="license"><i class="icon-bag"></i>LICENSE</a></li>
         {%- endif %}
         {%- if not bundle.canonicalConfig is empty -%}
-        <li><a href="#configuration" data-toggle="tab"><i class="icon-wrench"></i>Configuration</a></li>
+        <li><a href="#configuration" data-toggle="tab" data-target="configuration"><i class="icon-wrench"></i>Configuration</a></li>
         {%- endif %}
-        <li><a href="#activity" data-toggle="tab"><i class="icon-clock"></i>Recent activity</a></li>
+        <li><a href="#activity" data-toggle="tab" data-target="activity"><i class="icon-clock"></i>Recent activity</a></li>
     </ul>
 
     <div class="tab-content">