search

calband / calchart-viewer

An online show viewer and continuity generator for Calchart.
calband.github.io/calchart-viewer/
2 stars 2 forks source link

Added URL parameters #156

Closed brandonchinn178 closed 9 years ago

brandonchinn178 commented 9 years ago

Resolves #150

noahsark769 commented 9 years ago

Can we call it something other than URLUtils? I feel like this is probably the only url util we're gunna need, whereas if we called it like, DOMUtils, we could throw all kinds of js based utility functions in there

brandonchinn178 commented 9 years ago

Oh ok you want that? I have another URL Util function, but I can rename it DOMUtils or even JSUtils if you want

noahsark769 commented 9 years ago

I feel like there's no need to have a million util files, so we should name it something more general. The name is up to you though!

brandonchinn178 commented 9 years ago

Wait so do you wanna merge ClassUtils, MathUtils, URLUtils, and PDFUtils?

noahsark769 commented 9 years ago

I think you can merge URL utils and class utils into js utils, the others I think are large enough categories On Fri, Nov 28, 2014 at 12:35 AM Brandon Chinn notifications@github.com wrote:

Wait so do you wanna merge ClassUtils, MathUtils, URLUtils, and PDFUtils?

— Reply to this email directly or view it on GitHub https://github.com/calband/calchart-viewer/pull/156#issuecomment-64859855 .

noahsark769 commented 9 years ago

Sweet!