<p>This helper generates classes like <code>.hide_unless_logged_in</code> and <code>.hide_unless_administrator</code>, which are in turn relied on by helpers like <code>if_logged_in</code> and <code>if_administrator</code>. Continuing down the dependency chain, these helpers are used by the following templates:</p>

<ul>
<li>notes/show.jst.ejs</li>
<li>diary_entry/list.html.erb </li>
<li>diary_entry/view.html.erb</li>
<li>diary_entry/_diary_comment.html.erb </li>
<li>diary_entry/_diary_entry.html.erb </li>
<li>trace/list.html.erb </li>
<li>trace/view.html.erb</li>
</ul><p>It looks like action caching for diary entries and traces was disabled in <a href="https://github.com/openstreetmap/openstreetmap-website/commit/4c9a3d4803e4a30e157b3b179f630e7f51f38625" class="commit-link"><tt>4c9a3d4</tt></a>. Was this important functionality?</p>

<p>The notes template could be changed to use some template variables to determine whether or not to show the hide and close buttons. The variable value would come from data attributes -- e.g. instead of <code>.hide_if_logged_in { display: none !important }</code>, emit <code>data-logged-in=true</code>.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/openstreetmap/openstreetmap-website/issues/79#issuecomment-19431000'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/uTRSc6ihLa7Shf84BpiOpuSBf_md-PaNRdmqzHGAAX2YIztkkUer-qBjU_GlRcB-.gif' height='1' width='1'></p>