Merge lp:~dpm/help-app/phone-navigation-update-1st-tab-title into lp:~ubuntu-touch-coreapps-drivers/help-app/trunk

[David Planella]

Fixes the title of the first tab not being updated upon app launch. Again, thanks daker for the fix!

[Daniel Holbach (dholbach)]

Great work, thanks!

review: Approve

=== modified file 'edit-here/index.html'
--- edit-here/index.html	2015-03-18 08:11:00 +0000
+++ edit-here/index.html	2015-03-18 12:56:18 +0000
@@ -99,19 +99,23 @@
 
             // Dynamically load the content from individual FAQ pages
             // to each tab
-            $("li[data-page]").each(function(){
-                var tabId = $(this).data('page');
-
-                var that = $(this);
+            $("li[data-page]").each(function(i){
+                var tabId = $(this).data('page'),
+                    that = $(this),
+                    tabText,
+                    tabHTML,
+                    tabTitle,
+                    tabTextNoHeader;
 
                 $.get(tabId + "." + language + ".html", function (response) {
                     // This is asynchronous, so make sure
                     // any operations with tab content are done within
                     // this function.
-                    var tabText = response;
-                    var tabHTML = $.parseHTML(tabText);
-                    var tabTitle = $('<div/>').append(tabHTML).find('h1').text();
-                    var tabTextNoHeader = $(tabText).not('h1');
+                    tabText = response;
+                    tabHTML = $.parseHTML(tabText);
+                    tabTitle = $('<div/>').append(tabHTML).find('h1').text();
+                    tabTextNoHeader = $(tabText).not('h1');
+                    if (i == 0) $('[data-role="tabtitle"]').html(tabTitle);
 
                     // Populate the tab titles
                     that.html(tabTitle);
@@ -119,6 +123,7 @@
                     // Populate the tab pages
                     $("#" + tabId).html(tabTextNoHeader);
 
+                    // $('a').attr('target', '_blank');
                 });
             });
         };