pugin 0.8.9 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 90158b803120cb366be6df4c1d010ed4f7e80249
-  data.tar.gz: ba063200603d98c648cc533eb2200ce3f472e65c
+  metadata.gz: 3db68c37f4262bf6e04956cd1c23e47ac91b2387
+  data.tar.gz: 848d39e9671fe898ee99d799ddd54732143ec720
 SHA512:
-  metadata.gz: fcb9a801b71c8754efa4e10d8e4c6805dbc7953cd93aa76d80e45e5aa5d0c2942273c317e3b0c17300bb4f287c11e93bfb2a4204fb7e9365171050b3447b1eef
-  data.tar.gz: 4ff0f37907d3a3e08a0c6cb3e40ed011b25c197f702ff244fe5bbbccd13f73f07ecdebf5f9f8b8fa25a16dab6b3fce4e1fa05449eb86e26a44aada19de99299b
+  metadata.gz: 685144adb0ae12ada1cadf6c6affbcd4a01cb0a97e2710938e9fd1874787de99a3cec6331cd80b27bf82f7e3cdbc709c16ee63a444a0cc6a1be462485dda76ff
+  data.tar.gz: 73a6089c5a412bee168455595bd912ab9a3179f4323a635043b844bbcfec924aaaf848ea464928688f37d1408287712163409ac4363dd29e96ba33e18fc62163

data/app/views/pugin/cards/_person-list.haml

@@ -1,23 +1,25 @@
 -if people.empty?
   %ol.list--box
     %li
-      %p
-        = "#{I18n.t('no_content.empty_list.members.description')}" 
-        = params[:letter].nil? ? "" : "#{I18n.t('no_content.empty_list.members.for')} '#{params[:letter].upcase}'"
+      .list--details
+        %p
+          = "#{I18n.t('no_content.empty_list.members.description')}" 
+          = params[:letter].nil? ? "" : "#{I18n.t('no_content.empty_list.members.for')} '#{params[:letter].upcase}'"
 -else
   %ol.list--box
     - people.each do |person|
       - if person.respond_to?(:graph_id)
         %li
-          %h2
-            %a{:href => "/people/#{person.graph_id}"}= person.display_name
-          - if !person.statuses[:house_membership_status].empty? && !person.statuses[:house_membership_status].include?('Current MP')
-            %p= person.statuses[:house_membership_status].join(' and ')
-          - if !person.constituencies.empty?
-            - seat_incumbency = person.seat_incumbencies.select(&:current?).first
-            - unless seat_incumbency.nil?
-              %p= "#{I18n.t('pugin.cards.person-list.current_mp_for')} #{seat_incumbency.constituency.name}"
-          - if !person.parties.empty?
-            - party_membership = person.party_memberships.select(&:current?).first
-            - unless party_membership.nil?
-              %p= party_membership.party.name
+          .list--details
+            %h2
+              %a{:href => "/people/#{person.graph_id}"}= person.display_name
+            - if !person.statuses[:house_membership_status].empty? && !person.statuses[:house_membership_status].include?('Current MP')
+              %p= person.statuses[:house_membership_status].join(' and ')
+            - if !person.constituencies.empty?
+              - seat_incumbency = person.seat_incumbencies.select(&:current?).first
+              - unless seat_incumbency.nil?
+                %p= "#{I18n.t('pugin.cards.person-list.current_mp_for')} #{seat_incumbency.constituency.name}"
+            - if !person.parties.empty?
+              - party_membership = person.party_memberships.select(&:current?).first
+              - unless party_membership.nil?
+                %p= party_membership.party.name

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

@@ -2,9 +2,9 @@
 - if Pugin::Feature::Bandiera.show_list_images?
   %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}"}
+      %img{ src: "#{ENV['IMAGE_SERVICE_URL']}/#{lord.image_id}.jpeg?crop=CU_1:1&width=186&quality=80", alt: "#{lord.display_name}" }
     - else
-      %img{src: "#{Pugin::ASSET_LOCATION_URL}/#{Pugin::ASSET_VERSION}/images/placeholder_members_image.png", alt: "placeholder"}
+      %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

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

@@ -3,9 +3,9 @@
 - if Pugin::Feature::Bandiera.show_list_images?
   %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}"}
+      %img{ src: "#{ENV['IMAGE_SERVICE_URL']}/#{member.image_id}.jpeg?crop=CU_1:1&width=186&quality=80", alt: "#{member.display_name}" }
     - else
-      %img{src: "#{Pugin::ASSET_LOCATION_URL}/#{Pugin::ASSET_VERSION}/images/placeholder_members_image.png", alt: "placeholder"}
+      %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

data/lib/pugin.rb

@@ -1,7 +1,7 @@
 require 'pugin/version'
 
 module Pugin
-  ASSET_VERSION = '1.5.3'.freeze
+  ASSET_VERSION = '1.5.4'.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.9'.freeze
+  VERSION = '0.9.0'.freeze
 end

data/spec/views/pugin/cards/_person-list.html.haml_spec.rb

@@ -56,11 +56,13 @@ describe 'pugin/cards/_person-list.html.haml', type: :view do
 <<DATA
 <ol class='list--box'>
 <li>
+<div class='list--details'>
 <h2>
 <a href='/people/123'>Jane Smith</a>
 </h2>
 <p>Current MP for Hackney</p>
 <p>Labour</p>
+</div>
 </li>
 </ol>
 DATA
@@ -79,10 +81,12 @@ DATA
       expect(response).to eq(<<DATA
 <ol class='list--box'>
 <li>
+<div class='list--details'>
 <p>
 There are no results
 
 </p>
+</div>
 </li>
 </ol>
 DATA
@@ -158,11 +162,13 @@ DATA
 <<DATA
 <ol class='list--box'>
 <li>
+<div class='list--details'>
 <h2>
 <a href='/people/123'></a>
 </h2>
 <p>Current MP for Hackney</p>
 <p>Labour</p>
+</div>
 </li>
 </ol>
 DATA
@@ -204,10 +210,12 @@ DATA
 <<DATA
 <ol class='list--box'>
 <li>
+<div class='list--details'>
 <h2>
 <a href='/people/123'>Jane Smith</a>
 </h2>
 <p>Labour</p>
+</div>
 </li>
 </ol>
 DATA
@@ -248,10 +256,12 @@ DATA
 <<DATA
 <ol class='list--box'>
 <li>
+<div class='list--details'>
 <h2>
 <a href='/people/123'>Jane Smith</a>
 </h2>
 <p>Current MP for Hackney</p>
+</div>
 </li>
 </ol>
 DATA

metadata

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