Ticket #1754 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

Tweak:caret not seen in the end of the text

Reported by: diana Owned by: vlado
Priority: minor Milestone: M09_BETA1
Component: uncategorized Version: 2.0
Keywords: caret Cc: diana
Ticket_group: Effort: 0.5
Category: unknown Importance:
Estimated Number of Hours: Add Hours to Ticket:
Billable?: Total Hours:
Analysis_owners: diana Design_owners: diana
Imp._owners: diana Test_owners: deyan
Design_reviewers: meddle, meddle Analysis_reviewers: diana, meddle
Changelog: Imp._reviewers: meddle, meddle, meddle
Test_reviewers: Analysis_score: 3
Design_score: 3 Imp._score: 3
Test_score: 3

Description (last modified by deyan) (diff)

BUG_1754? The caret is not visible when the end of the text is reached.

Change History

Changed 3 years ago by diana

  • owner set to diana
  • status changed from new to s1a_analysis_started

Changed 3 years ago by diana

  • status changed from s1a_analysis_started to s1b_analysis_finished

Changed 3 years ago by diana

  • status changed from s1b_analysis_finished to s1c_analysis_ok

Changed 3 years ago by diana

  • status changed from s1c_analysis_ok to s2a_design_started

Changed 3 years ago by diana

  • status changed from s2a_design_started to s2b_design_finished

Changed 3 years ago by diana

  • status changed from s2b_design_finished to s2c_design_ok

Changed 3 years ago by diana

  • status changed from s2c_design_ok to s3a_implementation_started

Changed 3 years ago by diana

  • status changed from s3a_implementation_started to s3b_implementation_finished

Changed 3 years ago by deyan

  • milestone set to M09_BETA1

ticket fix

Changed 3 years ago by meddle

  • cc diana added
  • status changed from s3b_implementation_finished to s2c_design_ok
  • design_reviewers set to meddle
  • implementation_reviewers set to meddle

Hmm there is no design and implementation in this ticket.

  • The design is your strategy to fix it, and is important because it's the documentation of the bug.
  • The implementation is the most important thing, the changeset/branch_url where the fix is placed.

At this state I can not merge/try/test/review this bug.

0p (10m)

Changed 3 years ago by deyan

  • description modified (diff)

Changed 3 years ago by deyan

  • description modified (diff)

Changed 3 years ago by diana

  • status changed from s2c_design_ok to s3a_implementation_started

Add a new boolean variable in the method draw in class Atom which shows if the caret is in the end of the text. If and only if the caret is in the end of the text the boolean variable end is set to true and the caret is drawn at the end of the text.

Source code can be found here : [3914]

Changed 3 years ago by diana

  • status changed from s3a_implementation_started to s3b_implementation_finished

Changed 3 years ago by meddle

  • status changed from s3b_implementation_finished to s2c_design_ok
  • analysis_reviewers changed from diana to diana, meddle
  • analysis_score changed from 0 to 2.5
  • implementation_reviewers changed from meddle to meddle, meddle
  • design_score changed from 0 to 2.5
  • design_reviewers changed from meddle to meddle, meddle
  • implementation_score changed from 0 to 2.5

Hmm the fix works fine unless you chain frames, then it breaks the chaining so at this state I can not pass this fix :(

2.5p (15m)

Changed 3 years ago by vlado

  • owner changed from diana to vlado
  • status changed from s2c_design_ok to s3a_implementation_started

Changed 3 years ago by vlado

  • status changed from s3a_implementation_started to s3b_implementation_finished

Changed 3 years ago by meddle

  • status changed from s3b_implementation_finished to s3c_implementation_ok
  • design_score changed from 2.5 to 3
  • implementation_reviewers changed from meddle, meddle to meddle, meddle, meddle
  • implementation_score changed from 2.5 to 3
  • analysis_score changed from 2.5 to 3

Merged to the trunk in [4069].

Works bad with chaining the caret is lost, but the bug was not for that so....

3p (10)

Changed 3 years ago by deyan

testing started

Changed 3 years ago by deyan

  • status changed from s3c_implementation_ok to closed
  • test_owners set to deyan
  • resolution set to fixed
  • test_score changed from 0 to 3

fixed

Note: See TracTickets for help on using tickets.