pugin 0.8.8 → 0.8.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2463a6b85e377d1cd2e02188c6c62154a0a4089b
-  data.tar.gz: 26ad945fca8fd366742174d70d942a1ee9be45ea
+  metadata.gz: 90158b803120cb366be6df4c1d010ed4f7e80249
+  data.tar.gz: ba063200603d98c648cc533eb2200ce3f472e65c
 SHA512:
-  metadata.gz: a20dc06f48d7284661cecaf6c2e3a58753a883226dd534447dccf88641b9c5968aef80a978643b3d6fe0d53409ac579f88c7ac391437269a6a2a7d237c925b43
-  data.tar.gz: 7899f161c3e97de17f6eefc73a5b8c1c8d8e01a0b6b8a6e67ccc70166fab8eb6e0f33cbe4c5ef1cd84ac386ff77e3cc2db0e1ea34e12a598c0bb09b2a3c7c8f5
+  metadata.gz: fcb9a801b71c8754efa4e10d8e4c6805dbc7953cd93aa76d80e45e5aa5d0c2942273c317e3b0c17300bb4f287c11e93bfb2a4204fb7e9365171050b3447b1eef
+  data.tar.gz: 4ff0f37907d3a3e08a0c6cb3e40ed011b25c197f702ff244fe5bbbccd13f73f07ecdebf5f9f8b8fa25a16dab6b3fce4e1fa05449eb86e26a44aada19de99299b

data/app/views/pugin/people/list/lord/_lord.html.haml

@@ -1,16 +1,17 @@
 - current_incumbency = lord.house_incumbencies.sort{ |inc_a, inc_b| inc_b.start_date <=> inc_a.start_date }.first
 - if Pugin::Feature::Bandiera.show_list_images?
-  - if lord.image_id != "placeholder"
-    %img{:src => "#{ENV['IMAGE_SERVICE_URL']}/#{lord.image_id}.jpg?crop=CU_1:1&height=93", :alt => "#{lord.display_name}"}
-  - else
-    %img{:src => "https://s3-eu-west-1.amazonaws.com/web1live.pugin-website/images/uk_parliament_placeholder_list_image.png", :alt => "placeholder"}
+  %figure
+    - if lord.image_id != "placeholder"
+      %img{src: "#{ENV['IMAGE_SERVICE_URL']}/#{lord.image_id}.jpeg?crop=CU_1:1&width=186&quality=100", alt: "#{lord.display_name}"}
+    - else
+      %img{src: "#{Pugin::ASSET_LOCATION_URL}/#{Pugin::ASSET_VERSION}/images/placeholder_members_image.png", alt: "placeholder"}
 
 .list--details
   - if !instance_variable_get("@parliament").nil? # parliament lord
 
   - elsif current_incumbency.current? # current lord
     %h2
-      %a{:href => "/people/#{lord.graph_id}"}= lord.display_name
+      %a{href: "/people/#{lord.graph_id}"}= lord.display_name
     - unless lord.statuses[:house_membership_status].empty? && !lord.statuses[:house_membership_status].include?('Current MP')
       %p= lord.statuses[:house_membership_status].join(' and ')
     - unless lord.parties.empty?
@@ -20,6 +21,6 @@
 
   - else # former lord
     %h2
-      %a{:href => "/people/#{lord.graph_id}"}= lord.display_name
+      %a{href: "/people/#{lord.graph_id}"}= lord.display_name
     - unless lord.statuses[:house_membership_status].empty? && !lord.statuses[:house_membership_status].include?('Current MP')
       %p= lord.statuses[:house_membership_status].join(' and ')

data/app/views/pugin/people/list/member/_member.html.haml

@@ -1,14 +1,16 @@
 - current_incumbency = member.seat_incumbencies.select(&:current?).first
 
 - if Pugin::Feature::Bandiera.show_list_images?
-  - if member.image_id != "placeholder"
-    %img{:src => "#{ENV['IMAGE_SERVICE_URL']}/#{member.image_id}.jpg?crop=CU_1:1&height=93", :alt => "#{member.display_name}"}
-  - else
-    %img{:src => "https://s3-eu-west-1.amazonaws.com/web1live.pugin-website/images/uk_parliament_placeholder_list_image.png", :alt => "placeholder"}
+  %figure
+    - if member.image_id != "placeholder"
+      %img{src: "#{ENV['IMAGE_SERVICE_URL']}/#{member.image_id}.jpeg?crop=CU_1:1&width=186&quality=100", alt: "#{member.display_name}"}
+    - else
+      %img{src: "#{Pugin::ASSET_LOCATION_URL}/#{Pugin::ASSET_VERSION}/images/placeholder_members_image.png", alt: "placeholder"}
+        
 .list--details
   - if !instance_variable_get("@parliament").nil? # parliament member
     %h2
-      %a{:href => "/people/#{member.graph_id}"}= member.display_name
+      %a{href: "/people/#{member.graph_id}"}= member.display_name
     - unless member.constituencies.empty?
       - unless member.seat_incumbencies.nil?
         - member.seat_incumbencies.each do |seat_incumbency|
@@ -20,7 +22,7 @@
 
   - elsif !current_incumbency.nil? # current member
     %h2
-      %a{:href => "/people/#{member.graph_id}"}= member.display_name
+      %a{href: "/people/#{member.graph_id}"}= member.display_name
     - unless member.constituencies.empty?
       - seat_incumbency = member.seat_incumbencies.select(&:current?).first
       - unless seat_incumbency.nil?
@@ -33,7 +35,6 @@
   - else # former member
 
     %h2
-      %a{:href => "/people/#{member.graph_id}"}= member.display_name
+      %a{href: "/people/#{member.graph_id}"}= member.display_name
     - unless member.statuses[:house_membership_status].empty? && !member.statuses[:house_membership_status].include?('Current MP')
-      %p= member.statuses[:house_membership_status].join(' and ')
-      
+      %p= member.statuses[:house_membership_status].join(' and ')

data/lib/pugin.rb

@@ -1,7 +1,7 @@
 require 'pugin/version'
 
 module Pugin
-  ASSET_VERSION = '1.5.2'.freeze
+  ASSET_VERSION = '1.5.3'.freeze
   ASSET_LOCATION_URL = ENV.fetch('ASSET_LOCATION_URL', 'https://s3-eu-west-1.amazonaws.com/web1devci.pugin-website')
   STATIC_ASSET_LOCATION_URL = ENV.fetch('STATIC_ASSET_LOCATION_URL', 'https://s3-eu-west-1.amazonaws.com/web1devci.static-assets')
   class << self

data/lib/pugin/version.rb

@@ -1,3 +1,3 @@
 module Pugin
-  VERSION = '0.8.8'.freeze
+  VERSION = '0.8.9'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pugin
 version: !ruby/object:Gem::Version
-  version: 0.8.8
+  version: 0.8.9
 platform: ruby
 authors:
 - Matt Rayner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-17 00:00:00.000000000 Z
+date: 2017-07-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails