Issue 50376 - no visual feedback if Page Pane is focused
Summary: no visual feedback if Page Pane is focused
Status: CLOSED FIXED
Alias: None
Product: Draw
Classification: Application
Component: ui (show other issues)
Version: 680m106
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.2
Assignee: eric.savary
QA Contact: issues@graphics
URL:
Keywords: accessibility
Depends on:
Blocks:
 
Reported: 2005-06-06 14:03 UTC by Frank Schönheit
Modified: 2010-01-08 09:16 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
attached picture (465.50 KB, image/jpeg)
2009-08-12 15:01 UTC, christoph.lukasiak
no flags Details
suggested patch (3.74 KB, patch)
2009-09-03 20:54 UTC, Frank Schönheit
no flags Details | Diff
suggested patch (3.74 KB, patch)
2009-09-03 21:11 UTC, Frank Schönheit
no flags Details | Diff
yet another patch (4.55 KB, patch)
2009-09-04 09:41 UTC, Frank Schönheit
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description Frank Schönheit 2005-06-06 14:03:19 UTC
In any draw document, with a visible page pane (View|Page Pane), there is no
visual feedback when the page pane is focused. That is, the user cannot
distinguish wether the focus is currently in the document or the page pane.

Not only that this violates Accessibility rules, it also yields subtle problems:
If you marked a shape in the document, it's virtually impossible to predict what
will happen with the next "Delete" key stroke: You might delete the marked
shape, OR you might delete the entire current page - depending on where your
focus is. You can only know by trying ...
Comment 1 Frank Schönheit 2005-06-06 14:05:00 UTC
fs->cj: We once talked about this, and you asked for an issue so you can place
it in your queue :).

Strictly, we have two issues here: The missing focus indication (which is an
accessibility and usability problem), and the shape's mark handles not being
hidden when the document view is not focused. Not sure if you want to deal with
the latter in a separate issue.
Comment 2 mdxonefour 2005-09-29 11:09:31 UTC
re-targeted to OOo 3.0
Comment 3 christian.jansen 2006-08-07 13:41:00 UTC
Retargeted to 2.x
Comment 4 Martin Hollmichel 2008-01-27 07:48:22 UTC
set target to 3.0
Comment 5 christian.jansen 2008-06-19 17:16:17 UTC
Moved to target OOo 3.x
Comment 6 malte_timmermann 2008-11-26 14:58:47 UTC
MT->CJ: Please define 
a) How to get a better perceptible focus
b) If or if not to hide the handles in the drawing page when that page doesn't
have the focus

OOo 3.2
Comment 7 christian.jansen 2009-07-20 10:09:24 UTC
Frank please take over. Thank you :-)
Comment 8 frank.loehmann 2009-07-20 13:51:15 UTC
Please take over. Thank you!
Comment 9 christoph.lukasiak 2009-08-10 17:14:16 UTC
a) enlarge the frame size of selected slide about one pixel if an object get
selected
b) object loose handles when a slide get selected
Comment 10 groucho266 2009-08-11 09:18:42 UTC
@clu: Please be more specific (maybe with the help of a spec?)

regarding a): there seems to be an error here,  probably the selection of slides
in in the slide sorter bar should become less notable when an object in the edit
view is selected, not more.
Comment 11 christoph.lukasiak 2009-08-11 11:08:18 UTC
@af: regarding a): sorry, my foult :(

Comment 12 christoph.lukasiak 2009-08-12 14:57:54 UTC
a) enlarge the frame size of selected slide(s*) about two** pixel after they get
selected -> if f.e. an object (shape) get selected, the frame size switch back
to normal style
b) object loose handles when a slide get selected

* multiselection -> all selected slides get the larger frame
** after drawing it, two pixel more looks much more 'eye catching' than only one

p.s. hc -> larger frame also visuable in hc mode (no further solution needed)


attached picture: 

slide 1 is the new larger selection (two pixel thicker than current frame)

as comparison:
slide 2 is the normal (current) selection
slide 3 is the 'mouse over'(current) selection


@af: p.p.s. hope this helps .. else it must wait until i am back from vacation
Comment 13 christoph.lukasiak 2009-08-12 15:01:46 UTC
Created attachment 64094 [details]
attached picture
Comment 14 Frank Schönheit 2009-09-03 11:11:20 UTC
Did not follow this issue so far, but now came back to it after Andre asked me
to have a look into implementing it. Well, I have a few problems:

For one, in a current build (DEV300.m54), the border around the selected
slide(s) is already pretty thick, and looks much darker and thicker than the one
in the attached screen short. So, I am not sure whether this is my desktop theme
only, or if this has been fixed meanwhile? In the first case, we need a better
specification, since if the border is dependent on the theme, then "just make it
bigger" is not a good specification of what to do.

Second, the suggestion does not care for the very original problem: no visual
feedback whether the page pane is *focused*. That is, if I understand it right,
then it says we should just make the border around the selected slides thicker
(see above) and de-select all objects in the document window. However, this
still means that the user cannot distinguish a focused page pane from a
non-focused page pane. In particular, if no object is selected in the document
window, then the user doesn't know whether pressing the "Delete" key will do
nothing (when the focus is in the document) or remove the selected slide(s).

