gravatar using id

[MasseGuillaume]

widgets/src/main/scala/net/liftmodules/widgets/gravatar/Gravatar.scala

Line 67 in 9ae73ce

<div id="gravatar_wrapper"><div id="gravatar_image"><img src={in} alt="Gravater" /></div></div>

please change id="gravatar_image" to class="gravatar" as id must be unique and you can use this widget multiple times in a single page.

[d6y]

Presumably we should do the same for gravatar_wrapper too?

Perhaps an idea would be to have a method on the object to just get the image URL so anyone can use the class to create whatever kind of mark up they like.

[MasseGuillaume]

yes I think this widget should just create something like

private def html(in: String, alt: String, class: Option[String]): NodeSeq = {
    <img src={in} alt={alt} class={class} />
  }

[d6y]

If we want to change the behaviour, we should discuss that on the mailing list.

Adding additional behaviour may be nicer for anyone currently using the widget. This could do with some unit tests/specs too, to make sure we don't break compatibility.

Anyone using Gravatar/Widgets want to take this on? Otherwise, I'll get round to it at some point.