gravatar_image_tag icon indicating copy to clipboard operation
gravatar_image_tag copied to clipboard

Invalid HTML

Open mattherick opened this issue 10 years ago • 3 comments

Hi!

This

def self.url_params(gravatar_params)
  return nil if gravatar_params.keys.size == 0
  array = gravatar_params.map { |k, v| "#{k}=#{value_cleaner(v)}" }
  "?#{array.join('&')}"
end

causes invalid html, if I run the http://validator.w3.org

Line 93, Column 122: & did not start a character reference. (& probably should have been escaped as &.)…e66872a973005f1dae67?default=identicon&secure=true&size=50" width="50" height=…
Line 93, Column 134: & did not start a character reference. (& probably should have been escaped as &.)…a973005f1dae67?default=identicon&secure=true&size=50" width="50" height="50" />

Changing the "&" to "&' would fix it..

Can you fix this, or should i provide a pull request?

I know it is just a little "error", but if all other validations pass it would be fine if also this last one pass :-)..

Matthias

mattherick avatar Jun 25 '15 16:06 mattherick

Same here.

veye avatar Jul 07 '15 20:07 veye

+1

reiz avatar Jul 07 '15 20:07 reiz

+1

veye avatar Jul 07 '15 21:07 veye