aboutsummaryrefslogtreecommitdiffstats
path: root/src/attributes.js
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Non-existent attribute for jQuery.attr no longer needs to check for "undefined"timmywil2011-04-031-4/+4
| | | | | | | | | | | | | | | | | | - Remove an unnecessary var - Use variable in removeAttr for better minification
| * | Performance enhancement switching nodeType to a vartimmywil2011-04-031-8/+10
| | |
| * | Pass jslint, 2 missing semicolonstimmywil2011-04-031-2/+2
| | |
| * | Style edits according to comments from John and rwaldron.timmywil2011-04-031-9/+19
| | |
| * | Don't use extend when setting the action attrHook for IE6/7timmywil2011-04-031-2/+2
| | |
| * | No, don't return this.timmywil2011-04-031-2/+0
| | |
| * | Simplify jQuery.removeAttr and return thistimmywil2011-04-031-5/+6
| | |
| * | Full test suite now passes in all browsers! There are probably some tweaks ↵timmywil2011-04-031-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | we can make to shorten and simplify. - removeAttr now only uses setAttribute if camelCase setAttribute is not supported + Might want to rename jQuery.support.getSetAttribute - tabIndex is a special case now for hooks where undefined should be returned. + Should we be checking if hooks returns undefined? undefined might be the desired return value in future hooks. As of now, tabIndex is the only one that needs it, but the test suite will still pass if we don't check if hooks are undefined.
| * | Continuing IE7 testing, conditional attr fixes and hooks with feature ↵timmywil2011-04-031-67/+65
| | | | | | | | | | | | testing. Will figure out a way to shorten after the test suite passes.
| * | Fix feature test, accidentally got rid of closure endtimmywil2011-04-031-5/+6
| | |
| * | Didn't actually need the hooks anymoretimmywil2011-04-031-16/+9
| | |
| * | First proposed solution for IE6/7 get/setAttribute quirks. Needs more ↵timmywil2011-04-031-9/+30
| | | | | | | | | | | | testing, but solves some issues
| * | Speed up hasAttr a littletimmywil2011-04-031-11/+13
| | |
| * | Now passes in IE8, changed around $.hasAttr and switched the attrHook for ↵timmywil2011-04-031-11/+28
| | | | | | | | | | | | selected to be a propHook
| * | Make the new attr/prop changes pass the test suite (in Webkit). There are ↵timmywil2011-04-031-20/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | still errors in IE. + Added hooks for selected, checked, readonly, disabled to removeAttr if set to falsey + Removed all attrs from attrFix, these aren't needed for setAttribute + If prop is used for class, do we want a propFix for class? - We could just assume the user should know to use className with prop. I've done the latter for now. + Created tests for $.fn.prop and $.fn.removeProp - Actually all I did was change broken attr tests to prop where it made sense.
| * | Very crude first pass at splitting apart the attr/prop logic. Also adding in ↵jeresig2011-04-031-103/+165
| |/ | | | | | | attrHooks/propHooks. All of it is completely untested.
* / Very crude first pass at splitting apart the attr/prop logic. Also adding in ↵jeresig2011-03-061-103/+165
|/ | | | attrHooks/propHooks. All of it is completely untested.
* Make sure .val() works after form.reset() in IE. Fixes #2551.rwldrn2011-02-101-0/+5
|
* Change the way jQuery.data works so that there is no longer a chance of ↵Colin Snover2011-01-091-2/+2
| | | | collision between user data and internal data. Fixes #6968.
* Clean trailing whitespace from all files.Colin Snover2010-12-301-1/+1
|
* Fix attr() problems in IE with attribute nodes. Followup to commit b50f41a2.Anton M2010-12-071-2/+2
| | | | | IE can't add properties on an attribute-node element, thus we handle it like text-/comment-nodes.
* Merge branch 'restrictAttr' of https://github.com/jitter/jquery into ↵John Resig2010-12-061-64/+71
|\ | | | | | | jitter-restrictAttr
| * Make sure attr() works on non DOM element nodes. Fixes #7202, #7451, #7500.Anton M2010-11-131-1/+6
| | | | | | | | Also added some tests for this attr(name), attr(name, value) and removeAttr(name).
| * Partially revert commit a64dc0405064d68c7b7cd0f0fc8ea60086cbcd21.Anton M2010-11-101-64/+66
| | | | | | | | | | | | | | Removing the conditional was too liberal as now attr() would even work on plain javascript objects. Keeping a check to make sure it is at least a DOM Node is appropriate. Using nodeName as in the commit which introduced the nodeType === 1 check seems plausible.
* | Handle carriage return characters in className property. Fixes #7673.Anton M2010-12-011-1/+1
|/
* Remove conditional that prevents attr from working on non-Element nodes. ↵Colin Snover2010-11-091-66/+63
| | | | Fixes #7451.
* Make sure that when multiple variables are being declared that assignments ↵John Resig2010-11-091-3/+8
| | | | are each done on their own line.
* Moved jQuery.props to attributes since it was only used in support. Fixes ↵Alex Sexton2010-10-251-0/+13
| | | | #6897 - suggestion by dmethvin
* Coerce all array values to strings before comparison in val(). Fixes bug #7123.Colin Snover2010-10-081-0/+4
|
* Make .attr(name, null) equivalent to removeAttr(name). (Was roughly this ↵John Resig2010-09-281-1/+8
| | | | before - but is now consistent across platforms). Fixes #6341.
* Some minor cleanup of the last commit.jeresig2010-09-271-1/+1
|
* Add a feature test for options inside a disabled select. Follow-up to ↵jeresig2010-09-271-1/+2
| | | | 157a383dae5335ef1056d3818d7dd70ac81c25a7.
* Only do the hasAttribute check if the method exists - IE falls back to the ↵jeresig2010-09-271-1/+1
| | | | === null check below which allows this to still work. Follow-up to cb40495b21bcb7802f3ab6ae0f837f2bf5b385ed - thanks to @jitter for the catch.
* Bug fix for commit 2c4b20809e5d32e916c479c9b63a6b7528c880ce - technique ↵jeresig2010-09-271-1/+1
| | | | wasn't working as expected in WebKit browsers. Thanks to @jitter for the bug fix.
* Make .val(undefined) == .val("") and chainable; fixes #4130.dmethvin2010-09-251-4/+5
| | | | Ensure .val(null) sets an empty string on IE6/7; fixes #5163.
* Don't have .val() return selected-but-disabled options, or selected options ↵Dave Methvin2010-09-241-2/+4
| | | | inside a disabled optgroup. Doesn't change the .val() returned for a disabled select. Fixes #3240, adapted from Nathan Hammond's patch there.
* Some minor lint fixes for the RegExp.jeresig2010-09-221-4/+4
|
* Applied the RegExp issues reported by Jeff Robinson here: ↵jeresig2010-09-221-5/+5
| | | | http://jmrware.com/articles/2010/jqueryregex/jQueryRegexes.html Additionally broke out all remaining inline RegExp. Fixes #7062.
* Make sure that jQuery works even when the individual modules are loaded ↵jeresig2010-09-081-0/+4
| | | | separately AND jQuery.noConflict(true) is used. Fixes #7011.
* First pass at unifying the various CSS methods in jQuery (jQuery.style, ↵John Resig2010-09-051-4/+0
| | | | jQuery.curCSS, and jQuery.css are now all under jQuery.css).
* Make sure that missing attributes return undefined in Blackberry 4.7. Fixes ↵John Resig2010-08-241-0/+6
| | | | #6938.
* .val() on empty options returns the text value rather than the value in ↵John Resig2010-08-231-1/+4
| | | | Blackberry 4.7. Fixes #6932.
* Fixing getting/setting classes and makeArray(RegExp) for Blackberry 4.7. ↵John Resig2010-08-231-1/+2
| | | | Fixes #6930, #6931.
* Made it so that you no longer need to build jQuery in order to run the test ↵jeresig2010-03-231-1/+1
| | | | suite (but you'll still need a checkout of QUnit and Sizzle, at least).
* No need to run trim on every class iteration in addClass, thanks for the ↵jeresig2010-02-131-2/+3
| | | | heads-up Leeoniya.
* Make sure that no extra whitespace is leftover after an addClass. Fixes #6050.jeresig2010-02-131-1/+1
|
* spelling mistake (insead)tomviner2010-02-061-1/+1
|
* Make sure the resulting className from removeClass is nicely trimmed. Fixes ↵jeresig2010-02-011-1/+1
| | | | #5988.
* Centralize the logic for throwing exceptions. Fixes #5913.jeresig2010-01-231-1/+1
|
* Removed .removeAttr(Function), it didn't really make sense.jeresig2010-01-071-8/+1
|