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

Glossary links #788

Merged
merged 76 commits into from
Oct 15, 2015
Merged

Glossary links #788

merged 76 commits into from
Oct 15, 2015

Conversation

emileighoutlaw
Copy link
Contributor

Adds glossary links

Addresses sentence-case capitalization issues

Resolves /issues/555#issuecomment-143491873

@emileighoutlaw
Copy link
Contributor Author

Okay @noahmanger! Thanks for your great edits and patience. I think this is ready for your review (again).

@@ -1,4 +1,4 @@
<option value="">Select district</option>
<option value="">Select <span class="term" data-term="District">district</span></option>
Copy link
Contributor

Choose a reason for hiding this comment

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

This is an option in the select, so shouldn't have a span.

@emileighoutlaw
Copy link
Contributor Author

Well the silver lining of you having to look at this a million times @noahmanger is that I'm learning a ton about HTML. Made your edits, and thank you.

Conflicts:
	static/templates/committees.hbs
	static/templates/receipts.hbs
	templates/partials/committee/disbursements-tab.html
@emileighoutlaw
Copy link
Contributor Author

ping @noahmanger Anything I need to do?

@noahmanger
Copy link
Contributor

I don't think so...waiting for tests to pass.

@emileighoutlaw
Copy link
Contributor Author

Awesome! Just wanted to make sure I didn't drop the ball somehow. :)

Conflicts:
	templates/partials/committee/disbursements-tab.html
	templates/partials/committee/receipts-tab.html
	templates/search.html
noahmanger pushed a commit that referenced this pull request Oct 15, 2015
@noahmanger noahmanger merged commit 2f14881 into develop Oct 15, 2015
@noahmanger noahmanger deleted the glossary-links branch October 15, 2015 17:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants