Merge pull request #8674 from gehasia/sticky-docandtest-fix

[Doc] Sticky : Add documentation about stick-to-bottom. Visual test f…

Merge pull request #8730 from coreysyms/develop

change default behavior of tooltips

Merge pull request #8726 from flyingL123/flyingL123-bar-offset

Account for barOffset

Merge pull request #8712 from Owlbertz/dropdown-opens-inner

Fix open inner for dropdown menu.

Merge pull request #8567 from JamesJGarner/patch-1

Update foundation.tooltip.js to enable to parse a postionClass when c…

changes to tooltip default behavior

change default behavior of tooltips

Account for barOffset

Account for barOffset value when setting active menu item

Merge pull request #8720 from andycochran/accordion

fix defaults for accordion content color

Merge pull request #8704 from brettsmason/select-fix

Fix overlapping select arrow

fix defaults for accordion content color

Adding more tests

Adding bottom margin information

Fix open inner for dropdown menu.

Merge pull request #8706 from andycochran/accordion

fix radius and hover on Accordion

fix radius and hover on Accordion

Merge pull request #8691 from zurb/IS8300-click-open-not-being-respected

Fix data-click-open tooltip behavior

Merge pull request #8686 from zurb/IS8335-handle-arrays-with-extended-prototypes

Protect array for/in loops

Added padding to $global-right to allow for room for the arrow to appear without overlapping text.

Merge pull request #8616 from andycochran/stacked-button-group-margin

Fix $buttongroup-spacing

prevent rounding bug causing last expanded button in group to wrap

Merge pull request #8503 from Owlbertz/disable-slider

Add possibility to disable slider after initialization.

Merge pull request #8695 from andycochran/input-group-button-white-space

prevent input group buttons wrapping to multiple lines

Merge pull request #8678 from zurb/IS7458-reveal-animation-speed

Is7458 reveal animation speed

prevent input group buttons wrapping to multiple lines

Fix data-click-open tooltip behavior

Protect array for/in loops

Merge pull request #8449 from Owlbertz/touch-slider

Fixes touch interactions for slider

Merge pull request #8683 from artursopelnik/develop

Add inline-svg icon support

Merge branch 'develop' into stacked-button-group-margin

Add inline-svg icon support

fixes variable error in #8650

Merge pull request #8679 from designerno1/drilldown-wrapper-fix

fixes this.$wrapper return #8598

fixes this.$wrapper return #8598

Merge pull request #8650 from andycochran/disabledbuttonhover

prevent hover/focus effects on disabled buttons

Merge pull request #8646 from denisahac/develop

Appended "color: $accordion-content-color;" declaration in accordion-…

Merge pull request #8642 from designerno1/fixes-8624

fixes #8624 call stop on destroy

Merge pull request #8626 from dragthor/registerPlugin-spelling-typo

registerPlugin unregisterPlugin spelling typo

Merge pull request #8600 from designerno1/breakpoint-lint

fix 3 lint issues _breakpoint.scss

Merge pull request #8599 from designerno1/drilldown-closeOnClick

fixes closeOnClick

Merge pull request #8673 from designerno1/equalizer-stack-fix

possible fix for #8671

Add fast and slow classes to overlay if they exist on reveal

Test for animation of reveal

[Doc] Sticky : Add documentation about stick-to-bottom. Visual test fixed + stick-to-bottom test

possible fix for #8671
use getBoundingClintRect().top to detect stack

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

* 'develop' of https://github.com/zurb/foundation-sites:
  fixes #8643 do not reset val of radio and checkbox on reset the form reset prop checked false instead maybe a saving of the initial markup state of radio checked and checkbox checked would be great

Changed title-bar menu-icon side margins to be equal

Merge pull request #8657 from designerno1/fixes-8643

fixes #8643 do not reset val of radio and checkbox on reset

Added better styling to sticky callouts with anchors visual test

Added visual test for sticky nav from @gehasia

Fixed bug that was causing multiple reloads to occur for 1 file change in visual tests

Improved sticky callouts with anchors visual test

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

Modified gulp to watch for changes to html files in docs/assets/partials directory

Added sticky visual test with anchors and callouts

fixes #8643
do not reset val of radio and checkbox on reset the form reset prop checked false instead
maybe a saving of the initial markup state of radio checked and checkbox checked would be great

Merge pull request #8634 from designerno1/drilldown-docs-fix

Drilldown fixes a typo and adds fires open desc

prevent hover/focus effects on disabled buttons

Appended "color: $accordion-content-color;" declaration in accordion-content mixin

Merge pull request #8635 from sethkane/issue-7904

Issue 7904

Removing non needed aria from the body tag

fixes #8624 call stop on destroy

Removed automated aria-labelledby features as they aren't needed to ensure accessibility
Added documentation to explain how to utilize aria-labelledby

fixes a typo and adds fires open desc

Merge pull request #8614 from andycochran/media-object-margin-bottom

remove margin bottom from :last-child of .media-object-section

registerPlugin unregisterPlugin spelling typo

