Merge branch 'develop' into refactor/mouseleave-special-case

style: remove extranous linebreak in Touch

Merge pull request #11491 from ncoden/chore/fix-gulp-sass-watch

chore: update docs styles when SCSS change

Merge pull request #11497 from ncoden/docs/js-tree-shaking-path

docs: use named imports in javascript documentation

docs: use named import for javascript "direct import" fallback

docs: fix "zplugin" typo in Accordion & Tabs events

fix: close the active Tab when going back to a "tab-less" history state

See https://github.com/zurb/foundation-sites/pull/11477#issuecomment-419678146

refactor: clean up the Tab initialization

feat: add "Tabs._collapse()" to collapse the active tab

Merge pull request #11445 from ncoden/chore/tree-shaking

chore: improve "tree shaking" support and documentation

Revert "chore: add default export as it may be sometimes required for tree-shaking"

This reverts commit 5d9373b3dce67e29769a7afd73598dc609dc7dc1.

Merge pull request #11495 from gforcada/patch-1

Fix swipe docs

Fix swipe docs

See discussion on https://github.com/zurb/foundation-sites/issues/11490

test: add OffCanvas tests for openedEnd and closedEnd events

test: use "openedEnd" event in OffCanvas tests instead of low-level listenners

feat: add the "openedEnd" event to OffCanvas after opening transition

test: wait for OffCanvas to fully open then close to check for removed classes

fix: fix OffCanvas namespace fpr closed/closedEnd events

refactor: lint and simplify OffCanvas "sticky" methods

fix: only stick elements inside the OffCanvas content

Elements outside of the OffCanvas content are not affected by the "transform" property and should not be sticked.

docs: add missing code-block ending characters in OffCanvas doc

Merge branch 'develop' into SassNinja/feature/offcanvas-fixed-sticky

Merge branch 'develop' into refactor/xy-cell

chore: update docs styles when SCSS change

Documentation styles uses the SCSS mixins/functions, so they must be recompiled when the SCSS change.

Introduced in https://github.com/zurb/foundation-sites/pull/11454

Merge pull request #11488 from ncoden/chore/update-dependencies-2018-09-03

chore: update dependencies (2018-09-03)

chore: update dependencies (2018-09-03)

Major bump:

| Package                         | From          | To     |
| :------------------------------ | :------------ | :----- |
| gulp-postcss                    | 7.0.0         | 8.0.0  |
| gulp-babel                      | 8.0.0-beta.2  | 8.0.0  |
| @babel/core                     | 7.0.0-beta.54 | 7.0.0  |
| babel-loader                    | 8.0.0-beta.0  | 8.0.2  |
| @babel/preset-env               | 7.0.0-beta.54 | 7.0.0  |
| browserstack-runner             | 0.7.0         | 0.9.0  |

Minor bump:

| Package                         | From          | To     |
| :------------------------------ | :------------ | :----- |
| @commitlint/cli                 | 7.0.0         | 7.1.1  |
| @commitlint/config-conventional | 7.0.1         | 7.1.1  |
| autoprefixer                    | 9.0.1         | 9.1.3  |
| browser-sync                    | 2.24.6        | 2.24.7 |
| gulp-cache-bust                 | 1.3.0         | 1.4.0  |
| gulp-clean-css                  | 3.9.4         | 3.10.0 |
| gulp-prompt                     | 1.0.1         | 1.1.0  |
| gulp-zip                        | 4.1.0         | 4.2.0  |
| inquirer                        | 6.0.0         | 6.2.0  |
| sinon                           | 6.1.4         | 6.1.5  |
| webpack                         | 4.16.2        | 4.17.2 |
| webpack-stream                  | 5.0.0         | 5.1.1  |

Merge pull request #11485 from ncoden/fix/tabs-deep-link-custom-target-11484

fix: use data-tabs-target to update history in Tabs deep-linking #11484

Merge pull request #11486 from cyChop/develop

Fix a bug in tab changing

fix: use a safer selector for targetted tab title in "Tabs.selectTab"

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

Merge pull request #11487 from palicko/develop

Fix wrong responsive spacing prototype classes

docs: improve "tree shaking" documentation to promote named imports

chore: remove redundant "pkg.browser" field

