szurubooru/client/html/expander.tpl

<section class='expander'>
    <header>
        <a href>
            <%- ctx.title %>
            <i class='fa fa-chevron-down'></i>
        </a>
    </header>

    <div class='expander-content'></div>
</section>
client/expanders: extract HTML 2016-08-22 00:53:53 +02:00			`<section class='expander'>`
			`<header>`
client/general: add empty href for link buttons In e464e69 I removed href='#' but I noticed that it broke some things. Readding href serves two purposes: - it makes links reachable with Tab key - it makes links clickable with Enter key The alternative to this approach was to introduce [tabindex] and [role] attributes. But not only using tabindex=0 with <a/> is questionable, it'd require adding a keyboard handler that'd intercept space and return key presses and simulated link clicks. Since it's best to leave this kind of thing to the native UI, I went with readding hrefs instead. I believe that hash hrefs, even though being a common practice, are silly, so I decided to settle down with empty hrefs. As a bonus, I added a snippet that prevents middle mouse clicks from opening such links/buttons in new tabs, which was the motivation for e464e69. 2016-08-22 01:25:10 +02:00			`<a href>`
client/expanders: extract HTML 2016-08-22 00:53:53 +02:00			`<%- ctx.title %>`
			`<i class='fa fa-chevron-down'></i>`
			`</a>`
			`</header>`

			`<div class='expander-content'></div>`
			`</section>`