Bug 133265 - [UI] Please increase the 'needs save' indicator visibility
Summary: [UI] Please increase the 'needs save' indicator visibility
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.4.2.2 release
Hardware: All All
: medium enhancement
Assignee: Heiko Tietze
URL:
Whiteboard: target:7.3.0 target:7.2.0.0.beta2 tar...
Keywords: needsUXEval
Depends on:
Blocks: Statusbar
  Show dependency treegraph
 
Reported: 2020-05-22 07:08 UTC by Luke Kendall
Modified: 2021-12-14 14:47 UTC (History)
7 users (show)

See Also:
Crash report or crash signature:


Attachments
PNG screenshot, not especially narrow window (70.03 KB, image/png)
2020-05-22 07:08 UTC, Luke Kendall
Details
PNG screenshot showing the issue really (66.73 KB, image/png)
2020-05-22 09:00 UTC, Luke Kendall
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Luke Kendall 2020-05-22 07:08:33 UTC
Created attachment 161117 [details]
PNG screenshot, not especially narrow window

If you make the document window smaller (e.g. if you want to edit several on screen at the same time), arguably the most important indicator on the bottom of the window soon vanishes: the one indicating whether the document needs top be saved or not.

Could it please be given a much higher priority, shifting it much nearer the left margin?

Thanks for considering.

See attached screenshot of the missing indicator.
Comment 1 Luke Kendall 2020-05-22 07:09:34 UTC
Sorry, "needs to be", not "needs top be".
Comment 2 Ming Hua 2020-05-22 08:31:41 UTC
It's not going to help if you are customed of checking the infobar/statusbar, but there is indeed another "needs save" indicator that's still visible even after you narrow the window -- the "save" icon on the toolbar at the top.

In your screenshot (looks like Elementary icon theme), it's the white square icon with a downward blue arrow in it.  If your document is modified and needs saving, the color of the arrow turns red.  Other icon themes show it differently, but the save icon changes for all of them (i.e. showing two different icons depending on the document status).
Comment 3 Rizal Muttaqin 2020-05-22 08:54:11 UTC
If you g(In reply to Ming Hua from comment #2)
> 
> In your screenshot (looks like Elementary icon theme), it's the white square
> icon with a downward blue arrow in it.  If your document is modified and
> needs saving, the color of the arrow turns red.  Other icon themes show it
> differently, but the save icon changes for all of them (i.e. showing two
> different icons depending on the document status).

You are right. 

For this elementary icon case, sometime I feel the save state with only changing color has visibility issue which make it less prominent to user than other icons which has an right bottom depicted indicator. Please feel free to suggest your idea.
Comment 4 Luke Kendall 2020-05-22 09:00:15 UTC
Created attachment 161123 [details]
PNG screenshot showing the issue really

I apologise, I forgot that I had widened the windows so the Save indicator appeared.
I attach a fresh screenshot that really shows the problem.

It's good to know there's another indicator - the colour of the Save icon at the top.

My suggestion though is to move it so it's the leftmost indicator displayed, before even the "Page N of M" text. I think it's the most important indicator.
Comment 5 Heiko Tietze 2020-05-25 10:15:25 UTC
There is no such thing as priority as introduced for the Notebookbar. But the SB sections can be mandatory. When shrinking the window width, it starts with the first section from left that has set mandatory to false and continues. If one of the following sections is very large, eg. a telling page style, it might be shown again if this section is hidden. Anyway, the indicator will be shown now; patch submitted.
Comment 6 Luke Kendall 2020-05-25 12:18:39 UTC
By "priority" I should perhaps have said "likelihood of being visible".
In other words, probably move it to the left so it's still visible even if the user narrows the window.
Comment 7 Commit Notification 2020-05-25 18:11:39 UTC
Heiko Tietze committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/194538345691272bf9d7c642f069cc1061a0b4d9

Resolves tdf#133265 - Save state indicator must not be hidden

It will be available in 7.0.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 8 Commit Notification 2020-05-26 10:51:12 UTC
Stephan Bergmann committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/4659c2fd1972156bbd834d7f3e7bc8741aace684

Revert "Resolves tdf#133265 - Save state indicator must not be hidden"

It will be available in 7.0.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 9 Commit Notification 2021-06-28 15:38:12 UTC
Heiko Tietze committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/0707380f05d4cce2d17eb74c4b871dcf17d3ac1f

Resolves tdf#133265 - State indicator as first item

It will be available in 7.3.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 10 Commit Notification 2021-06-29 08:51:11 UTC
Heiko Tietze committed a patch related to this issue.
It has been pushed to "libreoffice-7-2":

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

Resolves tdf#133265 - State indicator as first item

It will be available in 7.2.0.0.beta2.

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 11 Stéphane Guillou (stragu) 2021-12-12 14:37:47 UTC
adding meta bug.

I would like to verify as fixed, and I see the icon on the left of the statusbar in the version below, but reducing the width of the window still hides the icon fairly early on, even before any other element of the status bar is hidden.

Version: 7.2.4.1 / LibreOffice Community
Build ID: 27d75539669ac387bb498e35313b970b7fe9c4f9
CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded

I would say the original issue was not completely fixed: "If you make the document window smaller [...], arguably the most important indicator on the bottom of the window soon vanishes"
Comment 12 Commit Notification 2021-12-13 13:08:23 UTC
Heiko Tietze committed a patch related to this issue.
It has been pushed to "master":

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

Resolvces tdf#133265 - Mandatory save status

It will be available in 7.4.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 13 Heiko Tietze 2021-12-13 13:20:35 UTC
Back when I made the patch it was not possible to set the mandatory flag to true, see https://gerrit.libreoffice.org/c/core/+/126714. Apparently it works now but not sure.
Comment 14 Stéphane Guillou (stragu) 2021-12-14 14:41:16 UTC
Thanks Heiko!

Verified as fixed in:

Version: 7.4.0.0.alpha0+ / LibreOffice Community
Build ID: 250e4886d85a7e131da76f181b3fa7be02d1a76d
CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded

Save status is persistent when reducing the window's width.
Comment 15 Commit Notification 2021-12-14 14:47:25 UTC
Heiko Tietze committed a patch related to this issue.
It has been pushed to "libreoffice-7-3":

https://git.libreoffice.org/core/commit/46313dd4fe80737bd2ba2359d5c70c1b2b2c59ea

Resolvces tdf#133265 - Mandatory save status

It will be available in 7.3.0.0.beta2.

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.