Merge lp:~dpm/help-app/more-phone-navigation-fixes into lp:~ubuntu-touch-coreapps-drivers/help-app/trunk

[David Planella]

Hardcode the max number of tabs shown on the phone to 8 until the bug in the app container to support tab scrolling is fixed.

<daker> the max-height is limited to 250px (5 tabs), but you should be able to scroll over the tabs to display the others
 the issue that ubuntu-html5-launcher is using qtwebkit for some reason which doesn't recognize -webkit-overflow-scrolling: touch;
<dpm> daker, oh, I see. But even if scrolling would work, my concern would be that there is no visual indication for the user that the tabs can do scrolling. If it were working, would -webkit-overflow-scrolling show a slider or something?
<daker> yes it should

[Daniel Holbach (dholbach)]

Thanks. Great work!

review: Approve

=== modified file 'edit-here/themes/phone/static/css/app.css'
--- edit-here/themes/phone/static/css/app.css	2015-03-18 10:03:45 +0000
+++ edit-here/themes/phone/static/css/app.css	2015-03-18 12:09:04 +0000
@@ -40,11 +40,24 @@
     display: none;
 }
 
+/* Do not show links on the
+   front page until we've figured out
+   how to make them work properly */
 #index a {
    display: none;
 }
 
+/* Disable links until they work
+   properly */
 a {
   pointer-events: none;
   cursor: default;
-}
\ No newline at end of file
+}
+
+/* Disable this when the app container
+   properly supports tab scrolling.
+   We hardcode support for showing
+   8 tabs (50px each) */
+[data-role="tabs"] {
+  max-height: 400px !important;
+}