Hi,

In the README.md the suggested file structure is:

Your structure should look like:
sites/all/libraries/caret.js
sites/all/libraries/at.js

This is a bit confusing for users who think they will need to drop just the JS files into the top level of the libraries folder.

I suggest you add an extra slash at the end of each line to it is clear that at.js and carat.js are folder names. This will then show:

Your structure should look like:
sites/all/libraries/caret.js/
sites/all/libraries/at.js/

Cheers

Tom

CommentFileSizeAuthor
#1 at.js-libraries-folder-structure-2266253-1.patch529 bytesblacklabel_tom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

blacklabel_tom’s picture

blacklabel_tom CreditAttribution: blacklabel_tom commented
FileSize
at.js-libraries-folder-structure-2266253-1.patch529 bytes

Only a small change, but patch attached.

blacklabel_tom’s picture

blacklabel_tom CreditAttribution: blacklabel_tom commented
Status: Active » Needs review
arnested’s picture

arnested CreditAttribution: arnested commented
Status: Needs review » Reviewed & tested by the community

Seems reasonable.

heylookalive’s picture

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

Committed, thanks!

Status: Fixed » Closed (fixed)

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