Commit Graph

67 Commits

Author SHA1 Message Date
Said El Bakkali cf4410cb1f
Replace className.indexOf with classList.contains (#1422)
* Compile rlt styles

* Replace className.indexOf with classList.contains

* Fix the composer comand

* Update js/navigation.js

Co-authored-by: Ismail El Korchi <ismail.elkorchi@gmail.com>

* Update js/navigation.js

Co-authored-by: Ismail El Korchi <ismail.elkorchi@gmail.com>

* Update js/navigation.js

Co-authored-by: Ismail El Korchi <ismail.elkorchi@gmail.com>

Co-authored-by: Ismail El Korchi <ismail.elkorchi@gmail.com>
2020-05-06 07:20:29 +00:00
Jared Caraway b44cdaecbc Remove unnecessary aria-expanded attribute on ul
Remove another irrelevant instance of aria-expanded
2020-04-11 20:43:26 +01:00
Ismail El Korchi 9dee764949 Fix a minor spacing issue 2020-04-10 03:04:28 +01:00
Ron Holt 29b0bb311f Close small menu when user clicks outside #1375 2020-04-10 02:55:53 +01:00
Ismail El Korchi f94fc1e59b Fix PHP, CSS AND JS coding standards 2020-04-05 21:49:27 +01:00
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
David A. Kennedy cc7ca98ab4 Merge pull request #1034 from joshmcrty/master
Update .site-title styles instead of anchor in customizer script.

Closes #990
2017-06-09 15:00:38 -04:00
Tammie Lister e38cfc29bc Merge pull request #1052 from samikeijonen/issue/1045
Remove aria-haspopup: props @samikeijonen
2016-12-23 14:30:14 +00:00
Sami Keijonen 9a988eae35 Fix skip link focus. 2016-11-26 19:42:51 +02:00
Sami Keijonen 25abf6d66f Remove aria-haspopup. 2016-11-26 19:35:00 +02:00
Josh McCarty e663b8d465 Update .site-title styles instead of anchor in customizer script 2016-10-16 10:17:47 -07:00
David A. Kennedy 8b60d6e2a6 Add new line at end of file for JS Hint. 2016-05-26 18:00:54 -04:00
David A. Kennedy a089ad81ac Fix errors in Travis CI from latest PR.
* `navigation.js` had a few spacing issues and variable placement problems causing JS Hint to throw errors.
* See: https://github.com/Automattic/_s/pull/900
2016-05-26 17:47:24 -04:00
Caroline Moore ad8a7c0ac7 Merge pull request #900 from iamtakashi/fix-menu-for-touch
Toggle focus class to allow submenu access on tablets. Props @iamtakashi
2016-05-26 16:03:16 -04:00
sixhours 0ab215d1c8 Clarify TAB key support in navigation header comment 2016-02-26 10:05:46 -05:00
Takashi Irie 7c5b0d8d18 Toggle focus class to allow submenu access on tablets 2016-01-13 11:13:59 +00: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
jrfnl b8d776c59c Slim down the travis script, add js checks and fix js errors thrown up.
What it will now do:
- Lint the php files against relevant PHP versions.
- Lint the js files once - the result won't change across PHP versions.
- Check against WPCS once - the result won't change across PHP versions.

What I have changed:
- Added linting against PHP 7 and HHVM, with HHVM being allowed to fail.
- Added js linting and style check per example from Twenty Sixteen.
- Moved to the faster container based environment for running travis.
- Script will no longer pull in PHPCS, WPCS and the JS linters in every build. Now they will only be pulled in when needed.
- Limited the clone depth for quicker cloning of external repositories.
- Removed the pulling in of WP and running builds against different WP versions as this wasn't used at all in the actual test scripts.
2015-12-13 15:03:43 +01:00
David A. Kennedy 666dbada97 Merge pull request #785 from carl-alberto/fixCustomizerHeaderTextColor
Fix Customizer header text color, applying the color change to the `a`, not the parent element.
2015-11-12 18:56:33 -05:00
carl alberto f2722f9b3f Fix for open issue #768 (Site title color in the Customizer)
This fixes the header text color link (blogname) to also update the same way as the blogdescription

Fix for open issue #768 (Site title color in the Customizer)

Fixed the missing Blog title text when "Display Header Text "option in the Customizer is toggled on/off

Updated the code to adjust the css color of .site-title a

Fix for the header color issue #816
2015-11-12 16:07:15 +08: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
Tammie Lister 79e0e54f69 Merge pull request #677 from mignonstyle/prototype_customizer
Reflect the change of color in the customizer.
2015-09-16 20:57:34 +01: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
Christopher Crouch 18f843ed03 _s: Tabbable dropdown menus.
Fixes #540.
2015-04-07 13:09:00 -07: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
Mignon Style c482072932 Reflect the change of color in the customizer.
Fixed ".site-title" to ".site-title a".
2015-01-09 17:09:22 +09: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
sixhours 204654ac1f Revert previous (unintentional) commit 2014-12-17 15:09:44 -05:00
sixhours 88f94a9a4e First pass at fixing drop-down menus for touch
This patch allows the user to view a site on a touch screen tablet and
access child menus by clicking on the parent once, or access the parent
by clicking on it twice. Props @iamtakashi for the original fix.
2014-12-17 15:03:17 -05:00
Konstantin Obenland d8d89bc681 _s: Improve menu toggle accessibility.
Makes it a little easier to use the primary navigation with assistive
devices.

See #545.
2014-11-05 17:37:48 -08:00
Philip Arthur Moore 53f62fa23a Merge pull request #459 from philiparthurmoore/menu-a11y
a11y improvements on primary navigation menu toggle
2014-04-27 11:48:11 +07: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
Philip Arthur Moore 4b883e2aee a11y improvements on primary navigation menu toggle 2014-04-13 04:02:17 -05:00
obenland 7c1067eee3 _s: Remove `image.php` and related code. Image attachments will be
displayed by `single.php`. See
http://codex.wordpress.org/Template_Hierarchy
2013-11-07 15:59:02 -04:00
Vinod Dalvi 430d379f4b Fixed minor WordPress coding standard issues. 2013-09-30 22:32:42 -07:00
Thomas Guillot f87479f70d _s: Fix element for left arrow key code
Previous one was deprecated
2013-08-27 10:29:46 +01:00
Shea Bunge e5a8931afc Add missing semicolon 2013-07-14 16:39:03 +10:00
Shea Bunge 4c99b2aba8 Use === and !== instead of == and != 2013-07-14 16:38:08 +10:00
Michael Fields ac3cda4543 Rework Navigation Menu Part 2
* Separate scoping and assignment so that a decision does not need to be made as to whether we should intermix spaces and tabs on the left
 * Restore the previously removed nav-menu class addition to the first unordered list in `#site-navigation`.
 * Restore the `.navigation-main.toggled-on .nav-menu` selector.
 * Remove rules for the `.main-small-navigation ul` selector.
2013-06-25 01:45:52 -07:00
Michael Fields 4c396f6f9b Rework menu navigation: these changes build off pull request #239 for issue #200
* Only set button and menu if container is not null.
 * Break up the "return early" conditional into multiple parts.
 * Shorten the name of the toggled class from `toggled-on` to `toggled`.
 * Restore the "Hide menu toggle button if menu is empty." documentation.
 * Remove the `nav-menu` class. The previous code would replace any custom custom classes applied via `wp_nav_menu()` or template file.
 * Replace the `nav-menu` selector is style.css with selectors that will target both a custom nav menu and a page menu. These changes could use critical feedback. Do you know of an edge case where they might not work as expected?
2013-06-16 20:29:17 -07:00
obenland f0ed253d5b Merge branch 'custom_header_fix' of https://github.com/iamtakashi/_s
into iamtakashi-custom_header_fix

Conflicts:
	inc/custom-header.php
2013-05-28 09:31:59 -07:00
Philip Arthur Moore c186c5ee0b Remove HTML5Shiv from _s. See #215. 2013-05-11 10:00:08 +09:00
Takashi Irie f367fefd72 Fix unlinked site title in the Customizer 2013-05-08 12:27:48 +01:00
Philip Arthur Moore 1ea2ccd909 Remove hgroup from theme. It's dead now: http://www.sitepoint.com/html5-hgroup-element-dropped/. In its place a div will be use that inherits the old class that was being applied to header. 2013-04-23 15:45:23 +09:00
Takashi Irie 8ceefdcdeb Fixes the broken display header text option in the Customizer. 2013-04-22 16:47:14 +01: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
obenland c7f2b7a1ee _s: Enable live preview for Header Text Color in Customizer. Change also
removes an !important declaration in Custom Header CSS rules.
2013-03-04 16:13:56 -08:00