Issue 67225 - Invisible objects can be selected with keyboard navigation
Summary: Invisible objects can be selected with keyboard navigation
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P4 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
URL:
Keywords: new_implementation
: 74522 74524 (view as issue list)
Depends on:
Blocks: 72764
  Show dependency tree
 
Reported: 2006-07-11 12:57 UTC by kla
Modified: 2013-02-24 21:20 UTC (History)
4 users (show)

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

Attachments
The patch file is about issue67225# (4.94 KB, text/plain)
2007-07-26 09:12 UTC, yonggang.mao 		no flags Details
This new patch file is about issue67225# (4.04 KB, text/plain)
2007-08-14 11:25 UTC, yonggang.mao 		no flags Details
The patch file for issue63144-v4#,please check it. (8.08 KB, text/plain)
2007-08-17 11:52 UTC, yonggang.mao 		no flags Details
Patch with fixed donut charts (8.75 KB, text/plain)
2007-08-17 16:15 UTC, bjoern.milcke 		no flags Details
The patch file for issue63144-v6#,please check it. (3.90 KB, text/plain)
2007-08-22 08:33 UTC, yonggang.mao 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description kla 2006-07-11 12:57:53 UTC 
Build an 3D-Pie chart 
There must be more then one Data Row.
Select the Chart
Toggle by using the Tab Key
Invisible Dataseries selected
Comment 1 kla 2006-07-11 12:58:34 UTC 
set keyword
Comment 2 bjoern.milcke 2006-09-28 10:42:26 UTC 
The objects selected are the axes, the grids and the wall/floor (can be seen now
in the status bar).

Prio 4.
Comment 3 IngridvdM 2006-12-01 21:02:03 UTC 
changed summary
Comment 4 bjoern.milcke 2007-01-17 10:57:06 UTC 
Changed title to make clear that the problem refers to more than just data
series, and also for other chart types.

The problem is that keyboard navigation selects the next model object in the
hierarchy without checking whether the object has a visual representation.
Comment 5 IngridvdM 2007-02-14 10:41:04 UTC 
*** Issue 74522 has been marked as a duplicate of this issue. ***
Comment 6 IngridvdM 2007-02-14 10:45:15 UTC 
*** Issue 74524 has been marked as a duplicate of this issue. ***
Comment 7 bjoern.milcke 2007-07-17 09:22:14 UTC 
Removed "new chart" from summary, as this is superfluous now.
Comment 8 yonggang.mao 2007-07-26 09:12:03 UTC 
Created attachment 47098 [details]
The patch file is about issue67225#
Comment 9 yonggang.mao 2007-08-14 11:25:56 UTC 
Created attachment 47525 [details]
This new patch file is about issue67225#
Comment 10 bjoern.milcke 2007-08-14 15:28:28 UTC 
->maoyg: Thanks for the latest patch, it looks very good! One small thing I
noticed: you define bIsThreeD, but nevver use it. It can be used for the
"floor-check" below.

And I noticed one other thing: when you create a pie chart (not donut, only a
pure pie) out of more than one column or row, you get more than one data series
internally. But, you should only be able to select the first series. I think it
would make sense to add a helper function in ChartTypeHelper to get the number
of displayed data series, which for a pie always returns 1, and for all other
charttypes returns the actual number of data series contained in the chart.
Comment 11 yonggang.mao 2007-08-17 11:52:03 UTC 
Created attachment 47590 [details]
The patch file for issue63144-v4#,please check it.
Comment 12 bjoern.milcke 2007-08-17 16:15:41 UTC 
Created attachment 47609 [details]
Patch with fixed donut charts
Comment 13 yonggang.mao 2007-08-22 08:33:02 UTC 
Created attachment 47713 [details]
The patch file for issue63144-v6#,please check it.
Comment 14 bjoern.milcke 2007-08-22 10:12:48 UTC 
maoyg: Thanks a lot for the patch. I applied it slightly modified to CWS chart11.
Comment 15 bjoern.milcke 2007-09-05 15:30:44 UTC 
->kla: Please verify in CWS chart11.

See Issue 80933 for a missing selection of grids. This one is different, as this
is a selectable object that is selected correctly, only the handles are missing.
Comment 16 bjoern.milcke 2007-09-05 19:20:50 UTC 
adapting target
Comment 17 kla 2007-09-07 10:09:36 UTC 
Seen ok in CWS chart11 -> verified
Comment 18 kla 2007-11-30 13:13:16 UTC 
Seen ok in current master -> closed