- 21 Apr, 2017 1 commit
-
-
Bardi Harborow authored
-
- 20 Apr, 2017 4 commits
-
-
Patrick H. Lauke authored
(with apologies, this slipped through due to build system problems on my end)
-
Patrick H. Lauke authored
Tweak to https://github.com/twbs/bootstrap/pull/22426, where the wrong selector slipped through the net (selecting all of `<body>`s grand-children rather than children)
-
Patrick H. Lauke authored
Same hack as in https://github.com/twbs/bootstrap/pull/22426 (modulo the selector, which is wrong in that PR and will be updated in a separate PR) to get tooltips to work correctly on iOS. Dynamically adds/removes empty (`noop`) `touchstart` event handlers to all children of `<body>` in order to coax iOS into proper event delegation/bubbling
-
Bardi Harborow authored
-
- 19 Apr, 2017 2 commits
-
-
Johann-S authored
-
Jesse Mandel authored
a bower search
-
- 18 Apr, 2017 3 commits
-
-
Patrick H. Lauke authored
-
Johann-S authored
-
Patrick H. Lauke authored
As the question often comes up about why Bootstrap does not do "true" ARIA menus (with their `role="menu"` etc), add an admittedly lengthy note (tl;dr because BS is generic, and ARIA menus are specific and quite limiting). Additionally, fixes up the `<a>` example for dropdown trigger, with the missing `role="button"` and neutering the `href` (which would be useless anyway since BS overrides the link-like nature of the `<a>` so it could never be triggered/followed anyway)
-
- 17 Apr, 2017 1 commit
-
-
Patrick H. Lauke authored
* Add carousel mouse listeners even if touch events enabled - touch events are enabled not just on "mobile", just also on touch-enabled desktop/laptop devices; additionally, it's possible to pair a mouse with traditionally touch-only devices (e.g. Android phones/tablets); currently, in these situations the carousel WON'T pause even when using a mouse * Restart cycle after touchend as `mouseenter` is fired as part of the touch compatibility events, the previous change results in carousels which cycle until the user tapped/interacted with them. after that they stop cycling (as `mouseleave` is not sent to the carousel after user scrolled/tapped away). this fix resets the cycling after `touchend` - essentially returning to the previous behavior, where on touch the carousel essentially never pauses, but now with the previous fix it at least pauses correctly for mouse users on touch-enabled devices. includes documentation for this new behavior.
-
- 16 Apr, 2017 5 commits
-
-
Patrick H. Lauke authored
A long overdue rewrite of the accessibility section - instead of the few snippets of strangely superficial and out-of-context advice (skip links, use correct heading levels), this tries to answer some of the fundamental questions about "is Bootstrap accessible", with emphasis on the fact that the final result will depend in large part on what BS is applied to/on (since BS relies on the markup etc authored by developers). This also sets out our ambition to have things work for keyboard and assistive tech users, and that we strive to make all our examples etc accessible and semantic. * Changes based on @mdo's feedback
-
Patrick Yeo authored
* Rename variable $input-border-focus to $input-border-color-focus * Rename variables $pagination-hover-border, $pagination-active-border, $pagination-disabled-border to $pagination-hover-border-color, $pagination-active-border-color, $pagination-disabled-border-color, respectively * Rename variables $state-*-border to $state-*-border-color, respectively * Rename variables $alert-*-border to $alert-*-border-color * Rename $list-group-active-border to $list-group-active-border-color * Rename $table-inverse-border to $table-inverse-border-color * Rename $btn-*-border to $btn-*-border-color * Rename $navbar-*-toggler-border to $navbar-*-toggler-border-color
-
Mark Otto authored
-
Patrick H. Lauke authored
This makes them keyboard-accessible. For mouse users, the only change here is that the focus outline will remain on the button once clicked (in future, this can be solved with :focus-ring, but for now this would require a polyfill). the tooltip is explicitly hidden on `mouseleave`, so even though the `<button>` retains focus after clicking, the tooltip won't stay visible once mouse user moves away. the mouse hover styles have explicitly not been make to also apply to :focus, so as to minimise the visual impact for mouse users (though the tooltip remains visible) - but due to the default outline and the custom tooltip, it should be fairly clear when keyboard users set focus to a copy button too.
-
Mark Otto authored
-
- 14 Apr, 2017 1 commit
-
-
Patrick H. Lauke authored
* Replace backdrop with simple noop mouse listener As discussed in https://github.com/twbs/bootstrap/pull/22422 the current approach of injecting a backdrop (to work around iOS' broken event delegation for the `click` event) has annoying consequences on touch-enabled laptop/desktop devices. Instead of a backdrop `<div>`, here we simply add extra empty/noop mouse listeners to the immediate children of `<body>` (and remove them when the dropdown is closed) in order to force iOS to properly bubble a `click` resulting from a tap (essentially, method 2 from https://www.quirksmode.org/blog/archives/2014/02/mouse_event_bub.html) This is sufficient (except in rare cases where the user does manage to tap on the body itself, rather than any child elements of body - which is not very likely in an iOS phone/tablet scenario for most layouts) to get iOS to get a grip and do the correct event bubbling/delegation, meaning the regular "click" event will bubble back to the `<body>` when tapping outside of the dropdown, and the dropdown will close properly (just like it already does, even without this fix, in non-iOS touchscreen devices/browsers, like Chrome/Android and Windows on a touch laptop). This approach, though a bit hacky, has no impact on the DOM structure, and has no unforeseen side effects on touch-enabled laptops/desktops. And crucially, it works just fine in iOS. * Remove dropdown backdrop styles * Update doc for dropdowns and touch-enabled devices
-
- 13 Apr, 2017 3 commits
-
-
Patrick H. Lauke authored
-
Patrick H. Lauke authored
-
Idan Cohen authored
-
- 12 Apr, 2017 5 commits
-
-
Pierre Vanduynslager authored
-
Patrick H. Lauke authored
...as touch is not exclusive to "mobile" anymore nowadays. also explicitly clarifies this is a fix for iOS, and that it impacts touch laptops etc as well. lastly, renames the variable from "dropdown" to "backdrop" for clarity/consistency
-
Patrick H. Lauke authored
...as touch is not exclusive to "mobile" anymore nowadays. also explicitly clarifies this is a fix for iOS, and that it impacts touch laptops etc as well. lastly, renames the variable from "dropdown" to "backdrop" for clarity/consistency
-
Pierre Vanduynslager authored
-
Pierre-Denis Vanduynslager authored
-
- 11 Apr, 2017 3 commits
-
-
Patrick H. Lauke authored
Closes #22061 Xref #22365
-
Patrick H. Lauke authored
-
Mark Otto authored
Remove docs files and Gruntfile from package managers
-
- 10 Apr, 2017 5 commits
-
-
Patrick H. Lauke authored
Per discussion on https://github.com/twbs/bootstrap/issues/22222#issuecomment-292819959 > browsers don't normally use the "not-allowed" cursor in these circumstances, so it makes BS a bit opinionated here. (and, at least on Windows, the "not-allowed" cursor looks rather fugly with its red colour compared to other regular mouse pointers) > >  Closes https://github.com/twbs/bootstrap/issues/22222
-
Patrick H. Lauke authored
-
Pierre Vanduynslager authored
-
Patrick H. Lauke authored
* Remove aria-expanded from collapse.js target element aria-expanded="true"/aria-expanded="false" only applies to the trigger, not the element that is being expanded/collapsed. * Tweak collapse.js accessibility section ...to make it clearer that the aria-expanded attribute always just goes on the control. * Fix collapse.js unit tests - reword some of the text to make it clear we're checking behavior of trigger/control - move incorrect aria-expanded out of the <div>s and to the actual trigger/control <a>s - fix incorrect test assertion text output false -> true
-
Patrick H. Lauke authored
* Only change aria-pressed if it's not an input-based radio or checkbox group aria-pressed="true"/aria-pressed="false" is really only useful for making on/off toggles out of, say, `<button>` elements. the attribute is useless (and potentially confusing/conflicting) on, say, `<label>` elements for an existing `<input type="radio">` or similar. * Add unit test for buttons.js and radio/checkbox inputs in button groups
-
- 09 Apr, 2017 7 commits
-
-
Pierre Vanduynslager authored
-
Patrick H. Lauke authored
`role="tabpanel"` is not appropriate as role for the overall container - it only applies to the individual content parts of each tab
-
Patrick H. Lauke authored
-
Patrick H. Lauke authored
While `placeholder` is nominally valid per spec http://rawgit.com/w3c/html-api-map/master/index.html#accessible-name-and-description-calculation it is inelegant, a fallback, and not supported in ios/safari/voiceover and android/chrome/talkback, to name a few combinations `aria-describedby` is also not really the right tool for providing a name/label equivalent, so remove that from the list of possible alternative methods / clarify how it can be used
-
Patrick H. Lauke authored
-
Patrick H. Lauke authored
-
Patrick H. Lauke authored
Prevent default for ESC in modal.js
-