Bug 137834 - UI: Navigator content not refreshed after renaming a page/slide in Draw/Impress
Summary: UI: Navigator content not refreshed after renaming a page/slide in Draw/Impress
Status: RESOLVED DUPLICATE of bug 106613
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Draw (show other bugs)
Version:
(earliest affected)
7.1.0.0.alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-10-28 18:53 UTC by Telesto
Modified: 2020-10-29 08:50 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2020-10-28 18:53:17 UTC
Description:
UI: Navigator content not refreshed after renaming a page/slide in Draw/Impress

Steps to Reproduce:
1. Open Draw
2. Set sidebar to Navigator
3. Right Click they slide/page panel & select rename
4. Type something and apply
5. Notice the name not being updating in navigator
6. Switch deck.. still not updating

Actual Results:
Navigator isn't refreshed after page name change

Expected Results:
Should be so


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.1.0.0.alpha0+ (x64)
Build ID: 94e6e140491de31c0788c91af855a75a3bb12709
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL
Comment 1 Telesto 2020-10-28 18:55:50 UTC
Also found in
6.0

Changing deck triggers and update with 4.4.7.2 (so this partly a regression)

Adding a new slide, and the deck is updated
Comment 2 Telesto 2020-10-28 18:57:55 UTC
@Jim
This is something you might be interested in or an easy hack, I guess
Comment 3 Jim Raykowski 2020-10-29 06:26:32 UTC
Hi Telesto,

I repro the sidebar Navigator not updating. The F5 floating Navigator updates as expected.

Here is a patch that makes sidebar Navigator update as expected:

https://gerrit.libreoffice.org/c/core/+/104988
Comment 4 Timur 2020-10-29 07:03:19 UTC

*** This bug has been marked as a duplicate of bug 106613 ***
Comment 5 Timur 2020-10-29 08:06:45 UTC
Let me explain QA: 
Normally we mark newer a duplicate of older. 
Sometimes we reverse, when fix is in the newer bug because it's more simple and better defined. 
But here as times before, Telesto made a duplicate although better defined and easily searchable bug is present. So reverse logic is not justified here. 
Invited dev spent time to find out what's already known, difference between sidebar Navigator and floating Navigator.
Comment 6 Telesto 2020-10-29 08:50:12 UTC
(In reply to Timur from comment #5)
> Let me explain QA: 
> Normally we mark newer a duplicate of older. 
> Sometimes we reverse, when fix is in the newer bug because it's more simple
> and better defined. 
> But here as times before, Telesto made a duplicate although better defined
> and easily searchable bug is present. So reverse logic is not justified
> here. 

Yes, I missed they duplicate.. and yes, this is additional work for DEV to adjusting the bug number.. 

> Invited dev spent time to find out what's already known, difference between
> sidebar Navigator and floating Navigator.

They STR is not that bad, I think. I assume Developers check stuff themselves anyhow. In bug 106613 you need to read through different comments too. So I assume it's OK

Also they bottom line: it's solved.