Ticket #2332 (closed planned_task: obsolete)
TEXT_POSITION_KIND
| Reported by: | diana | Owned by: | diana |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | uncategorized | Version: | 2.0 |
| Keywords: | text | Cc: | |
| Category: | unknown | Effort: | |
| Importance: | Ticket_group: | ||
| Estimated Number of Hours: | 0 | Add Hours to Ticket: | 0 |
| Billable?: | yes | Total Hours: | 0 |
| Analysis_owners: | diana | Design_owners: | diana |
| Imp._owners: | diana | Test_owners: | |
| Analysis_reviewers: | meddle | Changelog: | Changelog |
| Design_reviewers: | meddle, kyli | Imp._reviewers: | kyli |
| Test_reviewers: | Analysis_score: | 3 | |
| Design_score: | 3.5 | Imp._score: | 3.5 |
| Test_score: | 0 |
Description
Change History
comment:1 Changed 15 years ago by diana
- Owner set to diana
- Status changed from new to s1a_analysis_started
comment:2 Changed 15 years ago by diana
- Status changed from s1a_analysis_started to s1b_analysis_finished
comment:3 Changed 15 years ago by meddle
- Status changed from s1b_analysis_finished to s1c_analysis_ok
- Analysis_reviewers set to meddle
- Analysis_score changed from 0 to 3
I think as an analysis this is bad because there is technical stuff like classes... And the requirement number one is the same like the fires implementation idea... But I think it can pass, but next time I will consult with Deyan, so you better ask him how to write your analysis .
3p
comment:5 Changed 15 years ago by diana
- Status changed from s2a_design_started to s2b_design_finished
comment:6 Changed 15 years ago by meddle
- Status changed from s2b_design_finished to s2c_design_ok
- Design_score changed from 0 to 3.5
- Design_reviewers set to meddle, kyli
- Use some office free machines or Straxil when coding, because you create many conflicts with your buggy Vista.
- I like the design because it removes a lot of unnecessary code at last.
- You can try to find unused methods connected to the PosKinds in some util classes and delete them.
- Learn how to use the test to write stable logic.
3.5p
comment:7 Changed 15 years ago by diana
- Status changed from s2c_design_ok to s3a_implementation_started
comment:8 Changed 15 years ago by diana
- Status changed from s3a_implementation_started to s3b_implementation_finished
comment:9 Changed 15 years ago by kyli
- Status changed from s3b_implementation_finished to s2c_design_ok
- Imp._reviewers set to kyli
- Changelog set to [wiki:Changelog]
- You have warnings in your branch. Please, remove them.
- When you change a text, navigating through it causes assertion errors. For more information, you could refer to deyan (or me).
- I would really like to see some more tests about this feature (not only swing demos). Time has proved that small mistakes in HotPos could cause serious problems in the view and a good test could save a lot time, compared to debugging.
2p.
comment:10 Changed 15 years ago by diana
- Status changed from s2c_design_ok to s3a_implementation_started
comment:11 Changed 15 years ago by diana
- Status changed from s3a_implementation_started to s3b_implementation_finished
comment:12 Changed 15 years ago by kyli
- Status changed from s3b_implementation_finished to s3c_implementation_ok
- Imp._score changed from 0 to 3.5
- Again, it is good that you removed quite a lot of unnecessary code.
- We had some problems reviewing the task, because several times new problems appeared right after merging. I hope that all the problems are fixed now, but I doubt it..
- Good luck with the next task :)
3.5p, merged with text repo at [8721].
comment:13 Changed 13 years ago by meddle
- Status changed from s3c_implementation_ok to closed
- Resolution set to obsolete
Closing all the tickets before M Y1
Note: See
TracTickets for help on using
tickets.
