-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Slow load bib page placeholder #345
Comments
Do you have something in mind for this? |
Something like this http://cloudcannon.com/deconstructions/2014/11/15/facebook-content-placeholder-deconstruction.html Not important at the moment but it's just something I'd like to have in place eventually. |
that'd be cool... for the search results? for the item? both? it could be like a class for example, the search results would be: <li class="nypl-results-item placeholder">
<h2></h2>
<div class="nypl-results-item-description">
<p>
<span class="nypl-results-media"></span>
<span class="nypl-results-place"></span>
<span class="nypl-results-publisher"></span>
<span class="nypl-results-date"></span>
<span class="nypl-results-info"></span>
</p>
<p class="nypl-results-use"></p>
</div>
</li> |
Perhaps. It's tricky because not all the data is there and some DOM elements are not rendered if there's no data. So you don't want to display a placeholder for something that is not there. So I'd imagine it to be more of a generic case for data that we will definitely display there. For example, in the above, |
ok so then i design a |
Perhaps. This ticket was more of a front-end investigation on my part instead implementation. Need to figure out how to do it properly and if it's needed at all. |
got it |
for future reference: http://emptystat.es/ |
@danamansana I think I remember you mentioning something like this. Would definitely be a ~ post-launch ~. But thought you'd like to see the reference. |
I would love for this to actually get implemented now. As per a different comment, check "if it's needed at all". |
No description provided.
The text was updated successfully, but these errors were encountered: