From 0152271fb87ebd7c2598d0d4f0c053804bb8384a Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@odoo.com>
Date: Thu, 3 Oct 2019 10:45:39 +0200
Subject: [PATCH] [IMP] runbot_merge: layout responsivity

* only show 2 stagings on cellphones as 4 is way too much, moving to a
  vertical layout would probably be a bad idea as stagings can already
  be very tall and then we have multiple branches stacked on one
  another, unless we also make branches foldable

  the more complete list of stagings (per branch) is available on the
  branch's page anyway so providing a not-completely-broken home looks
  more useful, and at a fundamental level the current / last staging
  is really the one we care about
* remove the size bounds on stagings to avoid smushing all the cells
  together and overlapping text, sadly can't overflow scroll the
  stagings element because you can't have an overflow-x: scroll and an
  overflow-y: visible (that becomes auto)
---
 runbot_merge/views/templates.xml | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/runbot_merge/views/templates.xml b/runbot_merge/views/templates.xml
index ca57d724..72354973 100644
--- a/runbot_merge/views/templates.xml
+++ b/runbot_merge/views/templates.xml
@@ -7,10 +7,7 @@
                 align-items: stretch;
             }
             .stagings > li {
-                /* 4 to 6 stagings displayed, avoid stretching */
                 flex: 1;
-                min-width: 16%;
-                max-width: 25%;
 
                 padding: 0.1em;
                 padding-left: 0.5em;
@@ -117,7 +114,8 @@
                     <t t-if="pending">bg-info</t>
                     <t t-if="not (success or failure or pending)">bg-gray-lighter</t>
                 </t>
-                <t t-set="decorationclass">
+                <t t-set="decorationclass" >
+                    <t t-if="staging_index >= 2">hidden-xs</t>
                     <t t-if="staging_index >= 4">visible-lg-block</t>
                 </t>
                 <t t-set="title">
@@ -125,7 +123,7 @@
                     <t t-if="staging.state == 'ff_failed'">Fast Forward Failed</t>
                     <t t-if="staging.state not in ('canceled', 'ff_failed')"><t t-esc="staging.reason"/></t>
                 </t>
-                <li t-attf-class="staging {{stateclass}} {{decorationclass}}" t-att-title="title.strip() or None">
+                <li t-attf-class="staging {{stateclass.strip()}} {{decorationclass.strip()}}" t-att-title="title.strip() or None">
                     <ul class="list-unstyled">
                         <li t-foreach="staging.batch_ids" t-as="batch" class="batch">
                             <t t-esc="batch.prs[:1].label"/>