Third, I am not sure at all that un-selecting the objects in the document view,
when a slide gets selected, is a good idea. Not to mention "a slide gets
selected" is not a sufficient description (how, for instance, does focusing the
page pane via F6, or de-selecting a slide while the page pane still has the
focus, play together here?). If we start with that change here (I know I
suggested it myself above, but ...), where should we stop? Should the objects
also be de-selected when a slide is selected from the Tasks/Layouts pane? For
consistency reasons, they should.


Bottom line: I do not think the proposed changed addresses the original problem,
and I do not think that the proposal is mature enough, as it yields additional
(consistency) problems/questions. The second point would not necessarily hinder
me in implementing a change (who am I to doubt UX? :) ), the first, however,
actually does.
Comment 15 Frank Schönheit 2009-09-03 11:22:53 UTC
Playing more with it: The page pane actually *has* a focus rect, which is just
not shown when you click a slide with the mouse.

That is, try the following with a doc with 5 or so slides: Click into the page
pane, then repeatedly press TAB. You'll notice that a small dotted frame appears
around, which cycles through all available slides. Then, click into the
document, and the frame disappears.

So, there *is* a visual indication that the pane has the focus, it is just not
enabled when you focus the pane with the mouse. This should be fixed.
Also, the indication should be made somewhat more obvious, as it currently is
not discernible when painted around a selected slide.
Comment 16 groucho266 2009-09-03 12:21:45 UTC
The fact that the focus indicator (dotted frame) is only visible after moving
the focus via keyboard into the slide sorter bar is feature not a bug.  The
behavior once was different but was changed on express request by UX.
Comment 17 Frank Schönheit 2009-09-03 12:25:49 UTC
Now that's cool. A focus indicator which has been removed on UX request? Whereas
this issue here requests exactly this: a focus indicator?

/me is confused. Either we fix this issue here, which implies reverting the
reversal (no matter if we use the existing focus indicator or invent a new one).
Or this reversal was rightfully, then this issue here is not fixable.
Comment 18 Frank Schönheit 2009-09-03 20:54:49 UTC
Created attachment 64532 [details]
suggested patch
Comment 19 Frank Schönheit 2009-09-03 21:09:57 UTC
The attached patch implements the following changes:
- In FocusManager::FocusPage, a check is made whether the slide sort currently
  has the focus, and if so, ShowFocus is called (if the focus is not currently
  shown). This ensures that if the focus is in the slide sorter, and there is
  any focused page, this is also visualized.
- The SdPageObjectFocusPrimitive, responsible for rendering the focus
  primitive, is modified as follows:
  - Instead of using a 1-pixel stroke, a 3-pixel stroke is used
  - Instead of painting the stroked line black on white, it is painted
    - "menu highllight text color" on "menu highlight color" (i.e. your
      theme's colors for selected menu text is used) for selected slides
    - "window text color" on "window color" (also taken from the theme)
      for unselected slides
  This is an improvement insofar as previously, for selected slides, the focus
  stroke line was always rendered black on white, with the mentioned
  1-pixel stroke, which effectively made the focus rect invisible for a
  selected slide if the theme's "menu highlight color" (used to render the
  slide's selection frame) was sufficiently dark.
  Now, the focus rect for selected slides should always be discernible,
  otherwise chances are good that already your system's desktop theme is
  broken.

fs->af: your's to review ...
Comment 20 Frank Schönheit 2009-09-03 21:11:42 UTC
Created attachment 64533 [details]
suggested patch
Comment 21 Frank Schönheit 2009-09-03 21:12:26 UTC
oops, the first version of the patch still had a wrong color used (the focus
rect for unselected slides used the "menu selection color" instead of the
"window text color").
Comment 22 Frank Schönheit 2009-09-04 09:41:03 UTC
Created attachment 64540 [details]
yet another patch
Comment 23 Frank Schönheit 2009-09-04 09:43:28 UTC
fs->aw: The latest patch also fixes the problem that the focus rectangle was not
shown when you focused the slide sorter by means other than clicking a slide
with the mouse.
Comment 24 Frank Schönheit 2009-09-08 11:56:00 UTC
since clu asked to put a short and concise summary of the changes herein:

With the patch, the situation is as follows: If (and only if) the slide sorter
has the focus, then (exactly) one slide in the slide sorter is surrounded by a
stroke line. This stroke line (no matter whether on a selected (framed) slide or
not) is always visible, since it uses the desktop theme's colors.
Comment 25 groucho266 2009-09-14 15:18:54 UTC
Have applied the patch.
Comment 26 groucho266 2009-09-16 17:43:32 UTC
@es: Please verify.
Comment 27 eric.savary 2009-09-17 16:21:18 UTC
Verified in CWS impressaccessibility3
Comment 28 eric.savary 2009-09-17 16:39:44 UTC
.
Comment 29 malte_timmermann 2010-01-08 09:16:37 UTC
Fixed and integrated => closing now..