Bug Tracker

Ticket #3732 (closed bug: worksforme)

Opened 6 years ago

Last modified 5 years ago

Fix for css properties problem under IE

Reported by: sanaell Owned by:
Priority: major Milestone: 1.4
Component: core Version: 1.4a2
Keywords: Cc:
Blocking: Blocked by:

Description (last modified by brandon) (diff)

it seems have a lot of bugs in Internet Explorer when we use css properties like line-height background-position etc... to fix that problem just have to change in core.js the attr function * old line *

		if ( typeof name === "string" )
			if ( value === undefined ) {
				return this[0] && jQuery[ type || "attr" ]( this[0], name );

change for *

		if ( typeof name === "string" )
			if ( value === undefined ) {
				var camelCase = name.replace(/\-(\w)/g, function(all, letter){
					return letter.toUpperCase();
				});
				return this[0] && jQuery[ type || "attr" ]( this[0], camelCase );
			}

That seems to fix a lot of issue...

Attachments

index5.html Download (1.1 KB) - added by sanaell 6 years ago.
test-case very simple

Change History

comment:1 Changed 6 years ago by davidserduke

  • need changed from Commit to Test Case

Can you give an test case with a small example of something that doesn't work as you expect?

comment:2 Changed 6 years ago by sanaell

small example : good behavior (with the fix as i put before)  http://www.90alaiyon.com/index3.html

bad behavior (with .css usage as normal)  http://www.90alaiyon.com/index4.html

Under Internet Explorer --- beware : i use a tricky .css2 just because i m not going to change the core.js for the example, but the behave is exactly like that if we change the attr function as said before

and if we put a camelcase on all .css that be buggy only few case need that...

Changed 6 years ago by sanaell

test-case very simple

comment:3 Changed 6 years ago by brandon

  • Description modified (diff)

comment:4 Changed 6 years ago by brandon

  • Milestone changed from 1.3 to 1.3.3

see also #3667

comment:5 Changed 6 years ago by brandon

  • Status changed from new to closed
  • Resolution set to fixed

This is already being done in 1.3.x

comment:7 Changed 5 years ago by brandon

  • Status changed from closed to reopened
  • Resolution fixed deleted

I believe I mis-understood the ticket. Reopening.

comment:8 Changed 5 years ago by john

  • Status changed from reopened to closed
  • Version changed from 1.2.6 to 1.4a2
  • Resolution set to worksforme
Note: See TracTickets for help on using tickets.