Ticket #3403 (closed Task: fixed)
Adding 'hasAttribute' method
| Reported by: | garry.yao | Owned by: | martinkou |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | |
| Keywords: | Confirmed Review+ | Cc: |
Description
There's some cases where it's necessary to check whether a attributed is intrinsic OR assigned dynamically on an element.
Attachments
Change History
comment:3 Changed 4 years ago by martinkou
- Keywords Review? added
This removes the ieDefaults hacks in the forms plugin cleanly.
comment:4 Changed 4 years ago by garry.yao
- Keywords Review- added; Review? removed
The new method doesn't cover certain special attributes in sense of the attached TC in IE.
comment:5 Changed 4 years ago by martinkou
- Keywords Review? added; Review- removed
I'm not sure if this is worth it - the core code size is being increased to check for fringe cases. But anyway this should pass the tests.
comment:6 Changed 4 years ago by fredck
- Keywords Review- added; Review? removed
You are right Martin, we don't need it by default into the core.
Let's proceed in this way... let's have a basic implementation in the core and then override it inside the forms plugin, for IE only.
comment:9 Changed 4 years ago by martinkou
- Status changed from assigned to closed
- Resolution set to fixed
Fixed with [3438].
Click here for more info about our SVN system.
Note: See
TracTickets for help on using
tickets.

Much of our dialog fields play the role of the getter/setter of the DOM element node attribute, and we always use the following way to check the existence of a specified attribute:
It works in most cases but has one problem: