Ticket #801 (closed planned_task)

Opened 11 years ago

Last modified 10 years ago

TEXT_AUTO_CHAIN_BEHAVIOUR_R1

Reported by: Astea Owned by: boyan
Priority: 3 Milestone: M12_RELEASE
Component: TEXT_FLOWING Version: 2.0
Keywords: Cc:
Category: MAIN Effort: 1
Importance: 0 Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: deyan, milen Design_owners: boyan
Imp._owners: boyan Test_owners:
Analysis_reviewers: dido Changelog:
Design_reviewers: meddle Imp._reviewers: meddle, todor, meddle, todor
Test_reviewers: Analysis_score: 3.5
Design_score: 3 Imp._score: 3.5
Test_score: 0

Description

wiki page: TEXT_AUTO_CHAIN_BEHAVIOUR_R1 - effort: 1d

Change History

comment:1 Changed 10 years ago by deyan

  • Status changed from new to s1a_analysis_started
  • Total Hours set to 0
  • Analysis_score set to 0
  • Test_score set to 0
  • Add Hours to Ticket set to 0
  • Design_score set to 0
  • Billable? set
  • Owner changed from Astea to deyan
  • Estimated Number of Hours set to 0
  • Imp._score set to 0
  • Analysis_owners set to deyan

comment:2 Changed 10 years ago by milen

  • Analysis_owners changed from deyan to deyan, milen
  • Milestone changed from M07_ALPHA2 to M12_RELEASE

comment:3 Changed 10 years ago by milen

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:4 follow-up: ↓ 5 Changed 10 years ago by deyan

  • Status changed from s1b_analysis_finished to new
  • Analysis_score changed from 0 to 2

The aim of the task is not clear in the analysis section. Why do we need update of platform standards manual tests? Also, a lot of the analysis is a copy of the previous one.

comment:5 in reply to: ↑ 4 Changed 10 years ago by deyan

  • Status changed from new to s1a_analysis_started

Replying to deyan:

The aim of the task is not clear in the analysis section. Why do we need update of platform standards manual tests? Also, a lot of the analysis is a copy of the previous one.

mistake, not this ticket

comment:6 Changed 10 years ago by deyan

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:7 Changed 10 years ago by dido

  • Status changed from s1b_analysis_finished to s1c_analysis_ok
  • Analysis_reviewers set to dido
  • Analysis_score changed from 2 to 3.5
  • If there is 'Auto Chain' label inside the combo box, please make another one called 'None' and set it to default. Remove the empty item.
  • If it's possible develop the following behavior
    • When 'Auto Chain' is selected and the text overflows, after creation of new frame - the combo box content should be set to the name of the next frame, not to auto chain

Analysis Reviewed 3.5p (30m)

comment:8 Changed 10 years ago by boyan

  • Design_owners set to boyan
  • Owner changed from deyan to boyan
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:9 Changed 10 years ago by boyan

  • Status changed from s2a_design_started to s2b_design_finished

finished design

comment:10 Changed 10 years ago by boyan

finished implementation as well, now merging with trunk (after natives)

comment:11 Changed 10 years ago by meddle

  • Status changed from s2b_design_finished to s2c_design_ok
  • Design_score changed from 0 to 3
  • Design_reviewers set to meddle

Quite simple design... I guess

  • The hud in the design section differs with that from the analysis, why is that?
  • The resource name prefixes will be more stable without spaces after them (I'm talking about the page in your code).
  • I personally don't like ctrl+shift+F formatting...
  • You will fix TextSelectionTest, or? I'll be happy if you do that...
  • Do you have problems with the templating?
  • I should explain how will work deleting of a page, you will delete the frames in it, but in what order?
  • How will work auto-chaining work with groups?

Please explain these things for the implementation...

3p (30m)

comment:12 Changed 10 years ago by boyan

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

Notes on the comments of the design review:

  • The HUD in the analysis section is designed for full auto-chaining behavior: when you select auto-chaining mode, text flow is monitored in real time and frames are added/deleted correspondingly. After discussion with milo, we decided that at this point a button that recomputes the flow should be available and no auto-tracking will be present. That's why the HUD has a button instead of an item in the drop-down.
  • I thought that a space after the resource name prefixes makes them a bit more readable. I will get rid of them at all places.
  • I won't use Ctrl+Shift+F anymore :)
  • TextSelectionTest will not be fixed at this iteration. It is improved though but fails because of a missing icon.
  • Templating behaves strangely and unstable - try inserting some frames on a page and click twice on the Use Template halo button :) That's a great way of deleting frames...
  • The bug with page deleting is now fixed, all frames are deleted as well but there is no time to implement underflowing now.
  • Currently grouping will not affect auto-chaining in any way - no matter whether a frame is a group or not, default frames are inserted on next pages. This will be fixed soon together with the other bug related to chaining and grouping - 2074).

Now a lot of features are not implemented but we need to get something in trunk because the newly created branch will get outdated again and merging will be problematic.

comment:13 Changed 10 years ago by boyan

  • Status changed from s3a_implementation_started to s3b_implementation_finished

implementation finished

comment:14 Changed 10 years ago by meddle

  • Status changed from s3b_implementation_finished to s2c_design_ok
  • Imp._score changed from 0 to 2
  • Imp._reviewers set to meddle, todor
  • Desktop book doesn't behave properly (when autochain is used there new pages are created)
  • Deleting first frame results in exception
  • When deleting frame of the autochain, the chain is broken the next of the text is missing but the newly created pages after that doesn't disappear.
  • The relative ref in the HeadTextFrameH helper result is not right...
  • The design notes are important.
  • Your test does not pass.
  • Deleting the first page when you have many pages has the areas exceptions...
  • Try to fix your known issues for Monday, the next release.

2p (80m)

comment:15 Changed 10 years ago by boyan

  • Status changed from s2c_design_ok to s3a_implementation_started

restarting implementation

comment:16 Changed 10 years ago by boyan

  • Status changed from s3a_implementation_started to s3b_implementation_finished

finished

comment:17 Changed 10 years ago by meddle

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Imp._score changed from 2 to 3.5
  • Imp._reviewers changed from meddle, todor to meddle, todor, meddle, todor
  • I don't know if using ResourceRef#append instead ResourceRef#getRelative is bad. May be it is...
  • Some of the bugs like the html export are not fixed.
  • Fix the chaining when working with server.

I like the implementation though...

3.5p (2h)

comment:18 Changed 10 years ago by deyan

  • Status changed from s3c_implementation_ok to closed

Batch update from file query-10.csv

Note: See TracTickets for help on using tickets.