gravatar_image_tag 1.1.0 → 1.1.1

Sign up to get free protection for your applications and to get access to all the features.
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: []