The current slider widget template contains YUI grid system classes. That seems odd since YUI grid system isn't used. It might even conflict with a theme's YUI grid if someone actually uses one. Furthermore YUI is discontinued by Yahoo. This patch changes the YUI classes to something semantic.

CommentFileSizeAuthor
search_api_ranges-strip_yui.patch1.14 KBbasvredeling

Comments

ptmkenny’s picture

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

The patch looks good and updates the module to behave like other Drupal modules (provide classes based on the module name, not something else). Good catch!

basvredeling’s picture

basvredeling
Primary language Dutch
Location Amsterdam
commented

Project needs new maintainer. This is still ready for commiting.

Scott Robertson’s picture

Scott Robertson commented

Scott Robertson’s picture

Scott Robertson commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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