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.

None yet

3 participants