According to the specification for `pkg.browser`, there is no reason for us to use this field as the package itself it only for browsers (or any environement with a DOM). Without the field, ESM-compliant transpilers/bundlers will pick `module`, and the others will pick `main`.

https://github.com/defunctzombie/package-browser-field-spec

Fix wrong responsive spacing classes #11236

Fix a bug in tab changing

When using deepLink options, the handleTabChange will fail if using a non-standard markup (if there are DOM levels between the `.tabs-title` and the `a`).

This commit fixes precisely that.

fix: use data-tabs-target to update history in Tabs deep-linking #11484

Closes https://github.com/zurb/foundation-sites/issues/11484

docs: remove outdated comments in Accordion related to the initial state

refactor: move "init only" Accordion logic to its own methods

refactor: split DOM operations from logics in Accordion

Split DOM operations from logics in Accordion "up" and "down" methods

fix: fix initial state for Accordion without any panel active

When going back in history, the Accordion without any panel active is still considered as in its "initial state".

fix: return to initial state when reseting hash in Accordion & Tabs #11100

When deep-linking is enabled and the user goes back in the history up to when no hash is set, display the initially-selected content/tab instead of staying with the content/tab that was selected the first.

Closes https://github.com/zurb/foundation-sites/issues/11100
Replaces https://github.com/zurb/foundation-sites/issues/11101

Co-authored-by: Jamie Chong <jamie@hippocurious.com>

Merge pull request #11466 from ncoden/chore/simplify-github-template

chore: simplify GitHub templates

Merge pull request #11475 from Harmageddon/patch-1

fix: Added missing quotation mark in documentation

Added missing quotation mark.

Merge pull request #11472 from ncoden/fix/xy-grid-offset-zero-margin-11471

fix: fix offset when gutters are 0 in "xy-cell-offset" #11471

fix: fix offset calculation in "xy-cell-offset"

fix: fix generated offset for zero gutters in "xy-cell-offset" #11471

Closes https://github.com/zurb/foundation-sites/issues/11471

Use pull request #11469 from ncoden/fix/clear-nest-submenu-classes-11467 for v6.5.0

fcd02bdfc fix: clear submenu item classe in Nest #11467

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11462 from ncoden/fix/reveal-mobile-full-height-10879 for v6.5.0

fd0479899 fix: avoid viewport height for fullscreen Reveal as it may be incorrect #10879

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11454 from ncoden/fix/split-gulp-tasks-11452 for v6.5.0

6c45b451e fix: remove broken Sass linting from Sass compilation task and move it to build task
80c6def1f refactor: split the docs build task to `docs:pages` and `docs:search`
6b12883d6 refactor: move the customizer ZIPing to `customizer:zip` and `customizer:clean`

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11464 from ncoden/docs/remove-vertical-block-grid-11463 for v6.5.0

2bb3df02d docs: remove mentions of unsupported Y Grid from block grids section

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11458 from SassNinja/docs/foundation-cli for v6.5.0

be886fb2f add more foundation-cli install instructions for windows user

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11450 from DanielRuf/fix/correct-selector-11447 for v6.5.0

b5452e883 fix: use correct selector, fixes #11447

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11444 from aorinevo/I-11443_develop_AN_gulp_fix for v6.5.0

d0f405400 fix: i11443 gulp watch
c90c048cc fix: prevent "test:reload" task to block future reloads

Co-Authored-By: Nicolas Coden <nicolas@ncoden.fr>
Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11448 from melek/patch-1 for v6.5.0

47264afbf Update installation.md
9e5f5e228 docs: fix Foundation 6 CLI uninstallation note phrasing

Co-Authored-By: Nicolas Coden <nicolas@ncoden.fr>
Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11435 from ncoden/docs/motion-ui-components for v6.5.0

b08a56da4 docs: move Motion UI Toggler example to its own section
bd4bf11d2 docs: add a simpler example in Motion UI docs header

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11433 from ncoden/fix/dropdown-menu-top-level-items-11412 for v6.5.0

6a0e3240b fix: fix Dropdown Menu top level item properties #11412

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11425 from ncoden/fix/drilldown-submenu-height-11416 for v6.5.0

