From f1eaee46742c31c74bd73f40a98591f79f5ab0eb Mon Sep 17 00:00:00 2001
From: Nolan Lawson <nolan@nolanlawson.com>
Date: Wed, 31 Jan 2018 18:20:30 -0800
Subject: [PATCH] implement autoplay gif feature for avatars

---
 routes/_components/AccountProfile.html       | 20 ++++-----
 routes/_components/NonAutoplayImg.html       | 38 +++++++++++++++++
 routes/_components/status/Avatar.html        | 33 ++++++++++++---
 routes/_components/status/Status.html        |  2 +-
 routes/_store/store.js                       |  6 ++-
 routes/_utils/events.js                      | 17 ++++++++
 routes/settings/_components/SettingsNav.html |  1 +
 routes/settings/general.html                 | 44 ++++++++++++++++++++
 routes/settings/index.html                   |  1 +
 9 files changed, 142 insertions(+), 20 deletions(-)
 create mode 100644 routes/_components/NonAutoplayImg.html
 create mode 100644 routes/settings/general.html

diff --git a/routes/_components/AccountProfile.html b/routes/_components/AccountProfile.html
index b976a82e..43332a3d 100644
--- a/routes/_components/AccountProfile.html
+++ b/routes/_components/AccountProfile.html
@@ -1,9 +1,10 @@
-<div class="account-profile {{headerIsMissing ? 'header-is-missing' : ''}}" style="background-image: url({{profile.header}});">
+<div class="account-profile {{headerIsMissing ? 'header-is-missing' : ''}}"
+     style="background-image: url({{$autoplayGifs ? profile.header : profile.header_static}});">
   <div class="account-profile-grid-wrapper">
     <div class="account-profile-backdrop"></div>
     <div class="account-profile-grid">
       <div class="account-profile-avatar">
-        <img src="{{profile.avatar}}" aria-hidden="true">
+        <Avatar account="{{profile}}" size="big" />
       </div>
       <div class="account-profile-name">
         <ExternalLink href="{{profile.url}}">
@@ -101,11 +102,6 @@
   .account-profile-avatar {
     grid-area: avatar;
   }
