Bug 162878 - Reworked localized Impress templates look ugly in RU
Summary: Reworked localized Impress templates look ugly in RU
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
25.2.0.0 alpha0+
Hardware: All All
: medium normal
Assignee: Laurent Balland
URL:
Whiteboard: target:25.2.0 target:24.8.2 target:24...
Keywords:
Depends on:
Blocks: Templates
  Show dependency treegraph
 
Reported: 2024-09-09 05:56 UTC by Roman Kuznetsov
Modified: 2024-09-26 01:48 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
One slide from Candy template (146.56 KB, image/jpeg)
2024-09-09 05:57 UTC, Roman Kuznetsov
Details
New version of Candy template with autosize for titles (1.58 MB, application/vnd.oasis.opendocument.presentation-template)
2024-09-14 08:03 UTC, Laurent Balland
Details
New version of Freshes template with autosize for titles (1.42 MB, application/vnd.oasis.opendocument.presentation-template)
2024-09-14 12:38 UTC, Laurent Balland
Details
New version of Growing Liberty template with autosize for titles (398.38 KB, application/vnd.oasis.opendocument.presentation-template)
2024-09-14 13:05 UTC, Laurent Balland
Details
New version of Grey Elegant template with autosize for titles (60.31 KB, application/vnd.oasis.opendocument.presentation-template)
2024-09-14 13:09 UTC, Laurent Balland
Details
New version of Yellow Idea template with autosize for titles (48.66 KB, application/vnd.oasis.opendocument.presentation-template)
2024-09-14 13:17 UTC, Laurent Balland
Details
New version of Growing Liberty template with autosize for titles + fix slide 5 (398.36 KB, application/vnd.oasis.opendocument.presentation-template)
2024-09-15 13:38 UTC, Laurent Balland
Details
New version of Freshes template with autosize for titles (1.42 MB, application/vnd.oasis.opendocument.presentation-template)
2024-09-16 17:03 UTC, Laurent Balland
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Roman Kuznetsov 2024-09-09 05:56:11 UTC
Description:
Reworked localized Impress templates look ugly in RU

Steps to Reproduce:
1. Set LO's UI to Russian
2. Open a new presentation using Candy template
3. Look at slides -> it looks ugly (see attach for example)

Actual Results:
Reworked localized Impress templates look ugly in RU

Expected Results:
Reworked localized Impress templates look good in RU


Reproducible: Always


User Profile Reset: No

