Bug 42030 - UI: Page Preview Bar scaling slider function unclear
Summary: UI: Page Preview Bar scaling slider function unclear
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: x86-64 (AMD64) All
: medium enhancement
Assignee: Sahil Gautam
URL:
Whiteboard: target:24.8.0
Keywords: difficultyInteresting, easyHack, skillCpp
Depends on:
Blocks: Calc-UX
  Show dependency treegraph
 
Reported: 2011-10-19 23:38 UTC by Kevin Hunter
Modified: 2024-01-10 17:01 UTC (History)
8 users (show)

See Also:
Crash report or crash signature:


Attachments
PrintPreview1 with zoom buttons and scaling factor widget circled (69.07 KB, image/png)
2011-10-19 23:38 UTC, Kevin Hunter
Details
PrintPreview2 with scaling factor widget utilized (67.87 KB, image/png)
2011-10-19 23:42 UTC, Kevin Hunter
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin Hunter 2011-10-19 23:38:35 UTC
Created attachment 52567 [details]
PrintPreview1 with zoom buttons and scaling factor widget circled

In the attached "PrintPreview1", I claim that the "scaling factor" slider widget (circle (1)) is not visually clear as to it's purpose.

I originally clicked on the buttons expecting the behavior one gets when clicking the +/- buttons in circle (2).  When I did not get the desired visual appeal, I thought again that maybe I was messing with the margins.  It finally dawned on me what it was doing (see "PrintPreview2", but it took me more than a few seconds to kick myself from my preconceived notions.

I submit that this could/should be made more visually explicit before 3.5.  Unfortunately, I'm not a UI guy or I'd offer a suggestion.

This may apply to other LO components as well, but I was using Calc when I noted it.

This screenshots were made with master.  Last pulled commit:

$ git log -1 --format=oneline 5ad4d151dac1eb887d92200330e31af269d8d1fd migrate to StringRangeEnumerator in pdfexport
Comment 1 Kevin Hunter 2011-10-19 23:42:14 UTC
Created attachment 52568 [details]
PrintPreview2 with scaling factor widget utilized
Comment 2 Rainer Bielefeld Retired 2011-10-20 23:09:58 UTC
Not a particular Master problem, I also see it with LibO 3.3.3 Portable. Behavior is inherited from OOo, I also see it with OOo 3.1.1
No idea for what that is needed
- we have a zoom slider in the status bar
- no help available (see Bug 42081 - LOCALHELP for CALC 
  Page Preview Bar incomplete)

I can't find out any purpose for what that slider might be useful.

@David:
Please add findings to Help (also see "Bug 42081 - LOCALHELP for CALC Page Preview Bar incomplete").

@Kohei:
Please feel free to reassign (or reset Assignee to default) if it’s not your area or if provided information is not sufficient. Please set Status to ASSIGNED if you accept this Bug.
Comment 3 Björn Michaelsen 2011-12-23 13:22:49 UTC Comment hidden (obsolete)
Comment 4 QA Administrators 2015-02-19 15:33:05 UTC Comment hidden (obsolete)
Comment 5 Buovjaga 2015-03-07 12:05:17 UTC
Confirmed it's still the same in Calc. Giving to the design team.

Win 7 Pro 64-bit, LibO Version: 4.4.1.2
Build ID: 45e2de17089c24a1fa810c8f975a7171ba4cd432
Locale: fi_FI

Ubuntu 14.10 64-bit 
Version: 4.4.1.2
Build ID: 40m0(Build:2)
Locale: en_US
Comment 6 Robinson Tryon (qubit) 2016-08-25 04:44:47 UTC Comment hidden (obsolete)
Comment 7 Heiko Tietze 2017-11-28 15:32:45 UTC
Zoom (2) works similar to ctrl+wheel or the statusbar slider - it zooms into the document. The slider (1) scales the content, meaning you get more columns on one page with "-" or enlarge the font with "+".

The interaction has a clear feedback but the UI is missing a label "Scale" before the slider. The usability would also benefit from showing the actual scaling factor and when +/- steps become more discrete. 

Sounds like an easyhack.
Comment 8 Heiko Tietze 2018-01-03 15:13:45 UTC
The scale slider is handled in sc/source/ui/cctrl/tbzoomsliderctrl.cxx with a constant nSliderXOffset to both left and right that should be changed into an asymmetric offset to allow the output of 'Scale' before the slider, and perhaps also the current value right of it.
Comment 9 Kshitij 2018-02-04 11:12:20 UTC
Hi!I would like to work on this bug.

(In reply to Heiko Tietze from comment #7)
> The interaction has a clear feedback but the UI is missing a label "Scale"
> before the slider. The usability would also benefit from showing the actual
> scaling factor and when +/- steps become more discrete. 
> 

As I understand,I need to create label before slider and its actual value on right side in previewbar .
Comment 10 Heiko Tietze 2018-02-04 13:00:07 UTC
(In reply to Kshitij from comment #9)
> As I understand,I need to create label before slider and its actual value on
> right side in previewbar .

Exactly, there should be a translatable caption with the slider. I would also like to have the actual value on the other side. And when you touch the code and find an option to increase the steps when the user clicks on + or - that would be great. Sounds like we ideally go with a standard control, if there is one.

But having just a caption on the left side is fine at the moment.
Comment 11 Kshitij 2018-02-04 16:55:32 UTC
I have submitted the patch https://gerrit.libreoffice.org/#/c/49204/ with both caption and actual valuein right.@Heiko can you please review that :)
Comment 12 Heiko Tietze 2018-02-05 10:25:41 UTC
(In reply to Kshitij from comment #11)
> @Heiko can you please review that :)

Done on Gerrit.
Comment 13 Xisco Faulí 2018-03-08 03:42:05 UTC
A polite ping, still working on this bug?
Comment 14 Kshitij 2018-03-09 13:33:33 UTC Comment hidden (obsolete)
Comment 15 Xisco Faulí 2018-04-09 02:30:01 UTC Comment hidden (obsolete)
Comment 16 Xisco Faulí 2018-05-10 02:30:44 UTC Comment hidden (obsolete)
Comment 17 Xisco Faulí 2018-06-10 02:44:00 UTC Comment hidden (obsolete)
Comment 18 Xisco Faulí 2018-07-11 02:38:20 UTC Comment hidden (obsolete)
Comment 19 Xisco Faulí 2018-08-11 02:34:52 UTC Comment hidden (obsolete)
Comment 20 Xisco Faulí 2018-09-11 02:35:31 UTC Comment hidden (obsolete)
Comment 21 Xisco Faulí 2018-10-12 03:03:16 UTC Comment hidden (obsolete)
Comment 22 Xisco Faulí 2018-11-12 03:39:20 UTC Comment hidden (obsolete)
Comment 23 Xisco Faulí 2018-12-13 03:48:20 UTC Comment hidden (obsolete)
Comment 24 Xisco Faulí 2019-01-13 03:57:26 UTC
A polite ping, still working on this bug?
Comment 25 Xisco Faulí 2019-02-13 03:47:46 UTC Comment hidden (obsolete)
Comment 26 Luke 2021-02-17 18:24:09 UTC Comment hidden (obsolete)
Comment 27 George Bateman 2021-02-17 19:11:04 UTC
(In reply to Luke from comment #26)
> It's been 1,076 days since the assignee has responded. Marking as New.

Hi Luke, you have got the assignees mixed up. I am a new assignee with a patch at https://gerrit.libreoffice.org/c/core/+/110960 awaiting review. Marking as Assigned.
Comment 28 Xisco Faulí 2022-05-02 14:47:33 UTC
Dear  George Bateman,
This bug has been in ASSIGNED status for more than 3 months without any
activity. Resetting it to NEW.
Please assign it back to yourself if you're still working on this.
Comment 29 Julien Nabet 2023-03-23 10:52:49 UTC
Just for the record, patch in progress here:
https://gerrit.libreoffice.org/c/core/+/149419
Comment 30 Commit Notification 2024-01-10 17:01:58 UTC
Sahil committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/986b5bb3c183b038ecc1ce4e16cff1dca837932a

tdf#42030 Add UI label for page preview zoom slider

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.