Ticket #1703 (closed unplanned_task: obsolete)

Opened 10 years ago

Last modified 9 years ago

GROUP_CHAINING_AND_HALOS_R0

Reported by: deyan Owned by: nenko
Priority: major Milestone: M09_BETA1
Component: TEXT_FLOWING Version: 2.0
Keywords: Cc: nenko
Category: unknown Effort:
Importance: Ticket_group:
Estimated Number of Hours: Add Hours to Ticket:
Billable?: Total Hours:
Analysis_owners: nenko, deyan Design_owners: nenko
Imp._owners: nenko Test_owners:
Analysis_reviewers: dido Changelog:
Design_reviewers: meddle, meddle Imp._reviewers: meddle, pap
Test_reviewers: Analysis_score: 4
Design_score: 3.5 Imp._score: 3.5
Test_score: 0

Description (last modified by deyan) (diff)

Change History

comment:1 Changed 10 years ago by deyan

  • Milestone set to M09_BETA1

comment:2 Changed 10 years ago by deyan

  • Description modified (diff)

comment:3 Changed 10 years ago by deyan

  • Description modified (diff)

comment:4 Changed 10 years ago by deyan

  • Owner changed from nenko to deyan
  • Status changed from new to s1a_analysis_started

comment:5 Changed 10 years ago by nenko

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:6 Changed 10 years ago by dido

  • Status changed from s1b_analysis_finished to new
  • Analysis_reviewers set to dido
  • Analysis_score changed from 0 to 2.5
  • Тhe content should remains always in the top chain, when the chain is broke. No matter if it's evoked from hallo button or frame deletion.
  • The only exception may be if you delete the first frame from the chain - please discus this with Milo.
  • The halo buttons should also have tool tips with appropriate statement.(minor - could not be included in this revision)

Analysis review 2.5p (1h)

comment:7 Changed 10 years ago by deyan

  • Status changed from new to s1a_analysis_started

redid

comment:8 Changed 10 years ago by deyan

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:9 Changed 10 years ago by dido

  • Status changed from s1b_analysis_finished to s1c_analysis_ok
  • Analysis_score changed from 2.5 to 4

When the user delete the first frame of the chain the content is lost.
Analysis review 4p (10m)

comment:10 Changed 10 years ago by nenko

  • Owner changed from deyan to nenko
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:11 Changed 10 years ago by nenko

  • Design_owners set to nenko
  • Status changed from s2a_design_started to s2b_design_finished

comment:12 Changed 10 years ago by meddle

  • Status changed from s2b_design_finished to s1c_analysis_ok
  • Design_score changed from 0 to 2.5
  • Design_reviewers set to meddle

The problem with this design is described in TEXT_CHAINING_BEHAVIOUR_R0's implementation review so again but in other hand what do you think, is dido right about "When the user delete the first frame of the chain the content is lost." that thing and how can we fix it?

2.5p (2h)

comment:13 Changed 10 years ago by nenko

  • Status changed from s1c_analysis_ok to s2a_design_started

comment:14 Changed 10 years ago by nenko

  • Status changed from s2a_design_started to s2b_design_finished

comment:15 Changed 10 years ago by meddle

  • Status changed from s2b_design_finished to s2c_design_ok
  • Design_score changed from 2.5 to 3.5
  • Design_reviewers changed from meddle to meddle, meddle

The content gets larger and larger :)

3.5p (5m)

comment:16 Changed 10 years ago by nenko

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

comment:17 Changed 10 years ago by nenko

  • Status changed from s3a_implementation_started to s3b_implementation_finished

Yes, we should take actions... May be we can separate the chaining and styling support from basic text functions in next revision...

comment:18 Changed 10 years ago by meddle

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Imp._score changed from 0 to 3.5
  • Imp._reviewers set to meddle, pap

Code related notes:

  • Why are you using assertEquals(null, something), use assertNull(something), the same applies to assertEquals(true/false, something), use assertTrue and assertFalse.
  • Write all of your @parant descriptions.

Implementation related notes:

  • If you want your diagram paths to point to the trunk after the successfull review give them in chainsets as well.
  • There are known bugs :
    • With grouping the chaining breaks...
    • You can not select anything in other frames except the head.

3.5p (225m)

comment:19 Changed 9 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.