haml refactor

haml refactor

Initial ugly version:

    - if @post
      %div{ class: "blog-titulo#{ ' peque' if @blog.titulo.length > 60 }" }= @blog.titulo
    - else
      %h1{ class: "blog-titulo#{ ' peque' if @blog.titulo.length > 60 }" }= @blog.titulo


Nice, but doesn't work:

    - h1_or_div = @post ? '%div' : '%h1'
      = eval(h1_or_div){ class: "blog-titulo#{ ' peque' if @blog.titulo.length > 60 }" }= @blog.titulo


Works, but breaks DRY -> Seems WET (Will Enhance This)

    - clase = { class: "blog-titulo#{ ' peque' if @blog.titulo.length > 60 }" }
    - if @post
      %div{ clase }= @blog.titulo
    - else
      %h1{ clase }= @blog.titulo


EDIT: Finally, it could be like this:

    - clase_titulo = "blog-titulo#{' peque' if  @blog.titulo.length > 60}"
    = content_tag(@post ? :div : :h1, @blog.titulo, class: clase_titulo)

This works and is (more or less) what I was looking for:

= content_tag(@post ? :div : :h1, @blog.titulo, class: "blog-titulo#{ ' peque' if @blog.titulo.length > 60 }")

or in a cleaner way:

- clase_titulo = 'blog-titulo'
- clase_titulo += ' peque' if @blog.titulo.length > 60
= content_tag(@post ? :div : :h1, @blog.titulo, class: clase_titulo)