Ticket #2084 (closed bug: fixed)

Opened 10 years ago

Last modified 10 years ago

Missing indication of resize areas

Reported by: deni Owned by:
Priority: major Milestone: M12_RELEASE
Component: uncategorized Version: 2.0
Keywords: Cc:
Category: unknown Effort:
Importance: Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: deni Design_owners: deni
Imp._owners: deni Test_owners: vanya
Analysis_reviewers: Changelog:
Design_reviewers: Imp._reviewers: pap
Test_reviewers: Analysis_score: 0
Design_score: 0 Imp._score: 4
Test_score: 0

Description

When the user moves the mouse over a resize area (for a frame or sticky), it should change its color. In this way the user will know that he/she can drag the area to resize the element.

Change History

comment:1 Changed 10 years ago by deni

  • Status changed from new to s1b_analysis_finished

comment:2 Changed 10 years ago by deni

  • Status changed from s1b_analysis_finished to s2a_design_started

comment:3 Changed 10 years ago by deni

  • Status changed from s2a_design_started to s3b_implementation_finished
  • Modify SceneInteractionLogic to change the mouse over element of the scene appropriately
    • Currently, the mouseoverElement property of the scene is updated only if the event source of the new mouseover element is different than the event source of the old mouseover element.
    • This should be changed.
    • However, MOUSE_ENTERED and MOUSE_EXITED events should be fired only when the event source changes.
  • Instead of adding 8 different resize areas in each view of a resizable element, create a class that contains them all and add an object of this class (according to the implementation review of bug #1242).
    • Create an inner class of ResizeAreaSceneElement called ResizeSubAreaElement extending DefaultShapeSceneElement and move the position property, the contructor and the setup method that computes the area shape there.
    • Leave the inner and outer bound rectangles in ResizeAreaSceneElement, because they are common for all 8 subareas.
    • Make ResizeAreaSceneElement extend DefaultCompositeSceneElement.
    • Override setupSceneElements in ResizeAreaSceneElement to add the 8 different subareas.
    • Use a map to cache the subareas and avoid creating new ones every time (like in FrameView).
    • Make FrameView and StickyView consistent with this design.
  • ResizeSubAreaElement should take care to change its colour when the mouse is over it.
    • Add a new autoproperty in ResizeAreaSceneElement for the scene that contains the resize element. Its compute method should call an abstract method subclasses will implement.
    • In a setup method in ResizeSubAreaElement check whether the mouseover element of the scene is this, and if so, set the filling to green color. Otherwise, set the filling to transparent.

comment:4 Changed 10 years ago by pap

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Imp._score changed from 0 to 4
  • Imp._reviewers set to pap
  • Ok

comment:5 Changed 10 years ago by pap

comment:6 Changed 10 years ago by vanya

  • Status changed from s3c_implementation_ok to closed
  • Resolution set to fixed

comment:7 Changed 10 years ago by vanya

  • Test_owners set to vanya
Note: See TracTickets for help on using tickets.