From ea8a700796ced7fd4be3d1c65abf34e7c65413b0 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Thu, 1 Jun 2017 01:02:04 +0300
Subject: [PATCH 1/5] Move a bunch of status-related styles from App to status
 component, alter status header layout for better user experience hopefully,
 make status actions placing more consistent, adjust margins.

---
 src/App.scss                     |  59 ------------
 src/components/status/status.vue | 149 ++++++++++++++++++++++++-------
 2 files changed, 115 insertions(+), 93 deletions(-)

diff --git a/src/App.scss b/src/App.scss
index 0a7e1ce5..3db81537 100644
--- a/src/App.scss
+++ b/src/App.scss
@@ -134,11 +134,6 @@ main-router {
     background-color: rgba(0,0,0,0.1);
 }
 
-.media-body {
-    flex: 1;
-    padding-left: 0.5em;
-}
-
 .container > * {
     min-width: 0px;
 }
@@ -147,60 +142,6 @@ main-router {
     color: grey;
 }
 
-.status-actions {
-    width: 50%;
-    display: flex;
-
-    div, favorite-button {
-        flex: 1;
-    }
-}
-
-status-text-container {
-    display: block;
-}
-
-.status-el {
-    line-height: 18px;
-
-    .notify {
-        .avatar {
-            border-width: 3px;
-            border-style: solid;
-        }
-    }
-
-    .media-left {
-        img {
-            margin-top: 0.2em;
-            float: right;
-            margin-right: 0.3em;
-            border-radius: 5px;
-        }
-    }
-
-    .retweet-info {
-        padding: 0.7em 0 0 0.6em;
-
-        .media-left {
-            display: flex;
-
-            i {
-                align-self: center;
-                text-align: right;
-                flex: 1;
-                padding-right: 0.3em;
-            }
-        }
-    }
-
-    .media-heading {
-        small {
-            font-weight: lighter;
-        }
-        margin-bottom: 0.3em;
-    }
-}
 nav {
     z-index: 1000;
 }
diff --git a/src/components/status/status.vue b/src/components/status/status.vue
index 6fd53420..8d776307 100644
--- a/src/components/status/status.vue
+++ b/src/components/status/status.vue
@@ -43,38 +43,41 @@
             <user-card-content :user="status.user"></user-card-content>
           </div>
           <div class="user-content">
-            <h4 class="media-heading">
-              {{status.user.name}}
-              <small><router-link :to="{ name: 'user-profile', params: { id: status.user.id } }">{{status.user.screen_name}}</router-link></small>
-              <small v-if="status.in_reply_to_screen_name"> &gt;
-                <router-link :to="{ name: 'user-profile', params: { id: status.in_reply_to_user_id } }">
-                  {{status.in_reply_to_screen_name}}
-                </router-link>
-              </small>
-              <template v-if="isReply && !expandable">
-                <small>
-                  <a href="#" @click.prevent="gotoOriginal" ><i class="icon-reply"></i></a>
+            <div class="media-heading">
+              <div class="name-and-links">
+                <h4>{{status.user.name}}</h4>
+                <div class="links">
+                  <h4>
+                  <small><router-link :to="{ name: 'user-profile', params: { id: status.user.id } }">{{status.user.screen_name}}</router-link></small>
+                  <small v-if="status.in_reply_to_screen_name"> &gt;
+                    <router-link :to="{ name: 'user-profile', params: { id: status.in_reply_to_user_id } }">
+                      {{status.in_reply_to_screen_name}}
+                    </router-link>
+                  </small>
+                  <template v-if="isReply && !expandable">
+                    <small>
+                      <a href="#" @click.prevent="gotoOriginal" ><i class="icon-reply"></i></a>
+                    </small>
+                  </template>
+                  -
+                  <small>
+                    <router-link :to="{ name: 'conversation', params: { id: status.id } }">
+                      <timeago :since="status.created_at" :auto-update="60" style="opacity:0.8;"></timeago>
+                    </router-link>
+                  </small>
+                  </h4>
+                </div>
+              </div>
+              <div class="heading-icons">
+                <small v-if="unmuted">
+                  <a href="#" @click.prevent="toggleMute" ><i class="icon-eye-off"></i></a>
                 </small>
