After committing #2023527: Placeholder library should be attached to element, I realised that the library we recomment/reference in our docs does not contain a minified version anymore (this was changed fairly recently) We should change to https://github.com/mathiasbynens/jquery-placeholder and also switched back to minified.

CommentFileSizeAuthor
placeholder-mathiasbynens-lib.patch2.36 KBdamiankloip
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bleen’s picture

bleen CreditAttribution: bleen commented
Status: Needs review » Reviewed & tested by the community
damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x, I will also put a quick note on the project page for people upgrading.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

seanbfuller’s picture

seanbfuller CreditAttribution: seanbfuller commented
Issue summary: View changes
Status: Closed (fixed) » Active

I grabbed the placeholder.js from the updated repo today. There is no min file in there. Reopening since this causes errors for new installs.

idebr’s picture

idebr CreditAttribution: idebr commented
Status: Active » Closed (fixed)

The scope of this issue was to integrate with a different library. To update the integration with changes in the library, please open a new issue to it is easier for other people to find without wading through old comments. I have opened a new issue and attached a patch to update the module with the changes in the jquery.placeholder library, see #2297935: Update jquery placeholder library to integrate with the latest version