Ticket #390 (closed planned_task: obsolete)

Opened 14 years ago

Last modified 13 years ago

APP_LAYOUT_MYDOGGY_R1

Reported by: Astea Owned by: peko
Priority: blocker Milestone: M06_ALPHA1
Component: APP_PLUGIN Version: 2.0
Keywords: Cc: peko
Category: MAIN Effort: 2
Importance: 20 Ticket_group:
Estimated Number of Hours: Add Hours to Ticket:
Billable?: Total Hours:
Analysis_owners: peko Design_owners: peko
Imp._owners: peko Test_owners:
Analysis_reviewers: deyan Changelog:
Design_reviewers: pap Imp._reviewers: pap
Test_reviewers: Analysis_score: 3.5
Design_score: 3 Imp._score: 3.5
Test_score: 0

Description

wiki page: APP_LAYOUT_MYDOGGY_R1 - effort: 2d

Change History

comment:1 Changed 13 years ago by peko

  • Status changed from new to s1a_analysis_started
  • Analysis_score set to 0
  • Test_score set to 0
  • Design_score set to 0
  • Owner changed from Astea to peko
  • Imp._score set to 0
  • Analysis_owners set to peko

comment:2 Changed 13 years ago by peko

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:3 Changed 13 years ago by peko

  • Ticket_group set to GROUP_GUI_REDESIGN_R0

comment:4 Changed 13 years ago by deyan

  • Status changed from s1b_analysis_finished to s1c_analysis_ok
  • Analysis_reviewers set to deyan
  • Analysis_score changed from 0 to 3.5

comment:5 Changed 13 years ago by peko

  • Design_owners set to peko
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:6 Changed 13 years ago by peko

  • Ticket_group GROUP_GUI_REDESIGN_R0 deleted

comment:7 Changed 13 years ago by peko

  • Status changed from s2a_design_started to s2b_design_finished

comment:8 Changed 13 years ago by peko

  • Priority changed from 3 to blocker

comment:9 Changed 13 years ago by pap

  • Status changed from s2b_design_finished to s2c_design_ok
  • Design_score changed from 0 to 3
  • Design_reviewers set to pap
  • There are some quite unclear parts of the desing(mainly the applet), but let it be
  • In SophieDockedContainer class you have a @SuppressWarnings("static-access") - this is not acceptable
  • The names fixes of the package is not good.
  • I consider this task an improvement but still we need more things to correct there
  • Class names MDWindow and MDFrameWindow are not nice - these are main windows. In a moment I thought that MDDocumentWindow is a kind of MDWindow - so the names are mesleading
  • Somewhere when talking about DocumentWindows I read the word book. We should get rid of the idea that documents can be only books.
  • I have the feeling that the MD engine is still full of bad code so please try to correct at least part of it while implementing.
  • Good luck

comment:10 Changed 13 years ago by peko

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

Thanks about the comments.

  • I personally do not know about another way of integrating mydoggy and the layout model itself without having a MDAppletWindow. Later it may appear that this is going to be done another way. For the time being this is the only solutions provided and thought of.
  • The @SuppressWarnings("static-access") are removed from the SophieDockedContainer, but still this is a design phase and suck things had been thought of for in the implementation phase.
  • "fixes" is renamed to "mdcustomization" as discussed, although this is not a customization but a bug fix.
  • MDWindow is now renamed to MDMainWindow. Absolutely agree that that way misleading.
  • Renamed to opened documents.
  • I do not see any bad code there, since there are only two methods there. The actual showing of the MDMainWindow (whether with an applet or a frame) should of course be refactored, but for the time being we have no clear idea about this, so I am going to leave it that way till we discuss this with other people.

comment:11 Changed 13 years ago by peko

  • Status changed from s3a_implementation_started to s3b_implementation_finished

comment:12 Changed 13 years ago by pap

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Imp._score changed from 0 to 3.5
  • Imp._reviewers set to pap
  • I find this task relatively good implemented after some remarks. :)
  • Perhaps some ideas my be rethought - about anchoring and MDLayoutElements reusing
  • Good luck :)

comment:13 Changed 13 years ago by pap

  • Cc peko added

adding cc

comment:14 Changed 13 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.