Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | Invisible objects can be selected with keyboard navigation | ||
---|---|---|---|
Product: | General | Reporter: | kla <thomas.klarhoefer> |
Component: | chart | Assignee: | kla <thomas.klarhoefer> |
Status: | CLOSED FIXED | QA Contact: | issues@graphics <issues> |
Severity: | Trivial | ||
Priority: | P4 | CC: | issues, peter.junge, tony.galmiche.ooo, yonggang.mao |
Version: | 3.3.0 or older (OOo) | Keywords: | new_implementation |
Target Milestone: | --- | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Latest Confirmation in: | --- |
Developer Difficulty: | --- | ||
Issue Depends on: | |||
Issue Blocks: | 72764 | ||
Attachments: |
Description
kla
2006-07-11 12:57:53 UTC
set keyword The objects selected are the axes, the grids and the wall/floor (can be seen now in the status bar). Prio 4. changed summary 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. *** Issue 74522 has been marked as a duplicate of this issue. *** *** Issue 74524 has been marked as a duplicate of this issue. *** Removed "new chart" from summary, as this is superfluous now. Created attachment 47098 [details] The patch file is about issue67225# Created attachment 47525 [details] This new patch file is about issue67225# ->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. Created attachment 47590 [details] The patch file for issue63144-v4#,please check it. Created attachment 47609 [details]
Patch with fixed donut charts
Created attachment 47713 [details] The patch file for issue63144-v6#,please check it. maoyg: Thanks a lot for the patch. I applied it slightly modified to CWS chart11. ->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. adapting target Seen ok in CWS chart11 -> verified Seen ok in current master -> closed |