Bug 154253 - Draw: UI: Names of the default layers in the modify layer dialog are not localised
Summary: Draw: UI: Names of the default layers in the modify layer dialog are not loca...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Draw (show other bugs)
Version:
(earliest affected)
6.0.0.3 release
Hardware: All All
: low minor
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Not-Localizable
  Show dependency treegraph
 
Reported: 2023-03-18 10:12 UTC by lol
Modified: 2023-04-03 13:15 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Names of the dafault layers (121.36 KB, image/png)
2023-03-18 10:12 UTC, lol
Details

Note You need to log in before you can comment on or make changes to this bug.
Description lol 2023-03-18 10:12:26 UTC
Created attachment 186043 [details]
Names of the dafault layers

In each modify layer dialogue for all three default layers is the name of the layer not localised. Although the names are not editable, they should be translatable.

1. Open Draw.
2. Open the modify layer dialogue via double-click on a layer in the layer tab bar.

Occurs in 

Version: 7.4.6.2 (x86) / LibreOffice Community
Build ID: 5b1f5509c2decdade7fda905e3e1429a67acd63d
CPU threads: 2; OS: Windows 6.1 Service Pack 1 Build 7601; UI render: Skia/Raster; VCL: win
Locale: de-DE (de_DE); UI: de-DE
Calc: threaded

and

Version: 7.4.6.2 (x64) / LibreOffice Community
Build ID: 5b1f5509c2decdade7fda905e3e1429a67acd63d
CPU threads: 4; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win
Locale: de-DE (de_DE); UI: de-DE
Calc: threaded
Comment 1 Stéphane Guillou (stragu) 2023-04-01 13:10:45 UTC
Thank you for the report.
Yes, they should use the same string that is already used in the tabs.

Reproduced French UI in:

Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 1b463f697405e64a03378fb38a32172c4d3c25e6
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: fr-FR
Calc: threaded
Comment 2 Stéphane Guillou (stragu) 2023-04-03 09:03:45 UTC
Bibisected with linux-64-6.2 repository to first bad commit 5f599610abef8efb0c0bca0593fcef941fbe2ebc which points to the fix for bug 67248 and bug 119956:

commit a9ec212c1b545d745285f0ab3119a80713b1d1f2
author	Regina Henschel <rb.henschel@t-online.de>	Fri Sep 14 18:56:23 2018 +0200
committer	Regina Henschel <rb.henschel@t-online.de>	Fri Sep 21 11:32:34 2018 +0200
tdf#67248 Use localized layer name only for display in UI
Reviewed-on: https://gerrit.libreoffice.org/60507

Regina, what are your thoughts? The greyed-out name in the dialog does look like a bug currently, and should display the localised name in my opinion. We can use the UI name there while still disallowing the use of the internal name when the field is editable.

In addition, instead of the error message string:
"The name chosen already exists."
We could use the more transparent:
"This name can't be used: it is either reserved or the name of another layer."
Comment 3 Regina Henschel 2023-04-03 13:15:20 UTC
(In reply to Stéphane Guillou (stragu) from comment #2)
> Regina, what are your thoughts? The greyed-out name in the dialog does look
> like a bug currently, and should display the localised name in my opinion.
> We can use the UI name there while still disallowing the use of the internal
> name when the field is editable.

I don't know, whether it is possible to use the UI name in the field. On the other hand you now see, which terms are internally actually used and therefore forbidden for own layers. So for me it is not a bug to show the internal names.

> 
> In addition, instead of the error message string:
> "The name chosen already exists."
> We could use the more transparent:
> "This name can't be used: it is either reserved or the name of another
> layer."

That is possible. You need to create a new STR_WARN_LAYER_NAME_DUPLICATE because the current STR_WARN_NAME_DUPLICATE is also used for line ends.