Nicolas Coden [Sun, 25 Dec 2016 22:30:43 +0000 (23:30 +0100)]
Fix nested collapsed flex-grid row
Bug: columns in collapsed row doesn't have padding, but nested
collapsed flex-grid row still have negative margins.
Fix: apply the same path than in classic grids.
Note: Grids and flex-grids have a lot of bugs like this one and should
be refactorized. Please beat me if you see this commit 6 months later
and nothing changed.
Nicolas Coden [Sun, 25 Dec 2016 22:26:47 +0000 (23:26 +0100)]
Fix size of row nested inside collapsed (flex-)grid row.
See: https://github.com/zurb/foundation-sites/issues/9557
Problem: Every row has a max width, so when nested, it cannot be
enlarged with negative margins to handle the paddings of its columns.
Fix: expand the size of all row nested in a non-expanded row.
Note: THIS IS A TEMPORARY FIX. Grids and flex-grids have a lot of bugs
like this one and should be refactorized. Please beat me if you see
this commit 6 months later and nothing changed.
Problem: `@example` values are presented as default in the
documentation, and there is no way to get the default value
automatically. Plus, the "example" value is often not relevant at all.
> The reference section at the bottom is mostly meant to be a reference
> —key plugin options should be documented in the main body of the
> documentation, where there's more room to elaborate on things.
> -- @gakimball
Fix:
- Use `@default` instead of `@example`
- Make values in comment equal to the options default values
Warning: This require to update the value used to generate Js reference
in foundation-docs.
Replaced values:
Tooltip:
- `templateClasses`: `'my-cool-tip-class'` replaced to `''`
- `template`: `'<div class="tooltip"></div>'` replaced to
`''`
- `tip` text: `'Some cool space fact here.'` replaced to `''`
- `positionClass`: `'top'` replaced to `''`
Sticky:
- `container`: `'<div data-sticky-container class="small-6
columns"></div>'` replaced to `'<div
data-sticky-container></div>'`
- `anchor`: `'exampleId'` replaced to `''`
- `topAnchor`: `'exampleId:top'` replaced to `''`
- `btmAchor`: `'exampleId:bottom'` replaced to `''`
- `checkEvery`: `50` replaced to `-1`
Reveal:
- `animationIn`: `'slide-in-left'` replaced to `''`
- `animationOut`: `'slide-in-left'` replaced to `''`
- `showDelay`: `10` replaced to `0`
- `hideDelay`: `10` replaced to `0`
- `appendTo`: `false` replaced to `"body"`
Offcanvas:
- `transitionTime`: `500` replaced to `0`
- `forceTo`: `top` replaced to `null`
- `revealOn`: `reveal-for-large` replaced to `null`
- `revealClass`: `reveal-for` replaced to `reveal-for-`
- `trapFocus`: `false` replaced to `false`
Magelian:
- `animationEasing`: `'ease-in-out'` replaced to `linear`
- `deepLinking`: `true` replaced to `false`
- `barOffset`: `25` replaced to `0`
Equalizer:
- `equalizeOnStack`: `true` replaced to `false`
- `equalizeOn`: `'medium'` replaced to `''`
Dropdown:
- `parentClass`: `'dropdown-parent'` replaced to `null`
- `hoverPane`: `true` replaced to `false`
- `positionClass`: `'top'` replaced by `''`
- `autoFocus`: `true` replaced by `false`
Drilldrow:
- `backButton`: `'<\li><\a>Back<\/a><\/li>'` replaced by `'<li
class="js-drilldown-back"><a tabindex="0">Back</a></li>'
- `backButtonPosition`: `bottom` replaced by `top`
- `wrapper`: `<\div class="is-drilldown"><\/div>` replaced by
`'<div></div>'`
- `scrollTopOffset`: `100` replaced by `0
Don't listen to touchend events on dropdown element leafs.
The delay on touch devices between the click and touchend event
was preventing the link navigation, as browser security policies
don't allow link navigation on hidden elements.
PixelGrinch [Tue, 20 Dec 2016 19:58:12 +0000 (14:58 -0500)]
Update _menu.scss
It appears that centering the menu adds unnecessary/undesired height to the container.
My Proposed update to add vertical-align: top; to the inline-block applied container, fixes this issue.
Thank you
Bert (PixelGrinch)
Marc Langevin [Mon, 19 Dec 2016 14:31:29 +0000 (15:31 +0100)]
[Drilldown] Add an 'else' clause to not let the $wrapper property undefined
If the parent of the drilldown menu already has the 'is-drilldown' css class, the $wrapper property used everywhere in the plugin stays undefined. Fixes that issue.
Nicolas Coden [Thu, 15 Dec 2016 22:25:43 +0000 (23:25 +0100)]
Add menu modifier `menu-simple-direction` mixin
Changes:
- Make `@mixin menu-simple` taking a `$dir` parameter to generate
margins.
- Add `@mixin menu-simple-direction($dir: $global-left)`: Direction
modifier for a simple Menu.
- Add documentation for `@mixin menu-simple`
projects / thirdparty / foundation / foundation-sites.git / log
