Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
branch: master
Commits on Apr 8, 2012
  1. @shichuan
Commits on Apr 3, 2012
  1. @shichuan
  2. @shichuan

    Merge pull request #48 from StevenBlack/MoreSettingsDefaultsOptions_S…

    shichuan authored
    …emantics
    
    Whitespace normalization, followed by a semantic cleanup
Commits on Mar 30, 2012
  1. @shichuan

    Merge pull request #52 from stevearm/patch-1

    shichuan authored
    Comments were the same in the anti-pattern and preferred pattern
Commits on Mar 29, 2012
  1. @shichuan

    Merge pull request #47 from StevenBlack/SettingsDefaultsOptions_Seman…

    shichuan authored
    …tics
    
    Settings defaults options semantics
  2. @shichuan

    Merge pull request #43 from AutoSponge/master

    shichuan authored
    added design-patterns/multiton.html
Commits on Mar 28, 2012
  1. @shichuan

    Merge pull request #49 from actsasbuffoon/patch-1

    shichuan authored
    Fix typo in object-constants.html
  2. @stevearm
Commits on Mar 27, 2012
  1. @actsasbuffoon
  2. @StevenBlack

    Whitespace normalization, followed by a semantic cleanup of options, …

    StevenBlack authored
    …defaults, and settings
    
    Please see [pull request 47](shichuan#47) for some background on the semantics in-play here.
    
    Note how separating the concepts of settings, defaults, and options allows us to make the comments more succinct.  That's because the concept of "options" isn't overloaded anymore.
    
    **Options** come from the user.  **Defaults** come from the plugin.  **Settings* is what you run-with this time.
    
    Signed-off-by: Steven Black <steveb@stevenblack.com>
  3. @StevenBlack

    Fix for the potential semantic confusion between settings, defaults, …

    StevenBlack authored
    …and options.
    
    Prior to this change, these two files used settings, something called "defaultOptions", and options.
    
    Using "defaultOptions" is problematic because it does not cleanly separate defaults and options.  It needlessly confounds the two.
    
    The `namespaced-starter.html` file actually fell into this trap.  See the comment on line 36 here: shichuan@7f3d3d3#jquery-plugin-patterns/namespaced-starter.html
    
    Besides "defaults" is a better term than "defaultOptions".  This may matter for non-English native readers.
    
    THEREFORE
    options: values provided by the plugin's calling code.
    defaults: the settings provided  A-Priori by the plugin
    settings: the final, amalgamated run-time spec for the plugin
    
    An additional benefit: cleanly separating options, defaults, and settings makes these available for subsequent runs of the plugin without mutating the plugin's defaults, which can be restored at anytime without redefining the plugin.  This is not so much a problem for these two examples, but there are other jQuery plugin patterns in this collection where this definitely applies.
    
    For example, https://github.com/shichuan/javascript-patterns/blob/master/jquery-plugin-patterns/best-options.html where "options" is (for the moment) used in lieu of settings, thus destroying the user-provided options object.  I'll fix that separately.
    
    Signed-off-by: Steven Black <steveb@stevenblack.com>
  4. @StevenBlack

    Whitespace changes to better conform to jQuery coding standards, remo…

    StevenBlack authored
    …ved the mix of spaces and tabs, adjusted line spacing in these two files for consistency of conceptual segmentation, and made changes to pass jsHint
    
    The JQuery Core Style Guidelines can be found here: http://docs.jquery.com/JQuery_Core_Style_Guidelines
    
    Adjusted the in-code white-spacing for consistency -- prior to these changes there was no evident standard, or a standard had yet to be enforced.  Chose to adhere to jQuery Core Style Guidelines because students studying jQuery will come across many examples authored by core team members who, on balance, tend towards this standard.
    
    Fixed one minor jsHint-discovered error, an unnecessary semicolon at the end of a function definition.
    
    Stripped-away all trailing spaces on code and blank lines.
    
    By the way, I like your use of the leading semicolon to guard against prior concatenated code.
    
    Signed-off-by: Steven Black <steveb@stevenblack.com>
Commits on Mar 14, 2012
  1. @shichuan

    added basic and extend

    shichuan authored
  2. @shichuan

    added jquery plugin patterns

    shichuan authored
Commits on Mar 13, 2012
  1. @shichuan
  2. @shichuan

    added jquery plugin patterns

    shichuan authored
  3. @shichuan

    added jquery plugin patterns

    shichuan authored
Commits on Feb 19, 2012
  1. @shichuan

    Merge pull request #44 from mmariani/patch-1

    shichuan authored
    added example for name function expression
Commits on Feb 14, 2012
  1. @shichuan
  2. @shichuan

    Merge pull request #42 from Grom-S/cb-pattern-change

    shichuan authored
    callback pattern modification
  3. @mmariani
Commits on Feb 12, 2012
  1. @shichuan

    Merge pull request #41 from Grom-S/few-typos

    shichuan authored
    few typos corrected
Commits on Feb 11, 2012
  1. @Grom-S

    callback pattern modification

    Grom-S authored
  2. @Grom-S

    few typos corrected

    Grom-S authored
Commits on Feb 9, 2012
  1. @shichuan

    Merge pull request #39 from Grom-S/html_in_js_code

    shichuan authored
    Removed html tags from js code (typo)
Commits on Feb 4, 2012
  1. @Grom-S
Commits on Feb 2, 2012
  1. @shichuan

    Merge pull request #37 from ricardohbin/master

    shichuan authored
    Conditionals only using logical operators
  2. @shichuan

    Merge pull request #36 from shawjia/master

    shichuan authored
    I found a little typo, now fix it. please accept my pull request. thanks.
  3. Conditionals only using logical operators

    Ricardo Bin authored
  4. Conditionals only using logical operators

    Ricardo Bin authored
  5. @shawjia

    typo fixed

    shawjia authored
  6. @shichuan

    applied changes in issue #34

    shichuan authored
  7. @shichuan

    Merge pull request #35 from webpro/master

    shichuan authored
    Indentation using tabs (both HTML & JS)
Commits on Feb 1, 2012
  1. Proper indentation using tabs (both HTML and JS)

    Lars Kappert authored
Commits on Jan 31, 2012
  1. @shichuan
Something went wrong with that request. Please try again.