281165 - getSelectedItem selects wrong property because of typo

Status: RESOLVED FIXED

(Other Applications :: DOM Inspector, defect)

Description

[Frank Wein [:mcsmurf]]

While browsing through some source, i came across this line:
277     return tree.contentView.getItemAtIndex(tree.curentIndex);

I'm quite sure this should be currentIndex (and not curentIndex), also i don't
know if this coding error here has broken anything (that code has been there for
almost 3 years now).

Comment 1

[Frank Wein [:mcsmurf]]

Attachment: Patch

Fixes bug and includes some whitespace changes

Comment 2

[Frank Wein [:mcsmurf]]

Attachment: Patch 2 (diff -uw)

Improved wording so now it is clear what target is, etc.

Comment 3

[Frank Wein [:mcsmurf]]

Attachment: Patch 3 (diff -uw)

forgot to save :/

Comment 4

[Frank Wein [:mcsmurf]]

Attachment: Patch v4 (diff -uw)

Comment 5

[timeless]

Comment on attachment 173495 [details] [diff] [review]
Patch v4 (diff -uw)

we really need some localization notes, e.g., is "target" localizable, if so
what needs to be synced?

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 173495 [details] [diff] [review]
Patch v4 (diff -uw)

sr=bzbarsky.  Please let me know if this needs checking in.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Patch checked in.