Issue 64886

Created attachment 36135 [details]
patch

Reassigned to requirments.

changing type to patch - as it is :-)

back from requirements to default owner.

But before adding a patch which changes UI, it has to be verified by
UserExperience where also a spec will be created (which is needed for UI changes).

Hi Frank, could you please take over this patch and discuss it with development
(CC'ed Oliver Specht). Thank you!

FL->jianhuajiao: Please add a screenshot of the navigator to illustrate the
proposed change. 

Set target

Created attachment 36787 [details]
test file

Created attachment 36788 [details]
new feature's screenshot using test file

Created attachment 36800 [details]
Mockup with adjusted spacing

Fine I have just checked the mock-up. I think we should do this, but we need
some small adjustments. I have attached a mock-up (Draw document) with an
adjusted spacing for the different levels. Please load the document and the
switch from page one to page two to see the differences. This adjusted indent
and spacing (also to the heading text) makes it easier see the different heading
levels. A colleague told me that this spacing can be set using the
SetIndent(short) from the SvTreeListBox. Please adjust this spacing.

Furthermore we have to change the interaction a little bit, because we have a
function to set the visibility level from 1 to 10 in the toolbar of the
Navigator. If this option is set to “1”, only the first level headings is shown,
if it is 2, the first and the second level is shown. Then the user can further
“explore” the document by expanding headings. This setting also defines the
default heading level shown, if the Navigator will be closed and reopened and is
also executed if being changed.

Last but not least the expand or collapsed status of a heading must be saved at
least for runtime. This would allow to close and re-open the navigator and to
update a heading from a changed heading text in the document and keep the right
status.

Created attachment 36838 [details]
new patch let "Content view" has same behavior

in the new patch
change the Single click back to orig behavior
make "Content View" has same view.
 

FL: We need to have a look at the extension. Please check if current target is o.k.

I checked the current patch and it does not save the current state of
expanded/collapsed headings - but I doubt that it makes sense to do so. After
restarting/switching the Navigator the structure and content of the headings
might have changed significantly. The expanded/collapsed state can't be restored
anyway. 
It does not enable exploring of heading levels in case of restricted display of
these levels. I think it would be best if the level is increased automatically
once a heading is expanded.

I can live with the two restrictions above - but there's another problem:
Collapsed headings are expanded automatically once the heading text is edited in
the document. That needs to be fixed.

*** Issue 36567 has been marked as a duplicate of this issue. ***

move to next release because of code freeze.

move to even more later release because of code freeze.

jianhuajiao, this patch seems to be insufficient. Can we expect that it will be
reworked? Or can we close this issue?

mba, I need time to rework.

Thanks for letting us know this. I was just asking because I wanted to know if
this issue is still worked on. 

Created attachment 43697 [details]
update for hold the expend and collapse status

the new patch depend on former patch. it uses a map to map content objects
(address) and expend status (boolean).

it should using following code the deal with root view:

-   if(!bIsRoot || ((SwContentType*)pParent->GetUserData())->GetType() ==
CONTENT_TYPE_OUTLINE)
+   if(!bIsRoot || (((SwContentType*)pParent->GetUserData())->GetType() ==
CONTENT_TYPE_OUTLINE) ||
+            (nRootType == CONTENT_TYPE_OUTLINE))

jianhuajiao, if you want us to review your patch please assign it to OS. Thank you.

review patch. thanks

Should be my issue :-)

->jianhuajiao: I still see the problem with the initially not expanded headings.
If I open the navigator for a document with lots of headings only the first
heading level is visible and I have to expand all entries manually to see them
all. Please change the default expansion to the value of 'heading levels shown'
and expand/collapse the entries if changes are made to this value. 

->os: yes. I set the default behavior as you descripted.
I think expend too many headlines in the navigate view is not good idea.
It will fill all space and we could not see other elements that parallel
with headline.

maybe expanding headline in root mode be nice? 
but it may confilct with keep status because of two behavior change.

