Automattic / media-explorer

With Media Explorer, you can now search for tweets and videos on Twitter and YouTube directly from the Add Media screen in WordPress.
126 stars 30 forks source link

use button for pagination link #67

Closed mattheu closed 8 years ago

mattheu commented 9 years ago

I think technically this should use a button element here instead of a link.

I ran into this because this element is given the disabled attribute whilst loading more results. However this is not really honoured on links, only buttons, inputs etc. Visually this isn't a problem - CSS handles it OK. But the actual click event is still fired and you can easiliy trigger multiple requests.

nickdaugherty commented 8 years ago

Thanks!