Revert "Tidy up styling and correct declaration order"

This reverts commit 55526c00f98621a97bbbcaa4695bcccdcbf1a3aa.

Tidy up styling and correct declaration order

Fix incorrect fallback reveal position. Add box shadow as a pseudo-element (thanks for the idea Kevin!). Rename the off-canvas overlay for better consistency.

Add reflow wrapper for slider

Slightly refactor slider and add mutation observer

Remove some commented out code

Sass lint: fix various lint errors

Sass lint: fix various properties sort order

See: https://github.com/zurb/foundation-sites/pull/9333

Sass lint: replace scss-lint disable comments

- Replace `scss-lint` disable comments with the `sass-lint` ones
- Remove `scss-lint` disable comments when they are useless or not
supported

Sass lint: add missing empty lines and fix indentation

- Add missing empty lines after declaration blocks
- Fix bad indentation.

Note: `sass-lint` still wrongly throw indentation warnings.
See
https://github.com/sasstools/sass-lint/issues?utf8=%E2%9C%93&q=is%3Aissu
e%20is%3Aopen%20indentation

Sass lint: remove useless vendor prefixes

- Remove the vendor prefixes that are automatically added by
`autoprefixer` for all browsers.
- Disable `sass-lint` when a vendor prefix is required by one
particular browser.

Sass lint: use "stroustrup" as brace style convention

Sass lint: use "stroustrup" instead of "1TBS" as brace style convention.
See: https://en.wikipedia.org/wiki/Indent_style

Pros:
- Easily add or remove lines within a `@if`/`@else if`/`@else` block,
without having to rearrange the braces.
- Able to add comments that describe a condition (`@if (...)`, `@else
if (...)` or `@else`) or the whole conditional block.
- Clearly separate the different parts of the conditional block.

Cons:
- Not commonly used

Sass lint: `0` convention for no border

Use `border: 0;` instead of `border: zero;` to disable border.

See :
https://stackoverflow.com/questions/2922909/should-i-use-border-none-or-
border-0

Pros:
- More concise
- Used a lot

Cons:
- X

Other convention: `border: none;`

Sass lint: add missing leading-zero

Use a leading zero for numbers.

