110156 - Eliminate all <outliner> tags and make <outliner> into <tree>

Status: VERIFIED FIXED

(Core :: XUL, defect, P1)

Description

[David Hyatt]

This bug covers the automatic conversion of outliner to tree.  All outliner
files should be renamed, and all uses of outliner tags will become tree tags.

This bug is dependent on outliner working with a content model, and it is
dependent on listbox landing, as well as all arbitrary content trees being
converted over to listbox.

Comment 1

[Peter Trudelle]

*** Bug 114135 has been marked as a duplicate of this bug. ***

Comment 2

[Aaron Leventhal]

Needed for accessibility support

Comment 3

[Peter Trudelle]

nsbeta1+ per nav triage team

Comment 4

[Peter Trudelle]

cc aaronl, add access keyword

Comment 5

[Blake Ross]

*** Bug 132020 has been marked as a duplicate of this bug. ***

Comment 6

[Joe Hewitt (gone)]

Attachment: editor patch

Comment 7

[Charles Manske]

Comment on attachment 76164 [details] [diff] [review]
editor patch

At:
-  if (tree && tree.selectedItems && tree.selectedItems.length)
+  if (tree && tree.currenmt)

What is "currenmt" ?
Should it be:
tree.treeBoxObject.selection.count 
?
the rest looks good
r=cmanske with above fix

Comment 8

[Joe Hewitt (gone)]

Attachment: patch to layout/content/dom

This patch has the changes I made to layout/content/dom.  Absent from the patch
are the many file renames I did - any file that had Outliner it it's name was
renamed to Tree.  The patch wit all that in it was 1.3MB, and not really worth
reading.

Comment 9

[Axel Hecht]

are those file renames going to be done by copies in the repository, so that we
don't loose the history info? Renaming a good deal of files should get an
extra warning announcement in the newsgroups, as it pretty much kills existing
patches.

Comment 10

[Jan Varga [:janv]]

Comment on attachment 76296 [details] [diff] [review]
patch to layout/content/dom

r=varga

Comment 11

[Brendan Eich [:brendan]]

Hewitt, leaf is your man for cvs file copying so we don't lose logs.

/be

Comment 12

[Joe Hewitt (gone)]

Last week I sent leaf a complete list of all the files that need renaming.

Comment 13

[David Hyatt]

Comment on attachment 76296 [details] [diff] [review]
patch to layout/content/dom

sr=hyatt

Comment 14

[Joe Hewitt (gone)]

Attachment: mailnews patch

Comment 15

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

Comment on attachment 76614 [details] [diff] [review]
mailnews patch

dirTree should be single selection.  once you make that change, r=sspitzer (or
sr, depending on what you need)

Comment 16

[Scott MacGregor]

Comment on attachment 76614 [details] [diff] [review]
mailnews patch

sr=mscott modulo seth's comment. I'm also assuming that this mailnews patch
incorporates any issues mailnews QA found when they ran your test bits.

Comment 17

[Hixie (not reading bugmail)]

The following components of the application are broken on Linux depend builds:
   * The bookmarks sidebar (clicks don't work)
   * The document inspector sidebar (it's empty)
   * The filepicker drop down widget (it's empty)
   * The folder pane in mail (it's not sorted, clicks don't work)
   * The location bar (hitting enter doesn't do anything)

I'm trying a clobber build now.

Comment 18

[Hixie (not reading bugmail)]

All the above are fixed in a clobber build. Thanks for fixing it, dbaron. :-)

Comment 19

[Joe Hewitt (gone)]

fixed

Comment 20

[lchiang]

http://bugzilla.mozilla.org/show_bug.cgi?id=134729 may be a regression from this
fix for the fix for 110155.  Just fyi in case those items apply to other product
areas.

Comment 21

[John Morrison]

long gone. verified.