_fusion_skins_grid_options() does not properly account for custom grids with less than 10 columns.

Currently in Fusion, the naming convention requires that all grids are based on a two-digit number. If you wish to create a custom grid with less than 10 columns, the grid name must be prefaced with a 0. Thus, a 9-column grid must be named grid09-960, rather than grid9-960.

The auto-width assignments that Fusion renders for blocks within a region uses the zero-preface naming convention, and the Fusion documentation instructs users to setup their grid CSS files using the zero-preface naming convention.

However, the function used in Fusion Core's Skin definitions file (skins/core/core.inc) does not account for this zero-preface naming convention (it will create classes such as grid9-6 instead of grid09-6.) and thus creates a set of skins that deviate from Fusion's auto-width classes and the classes within a properly created grid CSS file.

I don't think that _fusion_skins_grid_options() is intended to be used in customized contexts (I get an undefined function error if I do not copy into my sub-theme and rename the function), so this is not a bug, but it would be helpful to have a universally-available function of this type to help in rendering custom grids.

Ideally, grid definitions would be completely de-coupled from the Fusion theme and handled by the Fusion Accelerator module. However, if it would be helpful to have a stopgap solution for users who wish to create Skins for a custom grid (without having to manually code Skin options for each grid width option).

This issue originally reported here: http://fusiondrupalthemes.com/forum/using-fusion/setting-width-regions-i...

Comments

aquariumtap’s picture

Assigned: aquariumtap » Unassigned
Poieo’s picture

Version: 7.x-2.0-alpha1 » 7.x-2.x-dev
alan d.’s picture

Probably related to #917660: No support for grid counts under 10 or over 99., namely fusion_core_grid_info() also assumes fixed width strings too.

Poieo’s picture

Status: Active » Closed (duplicate)

Marking as a duplicate of #917660: No support for grid counts under 10 or over 99. since that issue has a possible fix.

Poieo’s picture

Deleted comment.