gravatar_image_tag 1.1.0 → 1.1.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (3) hide show
  1. data/lib/gravatar_image_tag.rb +1 -1
  2. data/spec/gravatar_image_tag_spec.rb +16 -2
  3. metadata +13 -13
data/lib/gravatar_image_tag.rb CHANGED
@@ -66,7 +66,7 @@ module GravatarImageTag
66
66
  gravatar_overrides = options.delete(:gravatar)
67
67
  options[:src] = GravatarImageTag::gravatar_url(email, gravatar_overrides)
68
68
  options[:alt] ||= 'Gravatar'
69
- options[:height] = options[:width] = "#{GravatarImageTag::gravatar_options(gravatar_overrides)[:size]}px" if GravatarImageTag.configuration.include_size_attributes
69
+ options[:height] = options[:width] = "#{GravatarImageTag::gravatar_options(gravatar_overrides)[:size] || 80}px" if GravatarImageTag.configuration.include_size_attributes
70
70
  tag 'img', options, false, false # Patch submitted to rails to allow image_tag here https://rails.lighthouseapp.com/projects/8994/tickets/2878-image_tag-doesnt-allow-escape-false-option-anymore
71
71
  end
72
72
 
data/spec/gravatar_image_tag_spec.rb CHANGED
@@ -76,10 +76,24 @@ describe GravatarImageTag do
76
76
  end
77
77
 
78
78
  it 'should set the image tags height and width to avoid the page going all jiggy (technical term) when loading a page with lots of Gravatars' do
79
- (!!view.gravatar_image_tag(email).match(/height="50px"/)).should be_true
80
- (!!view.gravatar_image_tag(email).match(/width="50px"/)).should be_true
79
+ GravatarImageTag.configure { |c| c.size = 30 }
80
+ (!!view.gravatar_image_tag(email).match(/height="30px"/)).should be_true
81
+ (!!view.gravatar_image_tag(email).match(/width="30px"/)).should be_true
81
82
  end
82
83
 
84
+ it 'should set the image tags height and width attributes to 80px (gravatars default) if no size is given.' do
85
+ GravatarImageTag.configure { |c| c.size = nil }
86
+ (!!view.gravatar_image_tag(email).match(/height="80px"/)).should be_true
87
+ (!!view.gravatar_image_tag(email).match(/width="80px"/)).should be_true
88
+ end
89
+
90
+ it 'should set the image tags height and width attributes from the overrides on the size' do
91
+ GravatarImageTag.configure { |c| c.size = 120 }
92
+ (!!view.gravatar_image_tag(email, :gravatar => { :size => 45 }).match(/height="45px"/)).should be_true
93
+ (!!view.gravatar_image_tag(email, :gravatar => { :size => 75 }).match(/width="75px"/)).should be_true
94
+ end
95
+
96
+
83
97
  it 'should not include the height and width attributes on the image tag if it is turned off in the configuration' do
84
98
  GravatarImageTag.configure { |c| c.include_size_attributes = false }
85
99
  (!!view.gravatar_image_tag(email).match(/height=/)).should be_false
metadata CHANGED
@@ -1,7 +1,7 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: gravatar_image_tag
3
3
  version: !ruby/object:Gem::Version
4
- version: 1.1.0
4
+ version: 1.1.1
5
5
  prerelease:
6
6
  platform: ruby
7
7
  authors:
@@ -13,7 +13,7 @@ date: 2012-04-26 00:00:00.000000000 Z
13
13
  dependencies:
14
14
  - !ruby/object:Gem::Dependency
15
15
  name: activesupport
16
- requirement: &70174494367000 !ruby/object:Gem::Requirement
16
+ requirement: &70268241278300 !ruby/object:Gem::Requirement
17
17
  none: false
18
18
  requirements:
19
19
  - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
21
21
  version: '0'
22
22
  type: :development
23
23
  prerelease: false
24
- version_requirements: *70174494367000
24
+ version_requirements: *70268241278300
25
25
  - !ruby/object:Gem::Dependency
26
26
  name: actionpack
27
- requirement: &70174494366520 !ruby/object:Gem::Requirement
27
+ requirement: &70268241277300 !ruby/object:Gem::Requirement
28
28
  none: false
29
29
  requirements:
30
30
  - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
32
32
  version: '0'
33
33
  type: :development
34
34
  prerelease: false
35
- version_requirements: *70174494366520
35
+ version_requirements: *70268241277300
36
36
  - !ruby/object:Gem::Dependency
37
37
  name: guard
38
- requirement: &70174494366040 !ruby/object:Gem::Requirement
38
+ requirement: &70268241276200 !ruby/object:Gem::Requirement
39
39
  none: false
40
40
  requirements:
41
41
  - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
43
43
  version: '0'
44
44
  type: :development
45
45
  prerelease: false
46
- version_requirements: *70174494366040
46
+ version_requirements: *70268241276200
47
47
  - !ruby/object:Gem::Dependency
48
48
  name: guard-rspec
49
- requirement: &70174494365460 !ruby/object:Gem::Requirement
49
+ requirement: &70268241274900 !ruby/object:Gem::Requirement
50
50
  none: false
51
51
  requirements:
52
52
  - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
54
54
  version: '0'
55
55
  type: :development
56
56
  prerelease: false
57
- version_requirements: *70174494365460
57
+ version_requirements: *70268241274900
58
58
  - !ruby/object:Gem::Dependency
59
59
  name: jeweler
60
- requirement: &70174494364980 !ruby/object:Gem::Requirement
60
+ requirement: &70268241273980 !ruby/object:Gem::Requirement
61
61
  none: false
62
62
  requirements:
63
63
  - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
65
65
  version: '0'
66
66
  type: :development
67
67
  prerelease: false
68
- version_requirements: *70174494364980
68
+ version_requirements: *70268241273980
69
69
  - !ruby/object:Gem::Dependency
70
70
  name: rspec
71
- requirement: &70174494364400 !ruby/object:Gem::Requirement
71
+ requirement: &70268241272620 !ruby/object:Gem::Requirement
72
72
  none: false
73
73
  requirements:
74
74
  - - ! '>='
@@ -76,7 +76,7 @@ dependencies:
76
76
  version: '0'
77
77
  type: :development
78
78
  prerelease: false
79
- version_requirements: *70174494364400
79
+ version_requirements: *70268241272620
80
80
  description:
81
81
  email: mdeering@mdeering.com
82
82
  executables: []