Bug 90242 - SIDEBAR: Navigator in Draw/Impress incorrectly naming, numbering and iconifying objects
Summary: SIDEBAR: Navigator in Draw/Impress incorrectly naming, numbering and iconifyi...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Draw (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: low trivial
Assignee: Not Assigned
URL:
Whiteboard: target:24.8.0
Keywords:
Depends on:
Blocks: Navigator
  Show dependency treegraph
 
Reported: 2015-03-26 04:24 UTC by Yousuf Philips (jay) (retired)
Modified: 2024-03-10 01:44 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
screenshot (114.57 KB, image/png)
2015-03-26 04:24 UTC, Yousuf Philips (jay) (retired)
Details
demo of effort to improve unnamed custom shape names in (2.77 MB, video/x-matroska)
2023-06-03 03:42 UTC, Jim Raykowski
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yousuf Philips (jay) (retired) 2015-03-26 04:24:53 UTC
Created attachment 114361 [details]
screenshot

Looking in the navigator tab of the sidebar in impress and draw, we can see that things are incorrectly being labelled as 'Shape X', even when they are not shapes. Wrong icons are also placed beside the incorrectly named shapes. Even the numbering is off when it deals with unnamed shapes, as can be seen in the attached screenshot, as the numbering gets reset when it jumps into a group.

Version: 4.5.0.0.alpha0+
Build ID: 38c22ed2b8b8f3530b8a0fe5a23a329e8e511c76
TinderBox: Linux-rpm_deb-x86@45-TDF, Branch:master, Time: 2015-03-24_13:02:39
Comment 1 Cor Nouws 2015-03-26 09:23:36 UTC
A text box also is some sort of 'shape' ?
Comment 2 Yousuf Philips (jay) (retired) 2015-03-26 11:17:08 UTC
(In reply to Cor Nouws from comment #1)
> A text box also is some sort of 'shape' ?

Yes it could be considered a shape, thought it is normally referred to as a frame, it should be labelled as a Text box/frame similar to how we label it 'Text Frame' in the statusbar.

Same goes for inserting an image being labelled as a shape when the statusbar displays 'Bitmap'.

If a user is to distinguish between all these objects being labelled 'Shape' in navigator, unless he names each of them himself, he wouldnt know what it was.
Comment 3 Buovjaga 2015-03-31 09:12:58 UTC
Confirmed.

Win 7 Pro 64-bit Version: 4.5.0.0.alpha0+
Build ID: 8c3cf9dd48e40604867d3a28bddaccd65142df17
TinderBox: Win-x86@62-TDF, Branch:MASTER, Time: 2015-03-27_15:15:18
Locale: fi_FI
Comment 4 QA Administrators 2016-09-20 09:32:12 UTC Comment hidden (noise, obsolete)
Comment 5 QA Administrators 2019-12-03 14:35:00 UTC Comment hidden (noise)
Comment 6 Buovjaga 2020-11-13 17:09:46 UTC
Still repro

Arch Linux 64-bit
Version: 7.1.0.0.alpha1+
Build ID: c54e1c22f30c23d00e2fe7521217569fcec59cc4
CPU threads: 8; OS: Linux 5.9; UI render: default; VCL: kf5
Locale: fi-FI (fi_FI.UTF-8); UI: en-US
Calc: threaded
Built on 13 November 2020
Comment 7 QA Administrators 2022-11-14 03:32:56 UTC Comment hidden (noise)
Comment 8 Stéphane Guillou (stragu) 2023-05-09 13:18:39 UTC
In a recent master build, situation is slightly better:
- groups have "(Group object)" appended, with a dedicated icon
- text boxes have "(Text Frame '<contents>')" appended, but still the generic shape icon
- found no issue in numbering

Other objects not mentioned previously:
- QR/barcode labelled as "Shape <number> (SVG)" (but correct image icon)
- tables get "Shape <number> (Table)", but still the generic shape icon
- fontwork are automatically assigned a name based on their style name (so no consistency whatsoever). Removing it reveals the default "Shape" built-in name.
- formula objects, charts are assigned a generic "Object <number>" name, are shown in "Named Shapes" but editing their names shows an empty field (seems like a bug in itself)

My summary of issues:
- The word "Shape" is used as a default, but should probably be replaced for many objects, even if the existing appended parentheses clarify what it is (e.g. QR/barcodes, tables, images, text frames)
- Inconsistency in how some objects are automatically given a name (e.g. fontwork objects) while others don't get one. Default should be no custom name.
- Better icons needed for various objects (e.g. tables, formulas, charts)
- Some objects listed when using the Navigator's "Named shapes" filter but "context menu > Name" shows an empty field

Jim, maybe something of interest for you here?

Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: b10417eb1e5a4a6959e7fc1cdd9819e5b09d39a4
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded
Comment 9 Jim Raykowski 2023-06-03 03:42:45 UTC
Created attachment 187687 [details]
demo of effort to improve unnamed custom shape names in
Comment 10 Jim Raykowski 2023-06-03 03:48:07 UTC
(In reply to Jim Raykowski from comment #9)
> Created attachment 187687 [details]
> demo of effort to improve unnamed custom shape names in
Draw/Impress and improve Fontwork automatic naming in Writer/Calc

WIP patch:
https://gerrit.libreoffice.org/c/core/+/152568
Comment 11 Stéphane Guillou (stragu) 2024-03-07 06:42:05 UTC
(In reply to Stéphane Guillou (stragu) from comment #8)
> - Some objects listed when using the Navigator's "Named shapes" filter but
> "context menu > Name" shows an empty field
Tracked in bug 138866.

(In reply to Jim Raykowski from comment #9)
> Created attachment 187687 [details]
> demo of effort
Jim, this looks great, thank you. Any chance in reviving the patch? I'd be happy to test it now.
Comment 12 Jim Raykowski 2024-03-08 00:30:03 UTC
(In reply to Stéphane Guillou (stragu) from comment #11)
> Jim, this looks great, thank you. Any chance in reviving the patch? I'd be
> happy to test it now.
Revived :-)
Comment 13 Stéphane Guillou (stragu) 2024-03-08 04:33:09 UTC
Thanks Jim! :)

Testing the patch and reviewing my comment 8 list:

(In reply to Stéphane Guillou (stragu) from comment #8)
> - The word "Shape" is used as a default, but should probably be replaced for
> many objects, even if the existing appended parentheses clarify what it is
> (e.g. QR/barcodes, tables, images, text frames)
Still an issue. (Fixing this would also require changing the strings "Named Shapes" and "All Shapes" from the visibility dropdown. I guess replacing "Shapes" by "Objects" would be the obvious solution.)
> - Inconsistency in how some objects are automatically given a name (e.g.
> fontwork objects) while others don't get one. Default should be no custom
> name.
Solved by patch.
> - Better icons needed for various objects (e.g. tables, formulas, charts)
Still an issue. Because objects are grouped by page, extra visual hints are needed to navigate documents with dozens of objects. Currently, we can only tell apart 3 categories: images / OLE objects (math, chart) / everything else. In my opinion, we should have different icons at least for formulas, charts and tables.

Follow-up on what your patch focuses on: wondering if you could have the same kind of type appended to "Object N" for unnamed charts and formulas?
Comment 14 Stéphane Guillou (stragu) 2024-03-08 04:51:22 UTC
(and to be clear: I think we should eventually split this report into bitesize chunks. I just thought we could follow up on your "automatic naming" patch before splitting whatever there is left to do.)
Comment 15 Stéphane Guillou (stragu) 2024-03-08 04:55:36 UTC
Note also bug 90241, related to naming objects.
Comment 16 Commit Notification 2024-03-10 01:44:44 UTC
Jim Raykowski committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/1fd359790ed4a27902248d92ee343a4aeaf63c8e

tdf#90242 Navigator: Improve custom shape naming

It will be available in 24.8.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.