Issue 80933 - Selection of Grid with keyboard is not visible
Summary: Selection of Grid with keyboard is not visible
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: regression
Depends on:
Blocks: 72764
  Show dependency tree
 
Reported: 2007-08-22 11:23 UTC by bjoern.milcke
Modified: 2013-02-24 21:21 UTC (History)
3 users (show)

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

Attachments
patch_080506 (650 bytes, text/plain)
2008-05-06 10:08 UTC, weiz 		no flags Details
patch_080506 (650 bytes, text/plain)
2008-05-06 10:09 UTC, weiz 		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 bjoern.milcke 2007-08-22 11:23:00 UTC 
Selecting a major grid with the keyboard (Tab), you don't see the selection
handles on the grid.

The reason for this is that the selection code uses
ObjectIdentifier::createClassifiedIdentifierForGrid (with defaulted index -1)
which returns
"CID/D=0;CS=0;Axis=1,0;Grid=0"

VCoordinateSystem::createCIDForGrid returns
"CID/D=0;CS=0;Axis=1,0;Grid="
which is set at the shape. Due to this mismatch the selection is not visible.

->IHA: Please check which place to change. I think fixing the code in
VCoordinateSystem to add a "0" would be correct.
Comment 1 IngridvdM 2007-09-11 10:36:51 UTC 
accepted
Comment 2 IngridvdM 2008-01-16 13:39:02 UTC 
changed target due to limited resources
Comment 3 weiz 2008-05-06 10:08:47 UTC 
Created attachment 53408 [details]
patch_080506
Comment 4 weiz 2008-05-06 10:09:40 UTC 
Created attachment 53409 [details]
patch_080506
Comment 5 weiz 2008-05-06 10:13:35 UTC 
->iha: the patch is submitted. Please find it at the attachment. Thank you!
Comment 6 IngridvdM 2008-05-06 11:49:21 UTC 
->weiz, thanks for the patch it works fine! I checked it in to CWS chart25.
Comment 7 IngridvdM 2008-05-15 10:13:39 UTC 
@Thomas, please verify in CWS chart25.
Comment 8 kla 2008-05-20 07:54:21 UTC 
Seen ok in CWS chart25 -> verified
Comment 9 kla 2008-06-11 11:36:22 UTC 
Seen ok in current master -> closed