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

Integrate styling for Search elements #71

Merged
merged 12 commits into from
Aug 19, 2015
Merged

Integrate styling for Search elements #71

merged 12 commits into from
Aug 19, 2015

Conversation

charlesh88
Copy link
Contributor

Please review and integrate these changes into the 'search' branch. As soon as this is complete, we should file a pull request to integrate the 'search' branch into open.
Author checklist:

  • Changes address original issue? Y
  • Unit tests included and/or updated with changes? N/A
  • Command line build passes? N/A
  • Expect to pass code review? Y

VWoeltjen and others added 12 commits August 7, 2015 14:34
Addresses WTD-1506 (context menu does not disappear after
action is chosen); listen for click events on the menu
itself and dismiss the menu when these occur.
Add listener to menu element directly instead of
using ng-click to aid in testing (and for consistency
with related listeners.) WTD-1506.
Remove unused dependency from generic search worker,
avoids WTD-1511.
Remove unused dependency from generic search worker,
avoids WTD-1511.
Bump version number and restore snapshot status to
begin sprint Umbriel, WTD-825.
#70
Significant .scss and markup changes across search;
New symbols font (v2.2) char for clear icon;
To-do: style "load more" element;
@slhale slhale merged commit 7f65ba2 into search Aug 19, 2015
@slhale
Copy link
Contributor

slhale commented Aug 19, 2015

Review Checklist

  1. Changes appear to address issue? Y
  2. Appropriate unit tests included? N/A
  3. Code style and in-line documentation are appropriate? Y
  4. Commit messages meet standards? Y
  5. Project-specific information isolated to appropriate branches? Y

Integration Checklist

  1. Automated build passing after merge? Y
  2. No merge conflicts (or resolution trivial/obvious)? Y
  3. All master branches up-to-date after merge? N/A
  4. Project-specific information isolated to appropriate branches? Y

@charlesh88 charlesh88 mentioned this pull request Aug 19, 2015
@larkin larkin deleted the open70 branch October 9, 2015 16:56
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.

3 participants