Issue 67368 - don't show red selection handles for not rotatable objects
Summary: don't show red selection handles for not rotatable objects
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: 2006-07-14 11:38 UTC by kla
Modified: 2013-02-24 21:20 UTC (History)
4 users (show)

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

Attachments
patch_080417 (634 bytes, text/plain)
2008-04-17 07:34 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 kla 2006-07-14 11:38:38 UTC 
Create a 3D Pie Chart
Choose Rotationmode (red Handles)
Toggle by using the Tab Key
Its not possible to leave this mode
Comment 1 kla 2006-07-14 11:39:49 UTC 
set keyword
Comment 2 IngridvdM 2006-07-14 12:05:08 UTC 
Accepted. For objects that can not be rotated the selection mode should change
back to Move/Resize (show green handles instead of red rotation handles) also if
the selecion changes via tab key.
Comment 3 IngridvdM 2007-09-12 09:21:49 UTC 
adapting target
Comment 4 weiz 2008-04-17 07:34:24 UTC 
Created attachment 52945 [details]
patch_080417
Comment 5 weiz 2008-04-17 07:36:56 UTC 
->iha: The patch for this issue is submitted. Please let me know your 
suggestions. Thanks a lot!
Comment 6 IngridvdM 2008-04-18 08:46:14 UTC 
->weiz: Thanks! The patch works fine.
I only exchanged the condition m_eDragMode != SDRDRAG_MOVE with m_eDragMode ==
SDRDRAG_ROTATE as this is the exact case, that needs treatment. At the moment it
makes no difference as only SDRDRAG_MOVE and SDRDRAG_ROTATE are used. But that
is not necessarily the case in future. So we are more safe with the change.
Checked in to CWS chart25.
Comment 7 IngridvdM 2008-05-15 10:06:16 UTC 
@Thomas, please verify in CWS chart25.
Comment 8 kla 2008-05-22 07:36:47 UTC 
Seen ok in CWS chart25 -> verified
Comment 9 kla 2008-08-29 11:20:48 UTC 
Seen ok in current master -> closed