Additional Info:
Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: c8eb95e8407ef24436e0e8e218dce535df6bb2e5
CPU threads: 16; OS: Windows 10 X86_64 (10.0 build 19045); UI render: Skia/Raster; VCL: win
Locale: ru-RU (ru_RU); UI: ru-RU
Calc: CL threaded
Comment 1 Roman Kuznetsov 2024-09-09 05:57:38 UTC
Created attachment 196323 [details]
One slide from Candy template
Comment 2 Heiko Tietze 2024-09-09 07:50:02 UTC
Click to add Text => Для добавления текста щёлкните мышью
(https://translations.documentfoundation.org/search/libo_ui-24-2/?page=2&limit=100&q=STR_PRESOBJ_TEXT&sort_by=-priority%2Cposition)

Is it possible to make the translation shorter?
Comment 3 Mike Kaganski 2024-09-09 07:58:30 UTC
*Any* design decision that depends on localizable string length is simply invalid.
Here, it would possibly be better if the placeholder text just would fit automatically, not try to expand the frame - even if the frame is set otherwise to resize to fit text. So this could be a bug.

Of course, a small shortening of the translation could be possible, like "Нажмите, чтобы добавить текст", but that's not useful.
Comment 4 Heiko Tietze 2024-09-09 08:01:57 UTC
(In reply to Mike Kaganski from comment #3)
> *Any* design decision that depends on localizable string length is simply
> invalid.
That's quite generic. The label was previously Lorem, ipsum, dolor... and by that not translated. After clearing the strings we show the default text box content now.
Comment 5 Mike Kaganski 2024-09-09 08:08:36 UTC
(In reply to Heiko Tietze from comment #4)
> That's quite generic.

Which is why I *also* provided two other points (a thought on auto-size of placeholder text, and an unhelpful translation suggestion).
Comment 6 Roman Kuznetsov 2024-09-09 08:53:08 UTC
>auto-size of placeholder text

would be a good general solution
Comment 7 Laurent Balland 2024-09-10 07:04:34 UTC
I agree that it is ugly when you first look at this slide.
I will propose to set autosize to placeholder, and to enlarge them if possible.

Anyway, this ugly aspect disappears as soon as you enter text.
Comment 8 Laurent Balland 2024-09-14 08:03:44 UTC
Created attachment 196434 [details]
New version of Candy template with autosize for titles

This proposed new version of Candy template has autosize for titles and subtitles.
Please confirm that it fits your needs.
Report any other improvements.
Comment 9 Laurent Balland 2024-09-14 12:38:39 UTC
Created attachment 196438 [details]
New version of Freshes template with autosize for titles

This proposed new version of Freshes template has autosize for titles and subtitles.
Please confirm that it fits your needs.
Report any other improvements.
Comment 10 Laurent Balland 2024-09-14 13:05:51 UTC
Created attachment 196439 [details]
New version of Growing Liberty template with autosize for titles

This proposed new version of Candy template has autosize for titles and subtitles.
Please confirm that it fits your needs.
Report any other improvements.
Comment 11 Laurent Balland 2024-09-14 13:09:07 UTC
Created attachment 196440 [details]
New version of Grey Elegant template with autosize for titles

This proposed new version of Grey Elegant template has autosize for titles and subtitles.
Please confirm that it fits your needs.
Report any other improvements.
Comment 12 Laurent Balland 2024-09-14 13:17:39 UTC
Created attachment 196441 [details]
New version of Yellow Idea template with autosize for titles

This proposed new version of Yellow Idea template has autosize for titles and subtitles.
Please confirm that it fits your needs.
Report any other improvements.
Comment 13 Roman Kuznetsov 2024-09-15 08:17:33 UTC
(In reply to Laurent Balland from comment #8)
> Created attachment 196434 [details]
> New version of Candy template with autosize for titles
> 
It looks as should now! Thanks Laurent!!!
Comment 14 Roman Kuznetsov 2024-09-15 08:31:42 UTC
I checked all other new template versions, there are a pair of problems there:

Freshes template - 8th and 9th slides still have "lorem ipsum" items
Growing liberty template - 5th slide still has "lorem ipsum" items
Comment 15 Commit Notification 2024-09-15 10:52:48 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/082d15982c4dd2bebb6ab3b95fd0febaf06792c9

tdf#162878 Candy template: auto for titles

It will be available in 25.2.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.
Comment 16 Laurent Balland 2024-09-15 10:59:39 UTC
(In reply to Roman Kuznetsov from comment #14)
> I checked all other new template versions, there are a pair of problems
> there:

Thanks for your careful review.

> Freshes template - 8th and 9th slides still have "lorem ipsum" items
I don't know exactly how to fix this.
8th slide: If I add outline text, there will be some bullets. I may set subtitles, with some overformating. I'm going to think about that
9th slide: neither easy to replace with placeholders these text. Any suggestion welcome.

> Growing liberty template - 5th slide still has "lorem ipsum" items
I should replace these ones with subtitles. I forget why I did not change them before. ;-)
Comment 17 Commit Notification 2024-09-15 12:56:14 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/aa803323a6309c28188039031606fd5620c8c105

tdf#162878 Grey Elegant template: autosize for title

It will be available in 25.2.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.
Comment 18 Commit Notification 2024-09-15 12:57:17 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/2c88a374e2ade4e8614c210195415ad0e4078e4e

tdf#162878 YellowIdea template: autosize for title

It will be available in 25.2.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.
Comment 19 Laurent Balland 2024-09-15 13:38:39 UTC
Created attachment 196454 [details]
New version of Growing Liberty template with autosize for titles + fix slide 5

This version replaces Lorem Ipsum text by localized placeholders on slide 5
Comment 20 Laurent Balland 2024-09-16 17:03:15 UTC
Created attachment 196488 [details]
New version of Freshes template with autosize for titles

This new version of Freshes template modifies slide 8 (diagram): Lorem texts have been replaced with subtitle placeholders.

However, for slide 9 (table), I was not able to remove Lorem texts. I could not insert placeholders inside the table. And let placeholders over the table is not useful. Shall we simply drop this table?

@Heiko: some UX feedback would be welcome :-)
Comment 21 Heiko Tietze 2024-09-17 12:26:07 UTC
(In reply to Laurent Balland from comment #20)
> Shall we simply drop this table?

Or make it non-translatable, or use some other text like Alpha, Beta...
Comment 22 Commit Notification 2024-09-18 22:12:39 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "libreoffice-24-8":

https://git.libreoffice.org/core/commit/6a958f33c7233d300351c1e7686cced7baa7f473

tdf#162878 YellowIdea template: autosize for title

It will be available in 24.8.2.

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.
Comment 23 Commit Notification 2024-09-18 22:13:42 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "libreoffice-24-8":

https://git.libreoffice.org/core/commit/d393e9d38d8f734d52a6d64e22f57b50332478e1

tdf#162878 Candy template: auto for titles

It will be available in 24.8.2.

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.
Comment 24 Commit Notification 2024-09-18 22:13:45 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "libreoffice-24-8":

https://git.libreoffice.org/core/commit/c9e931f6e33eed4ab79ab9499f30713a65e81f39

tdf#162878 Grey Elegant template: autosize for title

It will be available in 24.8.2.

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.
Comment 25 Commit Notification 2024-09-21 09:10:46 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/5b907b2ccaa62a62ec343f62acea4a5e76bee164

tdf#162878 GrowingLiberty template: autosize for title

It will be available in 25.2.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.
Comment 26 Commit Notification 2024-09-23 13:39:21 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "libreoffice-24-8":

https://git.libreoffice.org/core/commit/842b73dc2ca0828bb0b3043b7823f4d7d172411d

tdf#162878 GrowingLiberty template: autosize for title

It will be available in 24.8.2.

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.
Comment 27 Commit Notification 2024-09-24 05:58:24 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/c2017f839162a5236afcd73636ecfa73349b9ea9

tdf#162878 Freshes template: autosize for title

It will be available in 25.2.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.
Comment 28 Laurent Balland 2024-09-24 06:01:10 UTC
I pushed Freshes in master, and open bug 163116 to fix this bug (pb with too long titles in placeholders) and separate the bug of remaining Lorem text.
Comment 29 Commit Notification 2024-09-26 01:48:12 UTC
Laurent Balland committed a patch related to this issue.
It has been pushed to "libreoffice-24-8":

https://git.libreoffice.org/core/commit/b0f6fc26231a5bfc7869645a3001088aa2e726dd

tdf#162878 Freshes template: autosize for title

It will be available in 24.8.3.

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.