-              </template>
-              -
-              <small>
-                <router-link :to="{ name: 'conversation', params: { id: status.id } }">
-                  <timeago :since="status.created_at" :auto-update="60"></timeago>
-                </router-link>
-              </small>
-              <template v-if="expandable">
-                -
-                <small>
-                  <a href="#" @click.prevent="toggleExpanded" ><i class="icon-plus-squared"></i></a>
-                </small>
-              </template>
-              <small v-if="unmuted">
-                <a href="#" @click.prevent="toggleMute" ><i class="icon-eye-off"></i></a>
-              </small>
-              <small v-if="!status.is_local" class="source_url">
-                <a :href="status.external_url" target="_blank" ><i class="icon-binoculars"></i></a>
-              </small>
-            </h4>
+                <a :href="status.external_url" target="_blank" v-if="!status.is_local" class="source_url"><i class="fa icon-binoculars"></i></a>
+                <template v-if="expandable">
+                  <a href="#" @click.prevent="toggleExpanded" class="expand"><i class="fa icon-plus-squared"></i></a>
+                </template>
+              </div>
+            </div>
 
             <div @click.prevent="linkClicked" class="status-content" v-html="status.statusnet_html"></div>
 
@@ -110,20 +113,66 @@
 
 <style lang="scss">
  @import '../../_variables.scss';
+
+ status-text-container {
+    display: block;
+}
+
  .status-el {
      hyphens: auto;
      overflow-wrap: break-word;
      word-wrap: break-word;
      word-break: break-word;
      border-left-width: 0px;
+     line-height: 18px;
+
+     .notify {
+         .avatar {
+             border-width: 3px;
+             border-style: solid;
+         }
+     }
+
+     .media-body {
+       flex: 1;
+       padding-left: 0.5em;
+     }
+
 
      .user-content {
+
        min-height: 52px;
        padding-top: 1px;
      }
 
+     .media-heading {
+       display: flex;
+       min-height: 1.4em;
+       margin-bottom: 0.3em;
+
+       small {
+           font-weight: lighter;
+       }
+       h4 {
+         margin-right: 0.4em;
+       }
+       .name-and-links {
+         flex: 1 0;
+         display: flex;
+         flex-wrap: wrap;
+         .links {
+
+         }
+       }
+     }
+
      .source_url {
-       float: right;
+       margin-right: -0.25em;
+     }
+
+     .expand {
+       margin-left: 0.5em;
+       margin-right: -0.25em;
      }
 
      .greentext {
@@ -144,6 +193,30 @@
          margin-top: 0.2em;
          margin-bottom: 0.5em;
      }
+
+     .media-left {
+         img {
+             margin-top: 0.2em;
+             float: right;
+             margin-right: 0.3em;
+             border-radius: 5px;
+         }
+     }
+
+     .retweet-info {
+         padding: 0.7em 0 0 0.6em;
+
+         .media-left {
+             display: flex;
+
+             i {
+                 align-self: center;
+                 text-align: right;
+                 flex: 1;
+                 padding-right: 0.3em;
+             }
+         }
+     }
  }
 
  .status-conversation {
@@ -151,7 +224,14 @@
  }
 
  .status-actions {
-     padding-top: 5px;
+     padding-top: 0.15em;
+     width: 100%;
+     display: flex;
+
+     div, favorite-button {
+        max-width: 5.5em;
+        flex: 1;
+     }
  }
 
  .icon-reply:hover {
@@ -187,12 +267,13 @@
  }
 
  .status {
-     padding: 0.65em 0.7em 0.8em 0.8em;
+     padding: 0.4em 0.7em 0.45em 0.7em;
      border-bottom: 1px solid;
      border-bottom-color: inherit;
      border-left: 4px rgba(255, 48, 16, 0.65);
      border-left-style: inherit;
  }
