Ticket #2173 (closed tweak: obsolete)

Opened 10 years ago

Last modified 8 years ago

nfr-better-text-rendering -- Character spacing between some characters.

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

Description (last modified by deyan) (diff)

text rendering needs to be much better

Take the following text:

The quick brown fox jumped over the lazy dog.
1234567890-+!@#$%^&*()_+|":{}<>?|

Use the attached font - regular.

  • Make the text so it looks as in the fonttest.png. (avoid zigzagging)

The font sizes are:
10; 18; 36

  • Make the rotated text so it looks like in the fonttestrotated.png

The font sizes are the same, the text is rotated at 45 deg.

Attachments

oldstandard-2.0.2.ttf.zip (505.2 KB) - added by deyan 10 years ago.
fonttest.png (939.5 KB) - added by deyan 10 years ago.
fonttestrotated.png (2.4 MB) - added by deyan 10 years ago.
beautiful_text.patch (8.3 KB) - added by tanya 10 years ago.

Change History

comment:1 Changed 10 years ago by deyan

  • Importance changed from 91 to 93
  • Summary changed from nfr-better-text-rendering to nfr-better-text-rendering – Character spacing between some characters.

Batch update from file report_1.csv

comment:2 Changed 10 years ago by deyan

  • Priority changed from major to critical
  • Summary changed from nfr-better-text-rendering – Character spacing between some characters. to nfr-better-text-rendering – Character spacing between some characters.

Batch update from file 0911261.csv

comment:3 Changed 10 years ago by deyan

  • Category set to unknown
  • Status changed from new to s1a_analysis_started
  • Description modified (diff)
  • Analysis_score set to 0
  • Test_score set to 0
  • Design_score set to 0
  • Owner set to deyan
  • Imp._score set to 0
  • Analysis_owners set to deyan

comment:4 Changed 10 years ago by deyan

  • Status changed from s1a_analysis_started to s1b_analysis_finished
  • Description modified (diff)
  • Summary changed from nfr-better-text-rendering – Character spacing between some characters. to nfr-better-text-rendering -- Character spacing between some characters.

Changed 10 years ago by deyan

Changed 10 years ago by deyan

comment:5 Changed 10 years ago by deyan

  • Description modified (diff)

comment:6 Changed 10 years ago by deyan

  • Description modified (diff)

Changed 10 years ago by deyan

comment:7 Changed 10 years ago by tanya

  • Status changed from s1b_analysis_finished to s1c_analysis_ok

comment:8 Changed 10 years ago by tanya

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

Changed 10 years ago by tanya

comment:9 Changed 10 years ago by tanya

  • Status changed from s2a_design_started to s2b_design_finished

TextLayout separated the text needed to be drawn to components. Using reflection, we get the components and draw their outline to the graphics. To get beautiful text, the outline of the components should be drawn.

comment:10 Changed 10 years ago by tanya

  • Design_owners set to tanya
  • Imp._owners set to tanya
  • Analysis_reviewers set to tanya

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

And that will not make sophie slower? If the speed is good, I guess it's ok, I would like some explanation like "I changed the some_method method to do some thing and added a new new_method to some_class". Because that is not a design even for a bug, though it passes now.

3p

comment:12 Changed 10 years ago by tanya

  • Status changed from s2c_design_ok to s3a_implementation_started

comment:13 Changed 10 years ago by tanya

  • Status changed from s3a_implementation_started to s3b_implementation_finished

Create new method drawLayout() in HotSegmentLayout. Do not use awtLayout.draw(graphics, 0f, 0f), but use drawLayout(graphics, 0f, 0f) instead.

comment:14 Changed 10 years ago by pap

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

Merged to the trunk at [8342].

I'm happy with not useing Ctrl+Shift+F from you and Deyan had some comments but talk with him if you want.

3.5p

comment:15 Changed 8 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.