Commit Graph

13 Commits

Author SHA1 Message Date
Julien Melissas b51b81a311 Consistent formatting for skip-link-focus-fix.js
Just noticed the spacing on the other files and not this one... See https://github.com/Automattic/_s/blob/master/js/customizer.js#L9 and https://github.com/Automattic/_s/blob/master/js/customizer.js#L42
2017-06-15 14:28:19 +02:00
Sami Keijonen 9a988eae35 Fix skip link focus. 2016-11-26 19:42:51 +02:00
jrfnl e1f4b0add4 Updated based on feedback.
* Include skip-link-focus-fix.js in the js hint/lint tests & fix up the file.
* Pull in .jshintrc from WP SVN.
* Add PHPCompatibility Sniffs.
* Sync the ignore statements to always exclude .min.js files.
* Slim down the tested against PHP versions even more.
2015-12-15 10:09:54 +01:00
David A. Kennedy 1703c44679 Change URL of pull request so it's not altered by generator
* Fixes https://github.com/Automattic/underscores.me/issues/39
* Related to #808
2015-10-29 13:25:18 -04:00
Konstantin Obenland e3516b0ad3 Fix typo. 2015-08-03 12:14:14 -07:00
PJ de9c9888b7 Updated documentation in comments at the top of files.
I tried to standardize and expand the documentation and links to
codex.wordpress.org in the comments at the top of most files.
2015-07-12 12:38:18 -04:00
oskar_cieslik dfd8a0e921 JSLint error
JSLint error: "regular expressions should be preceded by a left parenthesis assignment colon or comma"
2015-02-23 11:48:50 +01:00
Miguel Fonseca dea243f2eb Sanitize location.hash before passing it to getElementById
There is no actual vulnerability in the existing implementation, as we
can only fetch existing elements (no DOM injection is possible). Plus,
the only call occurring on those elements is `HTMLElement#focus`.

Consider this an extra, more future-proof precaution.
2015-02-17 17:52:36 +00:00
Julien Melissas 67fb1af3f3 Add braces around if statement.
Running jshint on this file returned an error, adding braces fixed it. Please add braces around if statements!!! :)
2014-12-24 00:19:34 -05:00
Jeffery To c6ad689d89 Removed addEvent support
As noted in #456, IE8 is no longer supported.

Also, undefined is falsey in JavaScript, so the typeof check was unnecessary.
2014-04-15 17:40:29 +08:00
sixhours 9f3e2cdd80 Remove blank tabs/trailing whitespace 2013-03-07 09:10:13 -05:00
obenland c4f2aa32ce _s: Formatting changes:
- Reduce element comments to either the element id or the first element
class.
- Tabs > Spaces.
- Improve adherence to formatting guidelines.
2013-03-04 16:41:38 -08:00
Aaron Jorbin 6ee6126089 On hashchange, make sure we set focus for elements that need it. Essentially the same as 136, but without jquery 2013-01-21 19:01:53 -05:00