mjtworks / jquery-ui-map

Automatically exported from code.google.com/p/jquery-ui-map
0 stars 0 forks source link

Source is unmaintainaible #28

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hi, first of all, thanks for this lib. It has helped me build a map more easily.

I think using that "unmaintainably short" variable names is making no one a 
favor.

I would recommend you to give a look at uglyfier, if you haven't already.

https://github.com/lautis/uglifier

It will allow you to use human-understandable variable names in the source code 
for humans, and then mangle them (selectively) into 1-letter variable names for 
computers, when minimizing.

In any case, thanks and regards!

Original issue reported on code.google.com by kik...@gmail.com on 5 Jan 2012 at 5:50

GoogleCodeExporter commented 9 years ago
I've been thinking of adding some build tools for the minifications. Now, 
unfortunately, I've been accustomed to the a, b, c variable names. The 3.0 
release will have proper names tho :).

Original comment by johansalllarsson on 10 Jan 2012 at 2:32

GoogleCodeExporter commented 9 years ago

Original comment by johansalllarsson on 15 Feb 2012 at 3:28

GoogleCodeExporter commented 9 years ago
It's far better ! Thanks

There may be a problem with this method comment (where is namespace parameter?) 
:
{{{
/**
         * @param namespace:string
         * @param name:string
         * @param prototype:object
         */
        $.a = function(name, prototype) {
}}}

Original comment by alexandr...@stadline.com on 15 Feb 2012 at 3:42

GoogleCodeExporter commented 9 years ago
hehe, i seem to have forgot to change that when i made some changes. It's 
changed :)

Original comment by johansalllarsson on 15 Feb 2012 at 3:49