+
  .muted {
    padding: 0.1em 0.7em 0.1em 0.8em;
    button {

From 807c0d1464c97319800d59edd945125d4fe2c194 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Thu, 1 Jun 2017 01:11:57 +0300
Subject: [PATCH 2/5] Remove silly opacity from timestamp, make unmute icon
 line up with the rest better.

---
 src/components/status/status.vue | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/src/components/status/status.vue b/src/components/status/status.vue
index 8d776307..a79634dd 100644
--- a/src/components/status/status.vue
+++ b/src/components/status/status.vue
@@ -62,16 +62,14 @@
                   -
                   <small>
                     <router-link :to="{ name: 'conversation', params: { id: status.id } }">
-                      <timeago :since="status.created_at" :auto-update="60" style="opacity:0.8;"></timeago>
+                      <timeago :since="status.created_at" :auto-update="60"></timeago>
                     </router-link>
                   </small>
                   </h4>
                 </div>
               </div>
               <div class="heading-icons">
-                <small v-if="unmuted">
-                  <a href="#" @click.prevent="toggleMute" ><i class="icon-eye-off"></i></a>
-                </small>
+                <a href="#" @click.prevent="toggleMute" v-if="unmuted"><i class="fa icon-eye-off"></i></a>
                 <a :href="status.external_url" target="_blank" v-if="!status.is_local" class="source_url"><i class="fa icon-binoculars"></i></a>
                 <template v-if="expandable">
                   <a href="#" @click.prevent="toggleExpanded" class="expand"><i class="fa icon-plus-squared"></i></a>
@@ -167,12 +165,11 @@
      }
 
      .source_url {
-       margin-right: -0.25em;
+
      }
 
      .expand {
-       margin-left: 0.5em;
-       margin-right: -0.25em;
+       margin-right: -0.3em;
      }
 
      .greentext {

From d063607b9d5e543004404d1ec6a399c26e93f4d6 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Thu, 1 Jun 2017 17:35:00 +0300
Subject: [PATCH 3/5] Mobile optimizations.

---
 src/components/status/status.vue | 50 ++++++++++++++++++++++++++++++--
 1 file changed, 47 insertions(+), 3 deletions(-)

diff --git a/src/components/status/status.vue b/src/components/status/status.vue
index a79634dd..9b17dfff 100644
--- a/src/components/status/status.vue
+++ b/src/components/status/status.vue
@@ -45,7 +45,7 @@
           <div class="user-content">
             <div class="media-heading">
               <div class="name-and-links">
-                <h4>{{status.user.name}}</h4>
+                <h4 class="user-name">{{status.user.name}}</h4>
                 <div class="links">
                   <h4>
                   <small><router-link :to="{ name: 'user-profile', params: { id: status.user.id } }">{{status.user.screen_name}}</router-link></small>
@@ -159,7 +159,7 @@
          display: flex;
          flex-wrap: wrap;
          .links {
-
+           margin-right: -4em;
          }
        }
      }
@@ -226,7 +226,7 @@
      display: flex;
 
      div, favorite-button {
-        max-width: 5.5em;
+        max-width: 6em;
         flex: 1;
      }
  }
@@ -304,4 +304,48 @@
    flex: 1;
  }
 
+ @media all and (max-width: 960px) {
+   .status-el {
+     .name-and-links {
+       margin-left: -0.25em;
+
+       .user-name {
+         flex: 100%;
+       }
+     }
+
+     .status-content {
+       margin-top: 0.5em;
+       margin-left: -3.7em;
+       margin-right: 0;
+     }
+     .status-actions {
+       margin-left: -3.7em;
+     }
+   }
+   .status {
+     max-width: 100%;
+   }
+
+   .status .avatar {
+     width: 40px;
+     height: 40px;
+
+     &.retweeted {
+       width: 34px;
+       height: 34px;
+       margin-right: 8px;
+       margin-bottom: 8px;
+     }
+   }
+
+   .status img.avatar-retweeter {
+     width: 22px;
+     height: 22px;
+     position: absolute;
+     margin-left: 18px;
+     margin-top: 18px;
+   }
+ }
+
 </style>

From 52e6b470be3420178bee02844fcfd8968f2797e7 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Thu, 1 Jun 2017 17:43:37 +0300
Subject: [PATCH 4/5] Revert some of those mobile optimizations, breaks mention
 notifs.

---
 src/components/status/status.vue | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/src/components/status/status.vue b/src/components/status/status.vue
index 9b17dfff..8fb8d894 100644
--- a/src/components/status/status.vue
+++ b/src/components/status/status.vue
@@ -308,19 +308,6 @@
    .status-el {
      .name-and-links {
        margin-left: -0.25em;
-
-       .user-name {
-         flex: 100%;
-       }
-     }
-
-     .status-content {
-       margin-top: 0.5em;
-       margin-left: -3.7em;
-       margin-right: 0;
-     }
-     .status-actions {
-       margin-left: -3.7em;
      }
    }
    .status {

From c149330e50fd54ba875c430c4854c9ced0c86b10 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Thu, 1 Jun 2017 17:50:09 +0300
Subject: [PATCH 5/5] Fix bugginess with heading links overlapping icons on the
 right.

---
 src/components/status/status.vue | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/components/status/status.vue b/src/components/status/status.vue
index 8fb8d894..5d0a50d3 100644
--- a/src/components/status/status.vue
+++ b/src/components/status/status.vue
@@ -158,9 +158,6 @@
          flex: 1 0;
          display: flex;
          flex-wrap: wrap;
-         .links {
-           margin-right: -4em;
-         }
        }
      }