Ticket #2226 (closed bug: obsolete)

Opened 2 years ago

Last modified 2 months ago

style-changes-undo -- Undo does not discard text style changes

Reported by: deyan Owned by:
Priority: major Milestone: X3
Component: uncategorized Version: 2.0
Keywords: Cc:
Ticket_group: Effort:
Category: unknown Importance: 50
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: deyan Design_owners: diana
Imp._owners: diana Test_owners:
Design_reviewers: pap Analysis_reviewers: diana
Changelog: Imp._reviewers: pap, todor
Test_reviewers: Analysis_score: 4.5
Design_score: 3 Imp._score: 3
Test_score: 0

Description

Enter some text, change font size; Try to undo, nothing happens.

Attachments

2226.patch Download (11.4 KB) - added by diana 2 years ago.

Change History

Changed 2 years ago by deyan

  • status changed from new to s1b_analysis_finished

Changed 2 years ago by deni

  • design_owners set to deni
  • status changed from s1b_analysis_finished to s2a_design_started
  • implementation_owners set to deni
  • analysis_reviewers set to deni

Changed 2 years ago by deni

  • design_owners deni deleted
  • status changed from s2a_design_started to new
  • implementation_owners deni deleted
  • analysis_reviewers deni deleted

Changed 2 years ago by diana

  • design_owners set to diana
  • status changed from new to s1b_analysis_finished
  • implementation_owners set to diana
  • analysis_score changed from 0 to 4.5
  • In TextView:EventIds:SET_TEXT add a new boolean parameter. If the parameter is true - the undo considers the fastness of typing (when undo all the changes applied in some time interval are undo), if false - the undo considers only the last change.
  • In HotTextLogic:ON_SET_TEXT if the boolean parameter of the event is lase, then the old text is set again to the model and is registered as a significant change and the applying of the new text is set as a significant change.

NOTE: This is VERY ugly...and I don't like the fix but cannot think of another...

Changed 2 years ago by diana

  • status changed from s1b_analysis_finished to s2a_design_started

Changed 2 years ago by diana

  • status changed from s2a_design_started to s3b_implementation_finished

Changed 2 years ago by diana

Changed 2 years ago by pap

  • status changed from s3b_implementation_finished to s3c_implementation_ok
  • analysis_reviewers set to diana
  • implementation_reviewers set to pap, todor
  • design_score changed from 0 to 3
  • design_reviewers set to pap
  • implementation_score changed from 0 to 3
  • Commited to trunk in [8605]
  • No need to set the current text again. Just put an empty AutoAction that is significant.
  • I find the naming a bit bad.
  • Copy/paste in the JavaDoc of the new parameter index constant is not fixed.
  • As I said you didn't use the new parameter corectly in places different than the style changes.
  • You left a commented out println :)
  • Your English is strangely strange in this ticket - both in the JavaDoc and in the explanation in the comments.
  • Write yourself as an analysis reviewer

Changed 2 months 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.