Pro:
- css-guidelines
(https://github.com/chris-pearce/css-guidelines#formatting)
- a mathemacial point of view (http://hey.georgie.nu/leadingzero/).

Cons:
- Codestyle.co (http://codestyle.co/Guidelines/css#Syntax)
- Less concise

Update sass-lint to 1.3.2

Update sass-lint to 1.3.2 to avoid a lot of fixed bugs, like
unreconized interpolation

Lint: remove EOL spaces

Remove spaces where they are at the end of a line

Clean up has-submenu-toggle on accordion menu destroy

Merge pull request #9377 from michaellennox/patch-1

[Docs] Match headers for large and small callouts

Merge branch 'new-offcanvas' of git://github.com/brettsmason/foundation-sites into brettsmason-new-offcanvas

Merge branch 'refactor/properties-order' of git://github.com/ncoden/foundation-sites into ncoden-refactor/properties-order

Merge pull request #9374 from ncoden/fix/useless-gulp-sass-vendor-path

Remove useless gulp sass-lint vendor path

Merge pull request #9373 from ncoden/chore/normalize-scss

Migrate from normalize 4.2 to normalize-scss for 5.0

Correctly remove the submenu-toggle when the accordion menu is destroyed.
Added 2 visual tests: standard submenu-toggle and a responsive menu accordion to dropdown.

Remove useless gulp sass-lint vendor path

[Docs] Match headers for large and small callouts

Previously the large and small callouts were headed with 'This is a secondary panel' even though it wasn't actually a secondary callout and as a visual description doesn't match the styles used in the coloring section.

This PR modifies the header text in these examples to match the style used in the coloring section and explains the example using the class names ('small callout' and 'large callout') in the same manner as with the coloring section.

Migrate from normalize 4.2 to normalize-scss 5.0

Migrate to `normalize v4.2.0` to `normalize-scss` for `normalize v5.0`.
Remove normalise from `/scss/_global.scss`.

Remove box-shadow transition as its not needed.

Merge branch 'new-offcanvas' of git://github.com/brettsmason/foundation-sites into brettsmason-new-offcanvas

Update toggle to bubble up if no target is defined

Merge branch 'develop' into v6.3

Merge pull request #8389 from tjhiggins/th_dropdown_parent

[Dropdown] Added parentClass for a dropdown container

Merge pull request #9324 from zurb/vertical-flex-classes

Flexbox mixins & helper classes for direction, container, …

Update names and add responsiveness for child classes

Merge branch 'develop' of https://github.com/zurb/foundation-sites into develop

Add reference to responsive menu js in responsive nav docs

Merge pull request #9369 from simshaun/tabs-aria

Make Tabs' active class names configurable

fixed doc, now also mentions 5 as the default value

Merge pull request #9367 from Zearin/patch-1

Minor edits to formatting and phrasing

Correct box shadow description

Correct box-shadow styling. Add option to set a panel as absolute positioned

Clean up a few glaring code-style issues

Make Tabs' active class names configurable

Show/hide tab panels based on aria-hidden attr

Minor edits to formatting and phrasing

Ensure that unstuck classes are set on load

add block-grid-column so bottom margin matches gutters

Fix rebase error

Sass lint: fix various unsorted properties

Sass lint: sort right and left

Sort properties with `right`/`margin-right`/`padding-right`/(...)
before `left`/`margin-left`/`padding-left`/(...), following the common
CSS order `top, [right], bottom, [left];`.

Sass lint: Ignore variables concatenation

Add `ignore-custom-properties` to ignore properties concatenated with
variables.

Sass lint: Sort components properties

Sort components properties following the Specific property sort order.
But keep for `position`/`margin`/`padding` `left` before `right` for
now.

See : https://gist.github.com/ncoden/d42f55df7c7970f548a02cd3468f9c86

Sass lint: Sort utils properties

Sort utils properties following the Specific property sort order.

See : https://gist.github.com/ncoden/d42f55df7c7970f548a02cd3468f9c86

Sass lint: Sort typography properties

Sort typography properties following the Specific property sort order.

See : https://gist.github.com/ncoden/d42f55df7c7970f548a02cd3468f9c86

Sass lint: Sort grid properties

Sort grid properties following the Specific property sort order. But keep `margin`/`padding` `-left` before `-right` for now.

See : https://gist.github.com/ncoden/d42f55df7c7970f548a02cd3468f9c86

Sass lint: Sort forms properties

Sort form properties following the Specific property sort order.

See : https://gist.github.com/ncoden/d42f55df7c7970f548a02cd3468f9c86

Sass lint: Sort global properties

Sort global properties following the Specific property sort order.

See : https://gist.github.com/ncoden/d42f55df7c7970f548a02cd3468f9c86

Sass lint: Add property-sort-order rule

Add `property-sort-order` Sass lint rule, with the "Specific" sort
order.

See: https://gist.github.com/ncoden/d42f55df7c7970f548a02cd3468f9c86

added an example to the docs

Fix transition check to run off `[data-off-canvas]`. Sets aria-hidden="false" when an off canvas panel is revealed using `reveal-for-x`. Fixes #8536.

Merge pull request #9356 from zurb/IS8952-pass-content-on-tab-events

modify tabs events to return content area as well as a link

Merge pull request #9353 from abdullahsalem/fix-tabs-auto-focus-scrolling

Fixed the scrolling behavior when auto-focus is active

Merge branch 'v6.3' into develop

Remove Foundation.Move function as its not needed.

Merge pull request #9338 from sujayjaju/develop

Drop menu fix for handling closeOnClickInside

Merge pull request #9362 from Lindsor/lindsor/bug-9361

Fix bug #9361

Merge pull request #9360 from abdullahsalem/abide-validation-on-blur-feature

Added feature on abide to enable validation on blur event

Fix bug #9361

The flex grid sets the display of a row that is aslo a column as 'block', remove this declaration since its not needed.

Remove will change property

Remove the `will-change` property as it was causing issues with the sticky plugin

Added feature on abide to enable validation on blur event

Add missed keyboard etc code

Rewrite of off canvas Sass and JavaScript. Initial commit, still needs tidying up.

Merge pull request #9295 from zurb/IS9291-fix-reveal-race-with-animations

Fix a race condition when reveal is using animations

modify tabs events to return content area as well as a link

Merge pull request #9355 from zurb/IS9309-fix-responsive-gutters

Fix reversed ordering in gutters mixin that broke responsive gutters

Merge pull request #9296 from zurb/autoprefixer-config

Add ios >= 7 to autoprefixer

Fix reversed ordering in gutters mixin that broke responsive gutters

Merge pull request #9298 from karland/patch-1

Bug fixes for sticky

Merge pull request #9317 from zurb/IS7815-orbit-resize

Fix resize behavior of orbit

Merge pull request #9280 from zurb/IS8846-slider-fix-initial-start-fill-calc

Is8846 slider fix initial start fill calc

Remove accidental whatinput requirement introduced in #9174

Fixed the scrolling behavior when auto-focus is active

Merge branch 'develop' into v6.3

Merge pull request #9351 from ahebrank/v6.3

tabs deeplink event trigger fix

tabs deeplink event trigger fix

Merge pull request #9301 from colin-marshall/sass-lint

Migrate from scss-lint to sass-lint (#7620)

Removed Ruby references in README

Added blank space between sass-lint rules

Remove rvm install script from travis.yml

No longer necessary since we dropped our last ruby dependency when we migrated to sass-lint from scss-lint

Remove scss-lint.yml and the scss-lint install script

No longer necessary after migrating to sass-lint.

Added no-vendor-prefixes linter to sass-lint.yml

In scss-lint the VendorPrefix linter was incorrectly titled VendorPrefixes so the rules did not migrate properly to sass-lint.

Added .sass-lint.yml config file for sass-lint

Replaced scss-lint with sass-lint in gulp lint and sass tasks

Removed gulp-scss-lint and added gulp-sass-lint to package.json

Restructure Scss file to use the `accordion-menu` class which is already setup to work with the responsive menu plugin.
This allows us to apply a border and/or background to the accordion and submenu toggle to make it work correctly visually.

Merge pull request #9335 from ncoden/build/upgrade_node

Upgrade node version

Update required node version in README.md

Merge pull request #9347 from ncoden/fix/button-radius-config-naming-collision

Fix button-radius config naming collision

Fixed syntax error

Merge branch 'develop' into v6.3