Ticket #1814 (closed unplanned_task: obsolete)

Opened 16 years ago

Last modified 15 years ago

HALOS_REDESIGN_R0

Reported by: tanya Owned by: mitex
Priority: major Milestone:
Component: uncategorized Version: 2.0
Keywords: Cc: mitex
Category: BASE Effort:
Importance: Ticket_group:
Estimated Number of Hours: Add Hours to Ticket:
Billable?: Total Hours:
Analysis_owners: tanya Design_owners: mitex
Imp._owners: mitex Test_owners:
Analysis_reviewers: dido Changelog:
Design_reviewers: meddle Imp._reviewers: pap
Test_reviewers: Analysis_score: 3.5
Design_score: 3.5 Imp._score: 3
Test_score: 0

Description

This task includes fixing problems with halos and huds.
HALOS_REDESIGN_R0

Change History

comment:1 Changed 16 years ago by tanya

  • Owner set to tanya
  • Status changed from new to s1a_analysis_started

comment:2 Changed 16 years ago by tanya

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:3 Changed 16 years ago by tanya

  • Analysis_owners set to tanya

comment:4 Changed 16 years ago by dido

  • Cc tanya, dido, peko, deyan, mitex added
  • Status changed from s1b_analysis_finished to s1c_analysis_ok
  • Analysis_score changed from 0 to 3.5

Here is a list of things that you have to keep in mind while implementing this.

  • First of all if the hud appears on roll over some halo button, the area could become sensitive and a lot of undesired huds could start popping up. This could be partially avoided by adding some delay. My personal opinion is that this should be first developed as prototype and then take decision is this going to be the final behavior. Same applies for pinning, positioning and "only on visible hud". It may happen that the behavior we have at this point is more appropriate. Please focus on other topics first.
  • Those two Change the hud halo button icon depending on hud visibility. and Change the hud halo button icon if mouse is pressed, rolled, if the halo is disabled. (selected or normal mode) looks like the same for me.
  • Halos and huds become part of the scene - for this one please make sure that doesn't effect the performance of the application.
  • Related section is not finished please review and add following tasks
  • Peko and me are working on adding new skins for the entire application, please communicate with us and Deyan.

Analysis Reviewed 3.5p (1h)

comment:5 Changed 16 years ago by dido

  • Analysis_reviewers set to dido

comment:6 Changed 16 years ago by mitex

  • Design_owners set to mitex
  • Owner changed from tanya to mitex
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:7 Changed 16 years ago by mitex

  • Status changed from s2a_design_started to s2b_design_finished

comment:8 Changed 16 years ago by meddle

  • Status changed from s2b_design_finished to s2c_design_ok
  • Design_reviewers set to meddle

I think the ideas are good, and the design is detailed, but:

  • These tests the old ones MUST run without fails on the implementation review.
  • You didn't write additional tests. For the implementation review I want running tests that test a halo with a sub-halo, the position of the sub-halo to the icon too.
  • Did you read the comments of dido, and most importantly that about the delay. Please discuss that things with me, peko and dido!

3.5p (35m)

comment:9 Changed 16 years ago by mitex

  • Design_score changed from 0 to 3.5

setting the design score, because it was missed

comment:10 Changed 16 years ago by mitex

  • Status changed from s2c_design_ok to s3a_implementation_started
  • Imp._owners set to mitex

comment:11 Changed 16 years ago by mitex

  • Status changed from s3a_implementation_started to s3b_implementation_finished

comment:12 Changed 16 years ago by pap

  • Cc tanya, dido, peko, deyan removed
  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Imp._score changed from 0 to 3
  • Imp._reviewers set to pap
  • I pass this implementation although it has some nice problems
  • First there isn't any demonstration of an extension hud - this is not nice.
  • I don't like the idea that is used for hud extensions.
  • However the general look and behavior of extensions may be changed.
  • There is something wierd about having the location property in AbstractHalo
  • I don't like that you've screwed up the LinksHud
  • Also I think that the state of the HudHaloButton should be changed on clicking it.
  • Otherwise the implementation is done well. You are improving the style of your code

comment:13 Changed 15 years ago by deyan

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

Batch update from file query-obsoleted.csv

Note: See TracTickets for help on using tickets.