this isn't necessary and is just plain confusing really

we should add the region's machine_name instead of human name and also this isn't necessary to do explicitly.

refer to: https://drupal.org/node/254940#closure

it's supposed to be there commented out in the first place as per the comment right above it

Issue #2046079 by alexweber: Added Allow displaying Drupal messages in a Foundation reveal modal via theme settings.

with the new theme settings it isn't necessary to leave this here as it's confusing and will break the default functionality that's controlled by theme settings

removed unecessary modal bg background (changed in Foundation 4)

we were introducing unecessary line breaks

Issue #2026519 by alexweber | chrisjlee: Added Refactor site name, slogan, general 'top' of page stuff.

set show-for-small on top bar when set to never visbible or mobile only

update code comments for zurb_foundation_links__system_main_menu() and update link to foundation top bar documentation page

small performance improvement

Jack Haas authored Jun 20, 2013 and Chris Lee committed Jun 20, 2013

Issue #2019249 by jerseycheese | chrisjlee: Update sass files and import statement structure from zurb foundation 3 to 4.

Alex Weber authored Jun 20, 2013 and Chris Lee committed Jun 20, 2013

Issue #2020135 by alexweber | chrisjlee, ishmael-sanchez: Added Basic theme settings for menu/mobile rendering.