Issue 67368 - don't show red selection handles for not rotatable objects
Summary: don't show red selection handles for not rotatable objects
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
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: ---

patch_080417 (634 bytes, text/plain)
2008-04-17 07:34 UTC, weiz
no flags Details

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]
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