From 42ea3b835ca9a5883dd44cfd8d326cb525e04575 Mon Sep 17 00:00:00 2001
From: Joseph Schorr <joseph.schorr@coreos.com>
Date: Mon, 12 Jan 2015 11:42:09 -0500
Subject: [PATCH 1/5] Fix NPE

---
 auth/permissions.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/auth/permissions.py b/auth/permissions.py
index ae398092d..f970450af 100644
--- a/auth/permissions.py
+++ b/auth/permissions.py
@@ -89,6 +89,8 @@ class QuayDeferredPermissionUser(Identity):
     if not self._permissions_loaded:
       logger.debug('Loading user permissions after deferring.')
       user_object = model.get_user_by_uuid(self.id)
+      if user_object is None:
+        return super(QuayDeferredPermissionUser, self).can(permission)
 
       # Add the superuser need, if applicable.
       if (user_object.username is not None and

From 5a0277b4654b64f178e76b11aa01ee1cfd7028de Mon Sep 17 00:00:00 2001
From: Joseph Schorr <joseph.schorr@coreos.com>
Date: Mon, 12 Jan 2015 12:17:00 -0500
Subject: [PATCH 2/5] Fix plans table on mobile

---
 static/css/quay.css                | 16 ++++++++++
 static/directives/plans-table.html | 50 ++++++++++++++++++++----------
 2 files changed, 50 insertions(+), 16 deletions(-)

diff --git a/static/css/quay.css b/static/css/quay.css
index 4df625503..bc23177df 100644
--- a/static/css/quay.css
+++ b/static/css/quay.css
@@ -3547,6 +3547,22 @@ p.editable:hover i {
   font-size: 16px;
 }
 
+.plans-table ul {
+  margin-top: 10px;
+  padding: 0px;
+}
+
+
+.plans-table ul li {
+  padding: 4px;
+  margin: 0px;
+}
+
+.plans-table ul li .plan-info {
+  padding: 4px;
+}
+
+
 .repo-breadcrumb-element .crumb {
   cursor: pointer;
 }
diff --git a/static/directives/plans-table.html b/static/directives/plans-table.html
index 2870a06de..d4badb3c6 100644
--- a/static/directives/plans-table.html
+++ b/static/directives/plans-table.html
@@ -1,23 +1,41 @@
 <div class="plans-table-element">
-  <table class="table table-hover plans-table-table" ng-show="plans">
-    <thead>
-      <th>Plan</th>
-      <th>Private Repositories</th>
-      <th style="min-width: 85px">Price</th>
-      <th></th>
-    </thead>
+  <ul class="plans-table-list visible-xs">
+    <li ng-repeat="plan in plans" ng-class="currentPlan == plan ? 'active' : ''">
 
-    <tr ng-repeat="plan in plans" ng-class="currentPlan == plan ? 'active' : ''">
-      <td>{{ plan.title }}</td>
-      <td>{{ plan.privateRepos }}</td>
-      <td><div class="plan-price">${{ plan.price / 100 }}</div></td>
-      <td class="controls">
         <a class="btn" href="javascript:void(0)"
            ng-class="currentPlan == plan ? 'btn-primary' : 'btn-default'"
            ng-click="setPlan(plan)">
-          {{ currentPlan == plan ? 'Selected' : 'Choose' }}
+          {{ plan.title }}
         </a>
-      </td>
-    </tr>
-  </table>
+
+      <div class="plan-info">
+        ${{ plan.price / 100 }} / month -
+        {{ plan.privateRepos }} repositories
+      </div>
+    </li>
+  </ul>
+
+  <div class="hidden-xs">
+    <table class="table table-hover plans-table-table" ng-show="plans">
+      <thead>
+        <th>Plan</th>
+        <th>Private Repositories</th>
+        <th style="min-width: 85px">Price</th>
+        <th></th>
+      </thead>
+
+      <tr ng-repeat="plan in plans" ng-class="currentPlan == plan ? 'active' : ''">
+        <td>{{ plan.title }}</td>
+        <td>{{ plan.privateRepos }}</td>
+        <td><div class="plan-price">${{ plan.price / 100 }}</div></td>
+        <td class="controls">
+          <a class="btn" href="javascript:void(0)"
+             ng-class="currentPlan == plan ? 'btn-primary' : 'btn-default'"
+             ng-click="setPlan(plan)">
+            {{ currentPlan == plan ? 'Selected' : 'Choose' }}
+          </a>
+        </td>
+      </tr>
+    </table>
+  </div>
 </div>

From 2878fd28c69bd58936aa3205617849981b1a8de4 Mon Sep 17 00:00:00 2001
From: Joseph Schorr <joseph.schorr@coreos.com>
Date: Mon, 12 Jan 2015 12:26:57 -0500
Subject: [PATCH 3/5] Fix header and some other minor fixes on mobile

---
 static/css/quay.css                   |  1 -
 static/directives/header-bar.html     | 15 ++++++++++++++-
 static/partials/new-organization.html |  4 ++--
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/static/css/quay.css b/static/css/quay.css
index bc23177df..e0aafa8e8 100644
--- a/static/css/quay.css
+++ b/static/css/quay.css
@@ -1528,7 +1528,6 @@ i.toggle-icon:hover {
 }
 
 .landing .call-to-action {
-  height: 40px;
   font-size: 18px;
   padding-left: 14px;
   padding-right: 14px;
diff --git a/static/directives/header-bar.html b/static/directives/header-bar.html
index 903de59bd..dc82a425a 100644
--- a/static/directives/header-bar.html
+++ b/static/directives/header-bar.html
@@ -19,8 +19,21 @@
     <li><a ng-href="{{ user.organizations.length ? '/organizations/' : '/tour/organizations/' }}" target="{{ appLinkTarget() }}" quay-section="organization">Organizations</a></li>
   </ul>
 
+  <!-- Phone -->
+  <ul class="nav navbar-nav navbar-right visible-xs" ng-switch on="user.anonymous">
+    <li ng-switch-when="false">
+      <a href="/user/" class="user-view" target="{{ appLinkTarget() }}">
+        <span class="avatar" size="32" hash="user.avatar"></span>
+        {{ user.username }}
+      </a>
+    </li>
+    <li ng-switch-default>
+      <a class="user-view" href="/signin/" target="{{ appLinkTarget() }}">Sign in</a>
+    </li>
+  </ul>
 
-  <ul class="nav navbar-nav navbar-right" ng-switch on="user.anonymous">
+  <!-- Normal -->
+  <ul class="nav navbar-nav navbar-right hidden-xs" ng-switch on="user.anonymous">
     <li>
       <form class="navbar-form navbar-left" role="search">
         <div class="form-group">
diff --git a/static/partials/new-organization.html b/static/partials/new-organization.html
index 0450e1ab5..85eaaed97 100644
--- a/static/partials/new-organization.html
+++ b/static/partials/new-organization.html
@@ -54,14 +54,14 @@
             <input id="orgName" name="orgName" type="text" class="form-control" placeholder="Organization Name"
                    ng-model="org.name" required autofocus data-trigger="manual" data-content="{{ createError }}"
                    data-placement="bottom" data-container="body" ng-pattern="/^[a-z0-9_]{4,30}$/">
-            <span class="description">This will also be the namespace for your repositories</span>
+            <span class="description">This will also be the namespace for your repositories. Must be alphanumeric and all lowercase.</span>
           </div>
 
           <div class="form-group nested">
             <label for="orgName">Organization Email</label>
             <input id="orgEmail" name="orgEmail" type="email" class="form-control" placeholder="Organization Email"
                    ng-model="org.email" required>
-            <span class="description">This address must be different from your account's email</span>
+            <span class="description">This address must be different from your account's email.</span>
           </div>
 
           <!-- Plans Table -->

From 7d412a5b685d87cdf7613ac56e204a9481e07c3f Mon Sep 17 00:00:00 2001
From: Joseph Schorr <joseph.schorr@coreos.com>
Date: Mon, 12 Jan 2015 12:35:06 -0500
Subject: [PATCH 4/5] Fix plans manager on mobile

---
 static/css/quay.css                 | 20 ++++++++++----------
 static/directives/plan-manager.html |  9 +++++++--
 2 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/static/css/quay.css b/static/css/quay.css
index e0aafa8e8..8636c07bc 100644
--- a/static/css/quay.css
+++ b/static/css/quay.css
@@ -1347,13 +1347,16 @@ i.toggle-icon:hover {
   position: relative;
 }
 
-.plan-price:after {
-  content: "/ mo";
-  position: absolute;
-  bottom: 0px;
-  right: 20px;
-  font-size: 12px;
-  color: #aaa;
+
+@media (min-width: 768px) {
+  .plan-price:after {
+    content: "/ mo";
+    position: absolute;
+    bottom: 0px;
+    right: 20px;
+    font-size: 12px;
+    color: #aaa;
+  }
 }
 
 .plans-list .plan .count {
@@ -1516,9 +1519,6 @@ i.toggle-icon:hover {
   right: 0px;
 }
 
-.landing-filter.signedin {
-}
-
 .landing-content {
   z-index: 2;
 }
diff --git a/static/directives/plan-manager.html b/static/directives/plan-manager.html
index 0c72e3b17..72a28dc7c 100644
--- a/static/directives/plan-manager.html
+++ b/static/directives/plan-manager.html
@@ -31,11 +31,16 @@
        ng-show="!planLoading"></div>
 
   <!-- Plans Table -->
+  <div class="visible-xs" style="margin-top: 10px"></div>
+
   <table class="table table-hover plans-list-table" ng-show="!planLoading">
     <thead>
       <td>Plan</td>
-      <td>Private Repositories</td>
-      <td style="min-width: 64px">Price</td>
+      <td>
+        <span class="hidden-xs">Private Repositories</span>
+        <span class="visible-xs"><i class="fa fa-hdd-o"></i></span>
+      </td>
+      <td style="min-width: 64px"><span class="hidden-xs">Price</span><span class="visible-xs">$/mo</span></td>
       <td></td>
     </thead>
 

From df9a417207df17ad3fe4dcf17d79eae84ac67d1c Mon Sep 17 00:00:00 2001
From: Joseph Schorr <joseph.schorr@coreos.com>
Date: Mon, 12 Jan 2015 12:37:01 -0500
Subject: [PATCH 5/5] Fix NPE

---
 static/js/app.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/static/js/app.js b/static/js/app.js
index 43fa2f5d4..69f135538 100644
--- a/static/js/app.js
+++ b/static/js/app.js
@@ -2645,7 +2645,7 @@ quayApp.directive('focusablePopoverContent', ['$timeout', '$popover', function (
 
         if (!scope) { return; }
         scope.$apply(function() {
-          if (!scope) { return; }
+          if (!scope || !$scope.$hide) { return; }
           scope.$hide();
         });
       };