Skip to content
Permalink
Browse files

Fix #13223. Re-allow leading space in HTML. Close gh-1264.

  • Loading branch information
dmethvin committed May 9, 2013
1 parent 0135a22 commit 00eafdf028f7730665ce1c05ab44e3f0bc80fbc2
Showing with 8 additions and 14 deletions.
  1. +1 −1 src/core.js
  2. +7 −1 test/unit/core.js
  3. +0 −12 test/unit/selector.js
@@ -55,7 +55,7 @@ var
// A simple way to check for HTML strings
// Prioritize #id over <tag> to avoid XSS via location.hash (#9521)
// Strict HTML recognition (#11290: must start with <)
rquickExpr = /^(?:(<[\w\W]+>)[^>]*|#([\w-]*))$/,
rquickExpr = /^(?:\s*(<[\w\W]+>)[^>]*|#([\w-]*))$/,

// Match a standalone tag
rsingleTag = /^<(\w+)\s*\/?>(?:<\/\1>|)$/,
@@ -33,7 +33,7 @@ test("jQuery()", function() {
div = jQuery("<div/><hr/><code/><b/>"),
exec = false,
lng = "",
expected = 20,
expected = 22,
attrObj = {
"text": "test",
"class": "test2",
@@ -108,6 +108,12 @@ test("jQuery()", function() {

equal( jQuery(document.body).get(0), jQuery("body").get(0), "Test passing an html node to the factory" );

elem = jQuery(" <em>hello</em>")[0];
equal( elem.nodeName.toLowerCase(), "em", "leading space" );

elem = jQuery("\n\n<em>world</em>")[0];
equal( elem.nodeName.toLowerCase(), "em", "leading newlines" );

elem = jQuery("<div/>", attrObj );

if ( jQuery.fn.width ) {
@@ -73,18 +73,6 @@ test("disconnected nodes - jQuery only", function() {
equal( $opt.is(":selected"), true, "selected option" );
});

test("jQuery only - broken", 1, function() {
raises(function() {
// Setting context to null here somehow avoids QUnit's window.error handling
// making the e & e.message correct
// For whatever reason, without this,
// Sizzle.error will be called but no error will be seen in oldIE
jQuery.call( null, " <div/> " );
}, function( e ) {
return (/syntax.err/i).test( e.message );
}, "leading space invalid: $(' <div/> ')" );
});

testIframe("selector/html5_selector", "attributes - jQuery.attr", function( jQuery, window, document ) {
expect( 35 );

0 comments on commit 00eafdf

Please sign in to comment.
You can’t perform that action at this time.