Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ADA Compliant Masthead and Cross Site Blogging Files #68

Closed
wants to merge 13 commits into from
Closed

ADA Compliant Masthead and Cross Site Blogging Files #68

wants to merge 13 commits into from

Conversation

jasonmarkle
Copy link

Added <fieldset> and <legend> selectors to form fields in the _wvu-search and _wvu-search--wvu-tech.html patterns.

Added wvu-masthead__search-and-submit styles to 1-base for masthead form elements.

Added blog_index.xml to the views folder

Added util and blog_index.xslt for cross site blogging

Added global variable for cross-site-blog-url in config.yml

Added mock data for cross-site-blog-url

Added .wvu-article__image to _wvu-blog.scss in 1-base folder

Deleted old, incomplete search patterns from previous attempt of creating fully ADA compliant masthead.

jasonmarkle and others added 12 commits March 10, 2016 13:57
Added the fieldset selector to the radio buttons for better ADA compliancy. This should alleviate the WAVE warning getting thrown.
…ech patterns for ADA compliance and rendered text off screen.
…ech patterns for ADA compliance and rendered text off screen.
…e blog template for brand patterns out of the box cross site blog functionality with XSLT.
* 'master' of https://github.com/jasonmarkle/brand-patterns:
  Added scss for masthead links in search pattern to give a bg color fallback for ADA compliance.
  Added <fieldset> and <legend> to the wvu-search and wvu-search--wvu-tech patterns for ADA compliance and rendered text off screen.
  Added files via upload
  Delete _wvu-search.html
  Delete _wvu-search--wvu-tech.html
  Added files via upload
@jasonmarkle
Copy link
Author

@adamglenn I added the XSLT stuff, and then just put the few changes for the masthead in here as well and closed the other pull request. I imagine you can pick and choose what to add? If not I can separate these. Just let me know what will be easiest for you to look over things or if maybe I should just work on integrating it with the super theme which shouldn't be very difficult either.

@jasonmarkle
Copy link
Author

closing because of a nomenclature error with the variable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant