bugGnash - The GNU Flash player - Bugs: bug #22463, text rendered at wrong size

 
 

bug #22463: text rendered at wrong size

Submitted by:  Sandro Santilli <strk>
Submitted on:  Mon 03 Mar 2008 12:21:54 PM UTC  
 
Category: NoneSeverity: 3 - Normal
Release: NoneStatus: Fixed
Privacy: PublicAssigned to: Benjamin Wolsey <bwy>
Open/Closed: Closed

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Wed 08 Jul 2009 04:11:08 PM UTC, comment #10:

I confirm size is good now (and width isn't)

Sandro Santilli <strk>
Project Member
Wed 08 Jul 2009 03:56:00 PM UTC, comment #9:

The size should be correct now. Device fonts are scaled in both directions using the y scale value.

The width of the textfield should be scaled according to the x scale, but that's not yet fixed (needs some changes to TextField.cpp).

The x position of some of the text is still slightly wrong in the two examples in comment #1, but I haven't worked out why. The test misc-ming.all/DeviceFontTest doesn't reproduce this problem.

Benjamin Wolsey <bwy>
Project MemberIn charge of this item.
Sun 09 Nov 2008 02:50:51 PM UTC, comment #8:

One problem is that we scale the font according using the TextField character's matrix. We shouldn't do this, at least for device fonts.

Benjamin Wolsey <bwy>
Project MemberIn charge of this item.
 Spam posted by an anonymous
Fri 14 Mar 2008 12:33:35 PM UTC, comment #7:

In gnash, device fonts create glyphs on demand.

Sandro Santilli <strk>
Project Member
Tue 11 Mar 2008 01:00:56 PM UTC, comment #6:

What am I doing? It does use those two device fonts; but the Arial font (ID: 8) has 49 glyphs, each with an associated shape. Is that normal for a device font?

Benjamin Wolsey <bwy>
Project MemberIn charge of this item.
Tue 11 Mar 2008 12:49:56 PM UTC, comment #5:

That should read "doesn't use device fonts".

Benjamin Wolsey <bwy>
Project MemberIn charge of this item.
Tue 11 Mar 2008 12:35:50 PM UTC, comment #4:

It does use device fonts: Arial and Symbol (this latter presumably just for the theta sign).

From debug log:

17810] 13:32:41: DEBUG: Advance value for glyph '4' is 284.5 (horiAdvance:569, scale:0.5)
17810] 13:33:58: DEBUG: Advance value for glyph '5' is 569.5 (horiAdvance:1139, scale:0.5)

Benjamin Wolsey <bwy>
Project MemberIn charge of this item.
Thu 06 Mar 2008 09:18:29 PM UTC, comment #3:

Tried the PP_COMPATIBLE_DEVICE_FONT_HANDLING, didn't work.
Looking at the code that macro only influences color, not size.
To be researched:

1. if the movies deal with device rather then embedded fonts.
2. if device fonts are supposed to be scaled.

Sandro Santilli <strk>
Project Member
Thu 06 Mar 2008 08:41:38 PM UTC, comment #2:

note to self (or to whoever wants to try that):
try uncommenting the PP_COMPATIBLE_DEVICE_FONT_HANDLING
define in server/edit_text_character.cpp

Sandro Santilli <strk>
Project Member
Mon 03 Mar 2008 12:39:35 PM UTC, comment #1:

Another example of the same bug:
http://www.upscale.utoronto.ca/GeneralInterest/Harrison/Flash/ClassMechanics/VertCircular/VertCircular.swf

Sandro Santilli <strk>
Project Member
Mon 03 Mar 2008 12:21:54 PM UTC, original submission:

http://www.upscale.utoronto.ca/GeneralInterest/Harrison/Flash/ClassMechanics/Projectile/Projectile.swf
version: SWF6
md5: 92458afd64496f8089b843e81317b0c9

The dynamic text value of y0 and t? is disaplayed in a weird
way (bigger then expected, and with wrong aspect ratio).
Could be one of those cases in which an application relies on
the NON transformation of device fonts (didn't test if it's device font btw)

Sandro Santilli <strk>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by bjacques (Updated the item)
  • -unavailable- added by bwy (Posted a comment)
  • -unavailable- added by strk (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 08 Jul 2013 12:27:34 PM UTCbjacquesStatusReady For Test=>Fixed
      Assigned toNone=>bwy
      Open/ClosedOpen=>Closed
    Wed 08 Jul 2009 03:56:00 PM UTCbwyStatusConfirmed=>Ready For Test
    Wed 17 Sep 2008 10:09:31 PM UTCstrkStatusNone=>Confirmed

    Back to the top


    Powered by Savane 3.1-cleanup