Ticket #240 (s2c_design_ok planned_task) — at Version 19

Opened 17 years ago

Last modified 15 years ago

BASE_LAYOUT_R0

Reported by: Astea Owned by: Tanya
Priority: 3 Milestone: M02b_PRE2b
Component: BASE_GUI_LIB Version: 2.0
Keywords: Cc: peko
Category: unknown Effort: 1.5
Importance: 20 Ticket_group:
Estimated Number of Hours: Add Hours to Ticket:
Billable?: Total Hours:
Analysis_owners: Design_owners: tanya, mira
Imp._owners: tanya Test_owners:
Analysis_reviewers: tanya Changelog:
Design_reviewers: Alex Imp._reviewers:
Test_reviewers: Analysis_score: 3.5
Design_score: 4 Imp._score: 0
Test_score: 0

Description (last modified by tanya) (diff)

wiki page: BASE_LAYOUT_R0 - effort: 1.5d

Change History

comment:1 Changed 17 years ago by todor

  • Owner changed from Astea to todor
  • Status changed from new to analyzing

comment:2 Changed 17 years ago by todor

  • Status changed from analyzing to analyzed

comment:3 Changed 17 years ago by todor

  • Status changed from analyzed to analysis_accepted

comment:4 Changed 17 years ago by Tanya

  • Owner changed from todor to Tanya
  • Status changed from analysis_accepted to designing

comment:5 Changed 17 years ago by Tanya

  • Status changed from designing to s1c_analysis_ok

comment:6 Changed 17 years ago by Tanya

  • Status changed from s1c_analysis_ok to s2a_designing

comment:7 Changed 17 years ago by Tanya

  • Status changed from s2a_designing to s2b_designed

comment:8 Changed 17 years ago by milo

  • Status changed from s2b_designed to s2b_design_finished

comment:9 Changed 17 years ago by pavlina

  • Status changed from s2b_design_finished to s2c_design_ok

3.5p. It is understandable and clear.

comment:10 Changed 17 years ago by milo

  • Status changed from s2c_design_ok to s1c_analysis_ok

2p - the approach is good, but this library needs better design

Hmm... i dont think that the design is ok because of the following:

  • There are no demonstrating unit tests (or at least not listed).
  • Properties are not written in correct case.
  • The listed classes need more descriptions - at least what their purpose is.
  • There is actually an implementation of this one, but it should be better designed.
  • In general, it is not clear enough how this library will be used, or how layout engines should be implemented.


It may not seem nice that I'm failing this one, but quality is not negotiable. Especially for things in BASE (their design affects more things).

comment:11 Changed 17 years ago by deyan

  • Milestone changed from M02_PRE2 to M02b_PRE2b

Milestone change

comment:12 Changed 16 years ago by Tanya

  • Design_owners set to Tanya
  • Analysis_reviewers set to Tanya
  • Analysis_score set to 0
  • Test_score set to 0
  • Design_score set to 0
  • Imp._score set to 0

comment:13 Changed 16 years ago by Tanya

  • Status changed from s1c_analysis_ok to s2a_design_started

comment:14 Changed 16 years ago by Tanya

  • Design_owners changed from Tanya to tanya
  • Analysis_reviewers changed from Tanya to tanya

comment:15 Changed 16 years ago by Tanya

  • Design_owners changed from tanya to tanya, mira

comment:16 Changed 16 years ago by Tanya

  • Status changed from s2a_design_started to s2b_design_finished

comment:17 Changed 16 years ago by tanya

  • Imp._owners set to tanya

comment:18 Changed 16 years ago by alex

  • Category set to unknown
  • Design_score changed from 0 to 4
  • Design_reviewers set to Alex
  • Status changed from s2b_design_finished to s2c_design_ok
  • Description modified (diff)

comment:19 Changed 16 years ago by tanya

  • Description modified (diff)

I think the design is good, I don't see any problems with it

Note: See TracTickets for help on using tickets.