<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1240#discussion_r67926640">app/views/layouts/_banner.html.erb</a>:</p>
<pre style='color:#555'>> + end
> +
> + if val.to_i <= queuePos
> + bannerSym = k
> + cookieStr = c
> + queuePos = val.to_i
> + end
> +end
> +
> +unless bannerSym.nil?
> + banner = active_banners[bannerSym]
> + cookies[cookieStr.to_sym] = active_banners.length # bump to end of queue
> +%>
> +<%= link_to (image_tag banner[:img], :alt => banner[:alt], :title => banner[:alt]), banner[:link] %>
> +<div class="close-wrap" id="<%= cookieStr %>"><span class="icon close"></span></div>
> +<% end %>
</pre>
<p>Sure I can put most of it into a helper. For the config of the banners - would we want to localize the alt text? If so maybe storing it in /config as yml would make it inconvenient.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1240/files/933335a0ea28f274bc7301c18611bcd90709fd99#r67926640">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/ABWnLQDUVSVLAiekAYsp6XA-1bjjfiwhks5qOC9ZgaJpZM4I6MD6">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLdnB5FFbL6CO58GnUl1pXcpe6-CRks5qOC9ZgaJpZM4I6MD6.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/openstreetmap/openstreetmap-website/pull/1240/files/933335a0ea28f274bc7301c18611bcd90709fd99#r67926640"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>