Capitalization in Search Facets/Categories is Title Case only?

Ecotrust / wc-data-registry

Data Registry for the West Coast Governors Alliance

Other

10 stars 8 forks source link

Capitalization in Search Facets/Categories is Title Case only? #110

Closed tchaddad closed 9 years ago

tchaddad commented 10 years ago

Looking at the Dev server, there are now a number of new additions to the list of Sources. Many of these involve acronyms in the names of the sources (e.g. NOAA, CSUMB, MPA, U.S., etc.).

Any source name that has an acronym with multiple caps is incorrectly rendered in "Title Case" as opposed to "ALL CAPS". Not sure if this is happening on the GeoPortal side or the Solr side (suspect the latter). Are we forcing Title Case anywhere on our end?

emiliom commented 10 years ago

+1. I've been meaning to point this out, too. Thanks, Tanya.

tchaddad commented 10 years ago

The problem is created by the TitleCase.js script. I'll try to find sometime this week to attempt a fix. (wc-data-registry/site_raw/_includes/js/filters/TitleCase.js)

tchaddad commented 10 years ago

The fix seems simple and I've pushed it to the Dev server where it looks like it is working well: https://github.com/tchaddad/wc-data-registry/commit/7775eb8ac7886832321db4a79bc98b86d7f0b8f9

emiliom commented 10 years ago

Great! Sure enough, it's fixed. Thanks!

emiliom commented 10 years ago

So this is fixed on Dev, and it works as expected -- thanks again Tanya! When and how will it be pushed to Production? More broadly, how do we track:

improvements to Dev that are ready for Production
when is it appropriate to push one or more improvements to Production
who is responsible and/or allowed to push to Production

I forget if we've decided any of this in the past and have it documented somewhere.

tchaddad commented 9 years ago

This issue can be closed when Tim pushes to production next week.

timglaser commented 9 years ago

Pushed to production. Closing this ticket.