Ticket #8783 (closed Bug: fixed)

Opened 3 years ago

Last modified 2 years ago

getAttribute( 'contenteditable' ) returns 'inherited' on IE7 and IE6 for elements created from code

Reported by: Reinmar Owned by: Reinmar
Priority: Normal Milestone: CKEditor 3.6.4
Component: General Version: 3.0
Keywords: IE6 IE7 Cc:

Description (last modified by j.swiderski) (diff)

For TC as below all browsers alert 'true', but IE7 and IE6 'inherited'.

<!DOCTYPE html>
<html>
<head>
	<script src="path/ckeditor.js"></script>
	<title>tc</title>
</head>
<body>
<div id="getAttributeContenteditable"></div>
<script>
	var element = CKEDITOR.document.getById( 'getAttributeContenteditable' );
	element.setAttribute( 'contenteditable', true );
	alert( element.getAttribute( 'contenteditable' ) );
</script>
</body>
</html>

Attachments

8783.patch (1.1 KB) - added by Reinmar 3 years ago.
Proposed fix for all 3 set/get/remove(Attribute) methods

Change History

comment:1 Changed 3 years ago by j.swiderski

  • Keywords IE6 IE7 added
  • Status changed from new to confirmed
  • Version set to 3.0
  • Description modified (diff)

Reproducible in IE6 and IE7 from CKE 3.0.

comment:2 Changed 3 years ago by j.swiderski

  • Summary changed from getAttribute( 'contenteditable' ) returns 'inherited' on IE7 for elements created from code to getAttribute( 'contenteditable' ) returns 'inherited' on IE7 and IE6 for elements created from code

Changed 3 years ago by Reinmar

Proposed fix for all 3 set/get/remove(Attribute) methods

comment:3 Changed 3 years ago by Reinmar

  • Owner set to Reinmar
  • Status changed from confirmed to review

comment:4 Changed 2 years ago by garry.yao

  • Status changed from review to review_passed

comment:5 Changed 2 years ago by Reinmar

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

Fixed with [7505].

comment:6 Changed 2 years ago by wwalc

  • Milestone set to CKEditor 3.6.4
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy