The module allows users to apply minimalist scrollbars to sections of content. It has been designed with ease-of-use in mind and allows designers to add scrollbars in just a few steps, without having to manually load plugin script files and css stylesheets. It is dependent on the Libraries API and loads two libraries. The mCustomScrollbar jQuery Plugin is the primary library, with the Mouse Wheel library available as an option.
Users can apply vertical scrollbars site-wide to blocks and nodes, or can specify CSS attributes to receive either vertical or horizontal scrollbars. Features include, a choice of 8 scrollbar themes, inertia/easing, autohide/show, touch scroll, dragger resize among others.
To my knowledge this is the first implementation of the mCustomScollbar jQuery plugin within a Drupal Module. In addition, a good deal of time has been spent exposing many of the features of this jQuery plugin to the user through Drupal's adminstration/configuration section, in order to make it availabe to novice developers/users.
Project Page:
https://drupal.org/sandbox/knooq/2087687
Git:
git clone http://git.drupal.org/sandbox/knooq/2087687.git neat_scrollbar
Reviews of other projects (1)
https://drupal.org/node/1924716#comment-7110488
https://drupal.org/node/1884178#comment-7017392
https://drupal.org/node/1923524#comment-7098122
Reviews of other projects (2)
https://drupal.org/node/2089507#comment-7863559
https://drupal.org/node/2081395#comment-7861657
https://drupal.org/node/2084559#comment-7862693
Comments
Comment #1
davidmac CreditAttribution: davidmac commentedAdded "PAReview: review bonus" tag.
Comment #2
davidmac CreditAttribution: davidmac commentedReview of the 7.x-1.x branch:
http://pareview.sh/pareview/httpgitdrupalorgsandboxknooq2087687git
No current issues in relation to the pareview.sh/code sniff tests.
Comment #3
klausimanual review:
But otherwise looks RTBC to me. Removing review bonus tag, you can add it again if you have done another 3 reviews of other projects.
Assigning to cweagans as he might have time to take a final look at this.
Comment #4
davidmac CreditAttribution: davidmac commentedThanks @klausi.
Changes made.
re-ran pareview.sh/code sniff - no issues.
http://pareview.sh/pareview/httpgitdrupalorgsandboxknooq2087687git
Comment #4.0
davidmac CreditAttribution: davidmac commentedminor typing corrections
Comment #5
davidmac CreditAttribution: davidmac commentedAdded PAReview: review bonus Tag.
please see 'Reviews of other projects (2)' above.
Comment #6
cweagansSorry for the delay in getting to this. Looking now.
Comment #7
cweagansThis will not block your application, but I'd like to point out that exporting all this stuff is a PITA with Features. This would be much nicer if it were all combined into one variable (for instance, "neat_scrollbar_settings") that just contained an array with all the settings.
Please move this into a separate Javascript file, use #attached instead of drupal_add_js(), and then use Drupal.settings to pass the information to your script.
Other than that, this looks good. Back to CNW for the second issue. I'll keep an eye on the issue. Please update this issue when you've fixed that issue, and I'll come back and take another look.
Comment #8
cweagansSorry, let me clarify about #attached: This is for your separate Javascript file as well as for your drupal_add_css() call.
Also, you can use #attached for loading libraries: https://drupal.org/node/1831222#comment-6841028, so you don't have to do hardly any work in hook_preprocess_page().
Comment #9
davidmac CreditAttribution: davidmac commentedThank you for your advice.
I have reworked this module to address the following:
Implement Drupal.settings, via #attached and pass PHP variables to scripts that way.
Use #attached to load the main Library as opposed to libraries_load().
Use #attached to load css & js files as opposed to drupal_add_css() & drupal_add_js().
Reduced the number of persistent variables in the Form and .install file from 19 to 4 in order to make it more manageable with modules such as Features.
Comment #10
cweagansThanks for your contribution, davidmac!
I updated your account to let you promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and get involved!
Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
Thanks to the dedicated reviewer(s) as well.
Comment #11
davidmac CreditAttribution: davidmac commentedmany thanks @cweagans, @klausi.
Comment #12.0
(not verified) CreditAttribution: commentedInserted second round of 'Reviews of Other Projects'