94652 - allow for no multiple select in outliner

Status: VERIFIED FIXED

(Core :: XUL, defect)

Description

[(not reading, please use [e-mail] instead)]

allow for no multiple select in outliner

by default, multiple select would be on.

hyatt suggested

seltype="multiple"
seltype="single"
and if no seltype, default to multiple.

Comment 1

[Jan Varga [:janv]]

Attachment: initial fix

Comment 2

[Jan Varga [:janv]]

I assume seltype is intended for <outlinerbody>.

"b.selection.currentIndex = row.value;" at the end of the patch is removed
because  it is redundant, rangedSelect() takes care of it

Comment 3

[David Hyatt]

No, it is intended for <outliner>.

Comment 4

[Jan Varga [:janv]]

ok, I'll fix it

Comment 5

[Jan Varga [:janv]]

Attachment: new fix

Comment 6

[Brian Ryner (not reading)]

Looks good to me.  r=bryner

Comment 7

[David Hyatt]

sr=hyatt

Comment 8

[Jan Varga [:janv]]

Thanks!

Comment 9

[Jan Varga [:janv]]

I'll attach a new patch for the current trunk (just for sure)

Comment 10

[Jan Varga [:janv]]

Attachment: new patch

Comment 11

[Jan Varga [:janv]]

patch checked in

Comment 12

[[e-mail] (gone - use [e-mail] instead)]

Verified FIXED:

mac os 9.1 - 2001-09-05-08
redhat 7.1 - 2001-09-06-08
windows 2k - 2001-09-06-03

tried using shift, ctrl, and the open apple keys to invoke multiple selection...