fix margin-bottom on buttons (don't use border)

remove margin bottom from :last-child of .media-object-section

Merge pull request #8610 from zurb/brettsmason-drilldown-visual-test

Create drilldown-menu-parent-link visual test

Create drilldown-menu-parent-link.html

Added a visual test to check the parent link option for drilldown menus.
Added to support [8595](https://github.com/zurb/foundation-sites/pull/8595).

Merge pull request #8595 from designerno1/fix-drilldown-parentlink

should fix #8593

Merge pull request #8608 from colin-marshall/customizer-menu-icon

Added missing menu-icon component to customizer config

Added missing menu-icon component to customizer config

Merge pull request #8564 from Owlbertz/dropdown-menu-fixes

Dropdown menu fixes

Merge pull request #8571 from Owlbertz/keyboard-enhancements

Enhance keyboard util.

Merge branch 'issue-7904' of https://github.com/sethkane/foundation-sites into issue-7904

# Conflicts:
# js/foundation.reveal.js

Issue #7904 should be addressed by updating various ARIA related issues.

Removed aria-hidden on both the body and the overlay. Since the modals are children of both of those anything that has an aria-hidden=true will be ignored by the screen reader. This is why with a keyboard you can tab around int he modal but when you use a screen reader and you tab around you can't access the content within the modal.

I also enhanced the aria-labeledby by checking to see if the modal has a H Tag (H1,2,3,4,5,6) of sorts. If it finds a H Tag the first one found will be associated to the aria-labeledby and thus will result with that text announced to the a screen reader after opening a modal. if there is no header it will utilize the element (href, button, etc...) that opened the modal and associated itself to the aria-labeledby and thus will result with that text announced to the a screen reader after opening a modal.

Removing Console Logs

fix 3 lint issues

var $body = $('body').not(_this.$wrapper);
not ignores clicks on links in the drilldown menu
fixes links with no sub and closeOnClick true doesn't fire

Issue #7904 should be addressed by updating various ARIA related issues.

Removed aria-hidden on both the body and the overlay. Since the modals are children of both of those anything that has an aria-hidden=true will be ignored by the screen reader. This is why with a keyboard you can tab around int he modal but when you use a screen reader and you tab around you can't access the content within the modal.

I also enhanced the aria-labeledby by checking to see if the modal has a H Tag (H1,2,3,4,5,6) of sorts. If it finds a H Tag the first one found will be associated to the aria-labeledby and thus will result with that text announced to the a screen reader after opening a modal. if there is no header it will utilize the element (href, button, etc...) that opened the modal and associated itself to the aria-labeledby and thus will result with that text announced to the a screen reader after opening a modal.

should fix #8593

Merge pull request #8583 from gehasia/drilldown-fix

[Bugfix] Drilldown - Cleaning attached events correctly on destroy() …

[Bugfix] Drilldown - Cleaning attached events correctly on destroy() call

Merge pull request #8579 from ncoden/fix-dropdown-position-classes

[Dropdown] Fix horizontal position with additional classes #8572

Merge pull request #8578 from alexej-d/patch-1

Remove duplication of max-width property in row mixin

Merge pull request #8576 from zurb/briantan888-patch-3

Fix missing checkbox requiredCheck

Merge pull request #8558 from dragthor/responsiveToggle-destroy-tab-destroy

tabs:destroy fixed; reponsiveToggle:destroy

Merge pull request #8553 from gehasia/sticky-bugfix

[Bugfix] Sticky - sticky-container should be 0px height when children…

Merge pull request #8543 from zurb/grid-col-pos-center-docs

Add description for the center option...

Merge pull request #8534 from andycochran/responsive-collapsed-row

add specificity for .expanded.row .[size]-collapse.row

[Dropdown] Fix horizontal position with additional classes #8572

Fix https://github.com/zurb/foundation-sites/issues/8572
> [Dropdown] Horizontal position does not work with additional classes

Remove duplication of max-width property

As a result of the `max-width` property being inside the `@each` statement, the property was needlessly duplicated while compiling.

Fix missing checkbox requiredCheck

requiredCheck for checkbox was removed from 6.2.1. Reinstating.

Enhance keyboard util.

Enhanced keyboard util by adding a parameter that is passed to the `handled` function based on the evaluation of the function that was handled.
It is now easier to only do certain actions (e.g. calling `preventDefaul()`) in the `handled` handler by letting the other function (e.g. `next`, `up`, ...) return some value, which is directly passed into `handled` as the first parameter.
I also adjusted the calls to the keyboard util for some plugins to fix issues and prevent bad behavior.

Update foundation.tooltip.js to enable to parse a postionClass when creating Tooltips using js

Changed code so the positionClass checks inside this.options rather than just looking at the elements css

Fixed keyboard interaction for dropdown menu.

Fixes behaviour for vertical menus to be correct (switch left/right with up/down compared to horizontal menus).
Fixed usage of `e.preventDefault()` to enable opening of links in the menu.

Fix for detection of overfloating dropdown menus.

Because the selectors were not specific enough, the styles of .opens-right set by the JS in case of an overflowing sub-menu was overwritten by the parent's .open-left styles.
Added visual tests to check for this. The tests include two cases: one for a right aligned dropdown using `.top-bar-right` (which was fixed earlier already) and one that is positioned right because of content (where this fix can be tested).
Also added anchors to the menu's links to check for correct behavior of the links, e.g. when using keyboard.

Merge pull request #8563 from colin-marshall/develop

Fixed background SVG bug in Edge

Fixed background SVG bug in Edge