c7ce8d5b9 fix: set the Drilldown height for on the currently opened (sub)menu #11416
43c1fac65 test: add unit tests for Drilldown resizing on toggling
7f7dfd83a test: check the Drilldrown wrapper height instead of the element height
90c4e45f8 fix: set a default for the current menu in Drilldown before it is used
1db6a1f2a test: add unit test for the Drilldown height when reopened from a submenu #11416
d4617ac08 clean: remove infinite timeout used for development in Drilldown tests

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11419 from ncoden/chore/update-dependencies-2018-07-26 for v6.5.0

d70627374 chore: update dependencies (2018-07-26)
4ed0ec2a9 chore: update lockfiles (2018-07-26)

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11411 from arloduff/fix/check-for-null-related-target for v6.5.0

ac88dac72 fix: resolve dropdown menu disappearing for #11410
ae8fa2def fix: move relatedTarget null check to setTimeout for #11410
a25180c3d style: improve dropdown hiding event readability in dropdownMenu
e46ddd1cc docs: improve description of #11410 bug fix in dropdownMenu
ebc4be784 docs: fix typo in bug description in dropdownMenu

Co-Authored-By: Nicolas Coden <nicolas@ncoden.fr>
Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11409 from veselinn/fix/slider-handle-negative-data-start for v6.5.0

81fc284e9 fix: slider handle negative data-start values
da0b014de fix: remove unnecessary if from slider
9851a17ff test: add tests for Slider._adjustValue

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11393 from DanielRuf/fix/matchmedia-polyfill-window-fix-ie11 for v6.5.0

e7554d60d fix: set matchMedia on window to fix issue on IE11, fixes #11358
1a36a44e7 chore: update MatchMedia polyfill to v0.3.1

Co-Authored-By: Nicolas Coden <nicolas@ncoden.fr>
Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11397 from ncoden/fix/xy-grid-responsive-modifiers-reset-10891-2 for v6.5.0

7e34dc3b5 perf: factorize XY grid responsive modifiers
21be51d21 fix: add missing flex reset in XY Grid responsive vertical cell widths

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Use pull request #11394 from ncoden/chore/dist-js-plugins-sourcemaps for v6.5.0

09a46c612 chore: update sourcemaps of distributed JS plugins

Signed-off-by: Nicolas Coden <nicolas@ncoden.fr>

Merge branch 'master-v6.5' into develop-v6.5

Merge pull request #11469 from ncoden/fix/clear-nest-submenu-classes-11467

fix: clean submenu items classes in Nest #11467

Merge pull request #11462 from ncoden/fix/reveal-mobile-full-height-10879

fix: avoid viewport height for fullscreen Reveal as it may be incorrect #10879

docs: make "direct import" for tree shaking sounds less imperative

chore: add missing "sideEffects" flag to ensure tree shaking with webpack

Merge pull request #11454 from ncoden/fix/split-gulp-tasks-11452

chore: use split gulp tasks instead of the unsuitable "finish" event #11452

fix: clear submenu item classe in Nest #11467

Closes https://github.com/zurb/foundation-sites/issues/11467

Merge pull request #11465 from ncoden/fix/avoid-stop-propagation-11457

fix: avoid stopping propagation when not necessary #11457

chore: simplify GitHub issue templates

chore: simplify the GitHub pull request template

fix: fix DropdownMenu body handler check to ignore the element itself

> `andSelf()` is deprecated and should be replaced with `addBack()`
> -- @SassNinja

Also, `andSelf` cannot be used that way. This commit replace it with the opposite approach: checking if the target is the child of the element or the element itself using `closest()`.

Merge pull request #11464 from ncoden/docs/remove-vertical-block-grid-11463

docs: remove mentions of unsupported Y Grid from block grids section

Merge pull request #11458 from SassNinja/docs/foundation-cli

foundation-cli docs for windows users

fix: avoid stopping event propagation when not necessary

Stopping the event propagation prevent the user to watch for the event at an higher level. If it was needed to prevent a duplicate behavior in a component, others methods should be used like debouncing the event.

Changes:
- Remove `stopPropagation` and `stopImmediatePropagation` from everywhere we do not want to cancel the event (AccordionMenu, Drilldown, DropdownMenu, Tabs, Tooltip).
- Update `DropdownMenu._removeBodyHandler` to also ignore click events on the element itself.