-  .account-profile-avatar img {
-    border-radius: 4px;
-    width: 100px;
-    height: 100px;
-  }
 
   .account-profile-username {
     grid-area: username;
@@ -159,10 +155,6 @@
   }
 
   @media (max-width: 767px) {
-    .account-profile-avatar img {
-      width: 80px;
-      height: 80px;
-    }
     .account-profile-name {
       font-size: 1.3em;
     }
@@ -190,14 +182,18 @@
 <script>
   import IconButton from './IconButton.html'
   import ExternalLink from './ExternalLink.html'
+  import Avatar from './Status/Avatar.html'
+  import { store } from '../_store/store'
 
   export default {
     computed: {
       headerIsMissing: (profile) => profile.header.endsWith('missing.png')
     },
+    store: () => store,
     components: {
       IconButton,
-      ExternalLink
+      ExternalLink,
+      Avatar
     }
   }
 </script>
\ No newline at end of file
diff --git a/routes/_components/NonAutoplayImg.html b/routes/_components/NonAutoplayImg.html
new file mode 100644
index 00000000..7da4482c
--- /dev/null
+++ b/routes/_components/NonAutoplayImg.html
@@ -0,0 +1,38 @@
+{{#if staticSrc === src}}
+  <img class="{{className || ''}}"
+       aria-hidden="{{ariaHidden}}"
+       alt="{{alt}}"
+       src="{{src}}"
+       on:imgLoadError />
+{{else}}
+  <img class="{{className || ''}} non-autoplay-zoom-in"
+       aria-hidden="{{ariaHidden}}"
+       alt="{{alt}}"
+       src="{{staticSrc}}"
+       on:imgLoadError
+       on:mouseover="onMouseOver(event)"
+       ref:node />
+{{/if}}
+<style>
+  .non-autoplay-zoom-in {
+    cursor: zoom-in;
+  }
+</style>
+<script>
+  import { imgLoadError, mouseover } from '../_utils/events'
+  export default {
+    methods: {
+      onMouseOver(mouseOver) {
+        if (mouseOver) {
+          this.refs.node.src = this.get('src')
+        } else {
+          this.refs.node.src = this.get('staticSrc')
+        }
+      }
+    },
+    events: {
+      imgLoadError,
+      mouseover
+    }
+  }
+</script>
\ No newline at end of file
diff --git a/routes/_components/status/Avatar.html b/routes/_components/status/Avatar.html
index e88f8eb1..b531120b 100644
--- a/routes/_components/status/Avatar.html
+++ b/routes/_components/status/Avatar.html
@@ -1,16 +1,33 @@
 {{#if error}}
-<svg class="{{className}} avatar" aria-hidden="true">
+<svg class="{{className}} avatar size-{{size}}" aria-hidden="true">
   <use xlink:href="#fa-user" />
 </svg>
-{{else}}
-<img class="{{className}} avatar" aria-hidden="true" alt=""
+{{elseif $autoplayGifs}}
+  <img class="{{className}} avatar size-{{size}}" aria-hidden="true" alt=""
      src="{{account.avatar}}" on:imgLoadError="set({error: true})" />
+{{else}}
+  <NonAutoplayImg className="{{className}} avatar size-{{size}}" ariaHidden="true" alt=""
+       src="{{account.avatar}}" staticSrc="{{account.avatar_static}}" on:imgLoadError="set({error: true})" />
 {{/if}}
 <style>
-  .avatar {
+  :global(.avatar) {
+    border-radius: 4px;
+  }
+  :global(.avatar.size-small) {
     width: 48px;
     height: 48px;
-    border-radius: 4px;
+  }
+
+  :global(.avatar.size-big) {
+    width: 100px;
+    height: 100px;
+  }
+
+  @media (max-width: 767px) {
+    :global(.avatar.size-big) {
+      width: 80px;
+      height: 80px;
+    }
   }
 
   svg.avatar {
@@ -19,10 +36,16 @@
 </style>
 <script>
   import { imgLoadError } from '../../_utils/events'
+  import { store } from '../../_store/store'
+  import NonAutoplayImg from '../NonAutoplayImg.html'
 
   export default {
     events: {
       imgLoadError
+    },
+    store: () => store,
+    components: {
+      NonAutoplayImg
     }
   }
 </script>
\ No newline at end of file
diff --git a/routes/_components/status/Status.html b/routes/_components/status/Status.html
index 71e4f997..8450cbd7 100644
--- a/routes/_components/status/Status.html
+++ b/routes/_components/status/Status.html
@@ -30,7 +30,7 @@
       </a>
     {{/if}}
   </div>
-  <Avatar account={{originalAccount}} className="status-sidebar"/>
+  <Avatar account={{originalAccount}} className="status-sidebar" size="small" />
   {{#if originalStatus.spoiler_text}}
     <div class="status-spoiler">{{originalStatus.spoiler_text}}</div>
   {{/if}}
diff --git a/routes/_store/store.js b/routes/_store/store.js
index 05fae157..881e1b3c 100644
--- a/routes/_store/store.js
+++ b/routes/_store/store.js
@@ -10,7 +10,8 @@ const KEYS_TO_STORE_IN_LOCAL_STORAGE = new Set([
   "instanceNameInSearch",
   "instanceThemes",
   "loggedInInstances",
-  "loggedInInstancesInOrder"
+  "loggedInInstancesInOrder",
+  "autoplayGifs"
 ])
 
 class PinaforeStore extends LocalStorageStore {
@@ -24,7 +25,8 @@ const store = new PinaforeStore({
   currentInstance: null,
   loggedInInstances: {},
   loggedInInstancesInOrder: [],
-  instanceThemes: {}
+  instanceThemes: {},
+  autoplayGifs: false
 })
 
 mixins(PinaforeStore)
diff --git a/routes/_utils/events.js b/routes/_utils/events.js
index c25266ca..6a4cf5ad 100644
--- a/routes/_utils/events.js
+++ b/routes/_utils/events.js
@@ -16,4 +16,21 @@ export function imgLoad (node, callback) {
       node.removeEventListener('load', callback)
     }
   }
+}
+
+export function mouseover(node, callback) {
+  function onMouseEnter() {
+    callback(true)
+  }
+  function onMouseLeave() {
+    callback(false)
+  }
+  node.addEventListener('mouseenter', onMouseEnter)
+  node.addEventListener('mouseleave', onMouseLeave)
+  return {
+    teardown () {
+      node.removeEventListener('mouseenter', onMouseEnter)
+      node.removeEventListener('mouseleave', onMouseLeave)
+    }
+  }
 }
\ No newline at end of file
diff --git a/routes/settings/_components/SettingsNav.html b/routes/settings/_components/SettingsNav.html
index f6f9cf04..6fbb0af0 100644
--- a/routes/settings/_components/SettingsNav.html
+++ b/routes/settings/_components/SettingsNav.html
@@ -43,6 +43,7 @@
   const NAV_ITEMS = {
     'settings': 'Settings',
     'settings/about': 'About Pinafore',
+    'settings/general': 'General',
     'settings/instances': 'Instances',
     'settings/instances/add': 'Add an instance',
   }
diff --git a/routes/settings/general.html b/routes/settings/general.html
new file mode 100644
index 00000000..3aa90278
--- /dev/null
+++ b/routes/settings/general.html
@@ -0,0 +1,44 @@
+<:Head>
+  <title>General Settings</title>
+</:Head>
+
+<Layout page='settings'>
+  <SettingsLayout page='settings/general' label="General">
+    <h1>General Settings</h1>
+
+    <h2>UI</h2>
+    <form class="ui-settings" aria-label="UI settings">
+      <div class="setting-group">
+        <input type="checkbox" id="choice-autoplay-gif"
+               bind:checked="$autoplayGifs" on:change="store.save()">
+        <label for="choice-autoplay-gif">Autoplay GIFs</label>
+      </div>
+    </form>
+
+  </SettingsLayout>
+</Layout>
+
+<style>
+  .ui-settings {
+    display: flex;
+    align-items: center;
+    background: var(--form-bg);
+    border: 1px solid var(--main-border);
+    border-radius: 4px;
+    padding: 20px;
+    line-height: 2em;
+  }
+</style>
+<script>
+  import Layout from '../_components/Layout.html';
+  import SettingsLayout from './_components/SettingsLayout.html'
+  import { store } from '../_store/store'
+
+  export default {
+    components: {
+      Layout,
+      SettingsLayout
+    },
+    store: () => store
+  };
+</script>
\ No newline at end of file
diff --git a/routes/settings/index.html b/routes/settings/index.html
index 22098525..50ade4d7 100644
--- a/routes/settings/index.html
+++ b/routes/settings/index.html
@@ -7,6 +7,7 @@
     <h1>Settings</h1>
 
     <SettingsList>
+      <SettingsListItem href="/settings/general" label="General"/>
       <SettingsListItem href="/settings/instances" label="Instances"/>
       <SettingsListItem href="/settings/about" label="About Pinafore"/>
     </SettingsList>