->jianhuajiao: Maybe we need a broader audience and start a discussion on the
sw-dev mailing list. But from my point of view it would make sense to see more
than just the first level headings. 

->os: yes. we need a broader audience

->jianhuajiao: Now the discussion on the mailing list seems to be over.
I'd say we should follow Christan Jansens post from 4/12/07,
on click expand the headings completely until the 'Number of headings' is reached. 
On the other hand changes in the 'Number of headings' setting should expand all
headings up to that level. 
Your mechanism to store the state of the expanded/collapsed headings doesn't
need to be changed. It's just the dependence to the 'Number of headings' setting.

jianhuajiao: are you fine with the approach outlined by Oliver?

->os,mba:Sorry. I have got the outline. It is good. And I will implement it soon.

target 2.x

Any news on that?

Any news on that? Are you still working on this issue or should be abandon the
patch?

After 5 months without a reaction I think it's a safe bet that the patch should
be considered as rejected. Back to "enhancement".

For the time being I take that issue

according to release status meeting -> target 3.x

Created attachment 52275 [details]
Update for Drag&Drop

*** Issue 89684 has been marked as a duplicate of this issue. ***

The 3.0.0 release for opensuse 11.0, i386 is missing a useful Navigator feature
that used to be there.

In the Navigator window, under Headings, there used to be a +/- box for
expanding/collapsing hierarchy at each level. Now the +/- is at only the
Headings level. I'm editing a 250-page document with over 300 headings. Being
able to collapse the hierarchy is a real time-saver for me, as I don't have to
scroll tediously through this large list over and over.

I can see the desire for some users not to have to use +/-. Please put this
feature back into OO as an option, as both use models are very useful.
I've tried the 2nd button from the left on the 2nd row of the Navigator. All it
seems to do is indent the entire hierarchy by about a millimeter.

Thanks

skelem: where did you get your OO.o 3.0 from ? this is a go-oo feature that is
not yet up-stream; are you using go-oo ?

*** Issue 95693 has been marked as a duplicate of this issue. ***

*** Issue 79603 has been marked as a duplicate of this issue. ***

*** Issue 110585 has been marked as a duplicate of this issue. ***

Add me to cc.

In my opinion Navigator with the "+/-"-function is required in long documents, 
i.e. expanding/collapsing hierarchy at each level.
Question: why no one wants to program this change?

I am waiting for the expand/collapse feature too. In fact this is the only
reason I am forced to use the go-oo branch right now. Their implementation of
expand/collapse works fine for me.
Regarding the discussion on all headings collapsed/expanded when navigator is
opened: imho this feature is needed to work with large documents. When a large
number of headings is expanded, you spend your time scrolling through the
expanded headings finding what you are looking for. The feature is needed to
enable the user to go directly to the heading he needs by opening subheadings in
the tree hierarchy. To do this the headings should be collapsed as the default
view, same as in every outliner.

I second your opinion,pk72.
In fact , I reiterate my inquiry about fixing this tiny but important detail
which is NOT a problem under Ubuntu -- I see it working just fine when I boot
that system up.
Why wouldn't the feature be equally effective under Windows?
Thanks to you guys, in development, for your attention. You've done a great job
with OpenOffice, it is undeniable. But since my original request, two or three
product releases ago, I've been hoping to find the utility reinstated. tono
avail, and much to my dismay.

*** Issue 112585 has been marked as a duplicate of this issue. ***

Created attachment 70186 [details]
Navigator in Debinan 1:3.2.0-7ubuntu4.1, 3.6.2010 01:57:32 UTC

As you can see at the attached screenshot: It is already there!!!

OpenOffice 3.2.0
OOO320m12 Build 9483
ooo-build 3.2.0.10
Debinan 1:3.2.0-7ubuntu4.1, 3.6.2010 01:57:32 UTC

Why in Ubuntu and not for the rest of the world?

Probably because they included the patch that os rejected as incomplete?

@mba: iterative development is sometimes a good choice. :-)

Reset assigne to the default "issues@openoffice.apache.org".