Others changes:
- Clean up `DropdownMenu._events` a bit.
- Add explainaitions where `stopPropagation` is still used (Triggers).

See https://github.com/zurb/foundation-sites/issues/11457

fix: prevent double Keyboard events handling without stopping propagation

Stopping the event propagation prevent the user to watch for the event at an higher level. If it was needed to prevent a duplicate behavior in a component, others methods should be used like debouncing the event.

Changes:
- Save and check for `event.zfIsKeyHandled` in Keyboard utility to prevent to handle twice the same event in different components.
- Remove `event.stopPropagation` from components' keyboard handlers (Accordion, DropdownMenu, OffCanvas and Tabs).

See https://github.com/zurb/foundation-sites/issues/11457

docs: add docs about "mobile scroll" bug #9707 in OffCanvas

Add documentation for the OffCanvas private methods `_recordScrollable` and `_stopScrollPropagation` to explicitely justify the need for `stopPropagation`.

See #9707

docs: remove mentions of unsupported Y Grid from block grids section

Block Grid is not supported for Y Grid and will not be implemented. It would add too much code for a feature that is will not be used in most projects.

Plus, Block Grid classes are not namespaced for the horizontal or vertical grid. Adding block grid classes for the vertical grid would lead of inconstencies or backward incompatibility and increased specificity.

Closes https://github.com/zurb/foundation-sites/issues/11463

Merge pull request #11434 from ncoden/feat/orbit-slide-active-state-attribute

feat: add Orbit element "data-slide-active-label" to target the bullet span to move

fix: avoid viewport height for fullscreen Reveal as it may be incorrect #10879

The CSS viewport is the larger possible view height and does not change with the visible area height. This is an intended behavior for optimization reason. See this article:

See: https://nicolas-hoizey.com/2015/02/viewport-height-is-taller-than-the-visible-part-of-the-document-in-some-mobile-browsers.html
See: https://github.com/zurb/foundation-sites/issues/10879#issuecomment-414087274

Closes https://github.com/zurb/foundation-sites/issues/10879

Merge pull request #11461 from DanielRuf/fix/interchange-remove-extraneous-regular-expression-11460

fix: remove extraneous regular expression in interchange, closes #11460

fix: remove extraneous regular expression in interchange, closes #11460

add more foundation-cli install instructions for windows user

Merge pull request #10884 from ncoden/feat/interchange-type-option-10883

feat: add "type" option to force interchange replacement method #10883

Merge branch 'develop' into feat/interchange-type-option-10883

feat: add support for "JPEG" picture in Interchange

docs: fix typos in Interchange

Merge pull request #11450 from DanielRuf/fix/correct-selector-11447

fix: use correct selector, fixes #11447

refactor: move the customizer ZIPing to `customizer:zip` and `customizer:clean`

The `finish` gulp event is an stream internal event and is not suitable to chain tasks.

Changes:
- Move the custom ZIP file generation to the `customizer:zip` task
- Move the cleaning of the customizer build directory to `customizer:clean` task

refactor: split the docs build task to `docs:pages` and `docs:search`

The `finish` gulp event is an stream internal event and is not suitable to chain tasks.

Changes:
- Move pages building to the `docs:pages` and `docs:pages:all` subtasks
- Move search entries building to the `docs:search` subtask

fix: remove broken Sass linting from Sass compilation task and move it to build task

Changes:
- Remove broken Sass linting from the `sass:foundation` task. The `finish` gulp event is an stream internal event and is not suitable to chain tasks.
- Add `lint:graceful`, `lint:graceful:sass` and `lint:graceful:javascript` to check for linting errors without throwing errors and breaking the chain of tasks.
- Call `lint:graceful` on the build task. For now, there is too much linting issues to display them at each Sass/Javascript compilation.

fix: use correct selector, fixes #11447

Merge pull request #11444 from aorinevo/I-11443_develop_AN_gulp_fix

fix: i11443 gulp watch

Merge pull request #11448 from melek/patch-1

docs: Fix sentence fragment in installation.md

fix: prevent "test:reload" task to block future reloads

docs: fix Foundation 6 CLI uninstallation note phrasing

chore: remove unrecommended "grouped" default exports

Default exports should only be used for "primary" objects or classes, not for libraries for which individual named exports should be prefered.

Update installation.md