Skip to content
This repository has been archived by the owner on Dec 23, 2017. It is now read-only.

Feature/search select #491

Merged
merged 15 commits into from
Aug 19, 2015
Merged

Feature/search select #491

merged 15 commits into from
Aug 19, 2015

Conversation

noahmanger
Copy link
Contributor

Adds classnames to dropdown components and also switches search toggle to search select.

(Note: I'm just commenting out the search radios for now because we may want to change them quickly back to radios)

Uses the styles from here: 18F/fec-style#17

Noah Manger and others added 14 commits August 17, 2015 22:03
Had to do this because candidate and committees don’t have them
It was erroring out when this file was empty
Moving testGlossaryLoadFromTerm to single_candidate_page_tests because
we may not always have a glossary term on the home page
- Includes classes for dropdown items
- Text in the filter toggle
Conflicts:
	static/styles/styles.scss
@@ -3,26 +3,38 @@
{% set size = 'combo--search--large' %}
{% endif %}
<form id="{{ location }}-search" action="/" autocomplete="off" class="search__container js-search">
<div class="search__radio toggles">
<label for="{{ location }}-radio--candidates">
<!-- <div class="search__radio toggles">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this block be deleted?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure.

@jmcarp jmcarp merged commit 5e94cf6 into develop Aug 19, 2015
@jmcarp jmcarp removed the plz-review label Aug 19, 2015
@jmcarp jmcarp deleted the feature/search-select branch October 2, 2015 16:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants