Skip to content

Commit

Permalink
Merge pull request #834 from paulcpederson/bugs-fix
Browse files Browse the repository at this point in the history
Final round of fixes for v1.0.0-beta.7
  • Loading branch information
paulcpederson authored Jul 19, 2017
2 parents 3177df9 + ea7e284 commit cb9e9c7
Show file tree
Hide file tree
Showing 13 changed files with 113 additions and 133 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,11 @@
- card captions should use the `<figcaption>` element (#827)
- update instagram icon (#812)
- `input-minimal` now has a transparent background (#804)
- content shouldn't scroll behind modal when open (#766)
- revert to using fonts.com service (#819)
- fix inputs in dark theme (#829)
- fix multiline tooltips in Edge (#773)
- fix focus state of checkboxes in firefox (#802)

### Breaking
- :warning: No longer support IE9 (this impacts only block groups, radio inputs, selects, drawers, and the loader)
Expand Down
15 changes: 1 addition & 14 deletions docs/source/assets/css/all.scss
Original file line number Diff line number Diff line change
Expand Up @@ -19,24 +19,11 @@ $include-calcite-icons: false;
background: url('../img/docs/banner.svg') no-repeat center center $off-white;
}

.text-gradient {
background-size: 100% 100%;
background-image: linear-gradient(110deg, #005e95 10%, #9bc19c 90%);
-webkit-text-fill-color: transparent;
-webkit-background-clip: text;
-moz-background-clip: text;
background-clip: text;
}

.huge-text {
font-size: 5vw;
font-size: calc(4vw + 16px);
line-height: 1.25;
}

.site-search {
margin-top: .25rem;
}

// Sub-Nav test Classes
.sub-nav {
background-color: $darker-blue;
Expand Down
2 changes: 1 addition & 1 deletion docs/source/documentation/grid/_scroll-show-and-hide.md
Original file line number Diff line number Diff line change
@@ -1 +1 @@
Some layouts call for elements to appear or disappear as the user scrolls down the page. An element with the `scroll-show` class will appear when it reaches a point specified by the `data-top=x` attribute, appearing when it comes within the given value of the top of the screen.
Some layouts call for elements to appear or disappear as the user scrolls down the page. An element with the `scroll-show` class will appear when it reaches a point specified by the `data-top=x` attribute, appearing when it comes within the given value of the top of the screen.
8 changes: 8 additions & 0 deletions docs/source/documentation/grid/_scroll-sticky.md
Original file line number Diff line number Diff line change
@@ -1 +1,9 @@
Elements that become position fixed as they reach a certain position in the viewport are refered to as 'sticky'. The `js-sticky` class takes an `data-top=x` attribute that defines the point from the top of the window the sticky element will stick.

For example, the following `<div>` will become fixed when it is 46 pixels from the top of the viewport:

```
<div class="js-sticky" data-top="46">
<a href="#" class="btn btn-clear">Back to Top</a>
</div>
```
4 changes: 3 additions & 1 deletion docs/source/documentation/javascript/_overview.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,6 @@ Currently, the following components and patterns rely on `calcite-web.js`:
- [Modals](../patterns/#modals)
- [Tabs](../patterns/#tabs)
- [Accordions](../patterns/#accordions)
- [Drawers](../patterns/#drawers)
- [Drawers](../patterns/#drawers)

The JavaScript library also includes helpers for "sticky" positioning and scroll events. You can read more about these in the [interactive layout helpers](../grid/#interactive-layout-helpers) section of the documentation.
2 changes: 1 addition & 1 deletion docs/source/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
<header class="bg-banner" role="banner">
<div class="grid-container">
<div class="column-24 padding-leader-4 padding-trailer-4">
<h1 class="text-center text-gradient huge-text avenir-light">Calcite Web</h1>
<h1 class="text-center text-blue huge-text avenir-light">Calcite Web</h1>
<p class="text-center text-darker-gray font-size-2">A Responsive HTML, CSS, and JS Framework for Esri</p>
</div>
</div>
Expand Down
3 changes: 2 additions & 1 deletion lib/js/helpers/classy.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ export function has (domNode, className) {
export function add (domNode, classes) {
classes.split(' ').forEach(function (c) {
if (!has(domNode, c)) {
domNode.setAttribute('class', domNode.getAttribute('class') + ' ' + c);
var existingClass = domNode.getAttribute('class') || '';
domNode.setAttribute('class', existingClass + ' ' + c);
}
});
}
Expand Down
5 changes: 5 additions & 0 deletions lib/js/patterns/modal.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import bus from '../helpers/bus';

function modal () {
// Cool nodes
var html = document.documentElement;
var wrapper = document.querySelector('.wrapper');
var footer = document.querySelector('.footer');
var toggles = dom.findElements('.js-modal-toggle');
Expand Down Expand Up @@ -46,11 +47,15 @@ function modal () {
modal.removeAttribute('tabindex');
event.add(document, 'focusin', fenceModal);
classy.add(modal, 'is-active');
classy.add(html, 'drawer-no-overflow');
aria.hide(dependentNodes());
modal.focus();
}

function closeModal (modalId) {
if (classy.has(html, 'drawer-no-overflow')) {
classy.remove(html, 'drawer-no-overflow');
}
if (!modalId) return classy.removeActive(modals);
var modal = document.querySelector(`.js-modal[data-modal="${modalId}"]`);
classy.remove(modal, 'is-active');
Expand Down
5 changes: 3 additions & 2 deletions lib/sass/calcite-web/components/_form.scss
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@

::-ms-value {
border: none;
background-color: white;
background-color: transparent;
}

::-ms-browse {
Expand Down Expand Up @@ -220,7 +220,8 @@
&:focus {
box-shadow: none;
border: none;
outline: auto;
outline: 1px dotted; // IE, FF, and Edge browsers don't understand 'auto', so set backup
outline: auto -webkit-focus-ring-color;
}
}
input[type='checkbox'] {
Expand Down
1 change: 1 addition & 0 deletions lib/sass/calcite-web/components/_tooltip.scss
Original file line number Diff line number Diff line change
Expand Up @@ -173,6 +173,7 @@

.tooltip-multiline {
&:after {
width: 22em;
width: -moz-max-content;
width: -webkit-max-content;
max-width: $multiline-max-width;
Expand Down
3 changes: 2 additions & 1 deletion lib/sass/calcite-web/patterns/_drawers.scss
Original file line number Diff line number Diff line change
Expand Up @@ -63,12 +63,12 @@ $opacity-transition: opacity 200ms linear;
@include transition-prefixed($transform-transition);
@include transform(translate3d($drawer-width,0,0));
}

@mixin drawer-right-is-active() {
@include transition-prefixed($transform-transition);
@include transform(translate3d(-$drawer-width,0,0));
}


@if $include-drawers == true {
.drawer { @include drawer();}
.drawer-nav { @include drawer-nav();}
Expand Down Expand Up @@ -107,4 +107,5 @@ $opacity-transition: opacity 200ms linear;
}
}
}
.drawer-no-overflow { overflow: hidden; }
}
2 changes: 1 addition & 1 deletion lib/sass/calcite-web/type/_config.scss
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ $indent: 1em;

// Header Family
$avenir-tracking: 0;
$avenir-family: 'Avenir Next', 'Helvetica Neue', sans-serif;
$avenir-family: 'Avenir Next W01', 'Avenir Next W00', 'Avenir Next', 'Avenir', 'Helvetica Neue', sans-serif;

// Code Family
$code-tracking: 0;
Expand Down
Loading

0 comments on commit cb9e9c7

Please sign in to comment.