[jfreechart-dev] [ jfreechart-Bugs-1995345 ] Bad font height conversion in SWTUtils.toSwtFontData

[SourceForge.net <noreply@so...>]

Bugs item #1995345, was opened at 2008-06-16 10:18
Message generated for change (Settings changed) made by nenry
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Matthew Hall (qualidafial)
>Assigned to: Henry Proudhon (nenry)
Summary: Bad font height conversion in SWTUtils.toSwtFontData

Initial Comment:
A blogger documented this problem on: http://www.schmeeky.co.uk/articles/2008-06/charts.aspx

See SWTUtils.java:

// convert the font size (in pt for awt) to height in pixels for swt
int height = (int) Math.round(font.getSize() * 72.0
        / device.getDPI().y);
fontData.setHeight(height);

But FontData.setHeight() actually expects points, not pixels:

>From FontData.java:

 * Sets the height of the receiver. The parameter is
 * specified in terms of points, where a point is one
 * seventy-second of an inch.

This conversion especially causes problems on a printer, where the DPI is usually much higher than on a display.    Theoretically you should be able to just say:

fontData.setHeight(font.getSize());

Matthew

----------------------------------------------------------------------

>Comment By: Henry Proudhon (nenry)
Date: 2008-06-18 01:02

Message:
Logged In: YES 
user_id=1379312
Originator: NO

Thanks for your input Matthew, I recall I had to use this to work around a
problem by just converting the font using points. This probably need to be
revisited now.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

[jfreechart-dev] [ jfreechart-Bugs-1995345 ] Bad font height conversion in SWTUtils.toSwtFontData

[SourceForge.net <noreply@so...>]

Bugs item #1995345, was opened at 2008-06-16 10:18
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Matthew Hall (qualidafial)
Assigned to: Nobody/Anonymous (nobody)
Summary: Bad font height conversion in SWTUtils.toSwtFontData

Initial Comment:
A blogger documented this problem on: http://www.schmeeky.co.uk/articles/2008-06/charts.aspx

See SWTUtils.java:

// convert the font size (in pt for awt) to height in pixels for swt
int height = (int) Math.round(font.getSize() * 72.0
        / device.getDPI().y);
fontData.setHeight(height);

But FontData.setHeight() actually expects points, not pixels:

>From FontData.java:

 * Sets the height of the receiver. The parameter is
 * specified in terms of points, where a point is one
 * seventy-second of an inch.

This conversion especially causes problems on a printer, where the DPI is usually much higher than on a display.    Theoretically you should be able to just say:

fontData.setHeight(font.getSize());

Matthew

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

[jfreechart-dev] [ jfreechart-Bugs-1995345 ] Bad font height conversion in SWTUtils.toSwtFontData

[SourceForge.net <noreply@so...>]

Bugs item #1995345, was opened at 2008-06-16 10:18
Message generated for change (Settings changed) made by nenry
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Matthew Hall (qualidafial)
>Assigned to: Henry Proudhon (nenry)
Summary: Bad font height conversion in SWTUtils.toSwtFontData

Initial Comment:
A blogger documented this problem on: http://www.schmeeky.co.uk/articles/2008-06/charts.aspx

See SWTUtils.java:

// convert the font size (in pt for awt) to height in pixels for swt
int height = (int) Math.round(font.getSize() * 72.0
        / device.getDPI().y);
fontData.setHeight(height);

But FontData.setHeight() actually expects points, not pixels:

>From FontData.java:

 * Sets the height of the receiver. The parameter is
 * specified in terms of points, where a point is one
 * seventy-second of an inch.

This conversion especially causes problems on a printer, where the DPI is usually much higher than on a display.    Theoretically you should be able to just say:

fontData.setHeight(font.getSize());

Matthew

----------------------------------------------------------------------

>Comment By: Henry Proudhon (nenry)
Date: 2008-06-18 01:02

Message:
Logged In: YES 
user_id=1379312
Originator: NO

Thanks for your input Matthew, I recall I had to use this to work around a
problem by just converting the font using points. This probably need to be
revisited now.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

[jfreechart-dev] [ jfreechart-Bugs-1995345 ] Bad font height conversion in SWTUtils.toSwtFontData

[SourceForge.net <noreply@so...>]

Bugs item #1995345, was opened at 2008-06-16 10:18
Message generated for change (Comment added) made by nenry
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Matthew Hall (qualidafial)
Assigned to: Henry Proudhon (nenry)
Summary: Bad font height conversion in SWTUtils.toSwtFontData

Initial Comment:
A blogger documented this problem on: http://www.schmeeky.co.uk/articles/2008-06/charts.aspx

See SWTUtils.java:

// convert the font size (in pt for awt) to height in pixels for swt
int height = (int) Math.round(font.getSize() * 72.0
        / device.getDPI().y);
fontData.setHeight(height);

But FontData.setHeight() actually expects points, not pixels:

>From FontData.java:

 * Sets the height of the receiver. The parameter is
 * specified in terms of points, where a point is one
 * seventy-second of an inch.

This conversion especially causes problems on a printer, where the DPI is usually much higher than on a display.    Theoretically you should be able to just say:

fontData.setHeight(font.getSize());

Matthew

----------------------------------------------------------------------

>Comment By: Henry Proudhon (nenry)
Date: 2008-07-01 12:50

Message:
Logged In: YES 
user_id=1379312
Originator: NO

In principle you're right Matthew but I included this code because just
passing font.getSize() produced noticely smaller font with SWT.
Now we should ask the question, do we want to match the size of awt fonts
on screen or not ?
David maybe you've got an idea on this ?

----------------------------------------------------------------------

Comment By: Henry Proudhon (nenry)
Date: 2008-06-18 01:02

Message:
Logged In: YES 
user_id=1379312
Originator: NO

Thanks for your input Matthew, I recall I had to use this to work around a
problem by just converting the font using points. This probably need to be
revisited now.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

[jfreechart-dev] [ jfreechart-Bugs-1995345 ] Bad font height conversion in SWTUtils.toSwtFontData

[SourceForge.net <noreply@so...>]

Bugs item #1995345, was opened at 2008-06-16 10:18
Message generated for change (Comment added) made by nenry
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Matthew Hall (qualidafial)
Assigned to: Henry Proudhon (nenry)
Summary: Bad font height conversion in SWTUtils.toSwtFontData

Initial Comment:
A blogger documented this problem on: http://www.schmeeky.co.uk/articles/2008-06/charts.aspx

See SWTUtils.java:

// convert the font size (in pt for awt) to height in pixels for swt
int height = (int) Math.round(font.getSize() * 72.0
        / device.getDPI().y);
fontData.setHeight(height);

But FontData.setHeight() actually expects points, not pixels:

>From FontData.java:

 * Sets the height of the receiver. The parameter is
 * specified in terms of points, where a point is one
 * seventy-second of an inch.

This conversion especially causes problems on a printer, where the DPI is usually much higher than on a display.    Theoretically you should be able to just say:

fontData.setHeight(font.getSize());

Matthew

----------------------------------------------------------------------

>Comment By: Henry Proudhon (nenry)
Date: 2008-07-01 13:00

Message:
Logged In: YES 
user_id=1379312
Originator: NO

sorry for the confusion, just passing the awt font size produce much
larger results with swt.
since in a chart the space (for axis, legend...) is computed from the awt
entities, I think we should match as closely as possible the size of the
awt font. hence the the of the ensureSameSize boolean.
Comments are welcome

----------------------------------------------------------------------

Comment By: Henry Proudhon (nenry)
Date: 2008-07-01 12:50

Message:
Logged In: YES 
user_id=1379312
Originator: NO

In principle you're right Matthew but I included this code because just
passing font.getSize() produced noticely smaller font with SWT.
Now we should ask the question, do we want to match the size of awt fonts
on screen or not ?
David maybe you've got an idea on this ?

----------------------------------------------------------------------

Comment By: Henry Proudhon (nenry)
Date: 2008-06-18 01:02

Message:
Logged In: YES 
user_id=1379312
Originator: NO

Thanks for your input Matthew, I recall I had to use this to work around a
problem by just converting the font using points. This probably need to be
revisited now.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

[jfreechart-dev] [ jfreechart-Bugs-1995345 ] Bad font height conversion in SWTUtils.toSwtFontData

[SourceForge.net <noreply@so...>]

Bugs item #1995345, was opened at 2008-06-16 10:18
Message generated for change (Comment added) made by nenry
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Matthew Hall (qualidafial)
Assigned to: Henry Proudhon (nenry)
Summary: Bad font height conversion in SWTUtils.toSwtFontData

Initial Comment:
A blogger documented this problem on: http://www.schmeeky.co.uk/articles/2008-06/charts.aspx

See SWTUtils.java:

// convert the font size (in pt for awt) to height in pixels for swt
int height = (int) Math.round(font.getSize() * 72.0
        / device.getDPI().y);
fontData.setHeight(height);

But FontData.setHeight() actually expects points, not pixels:

>From FontData.java:

 * Sets the height of the receiver. The parameter is
 * specified in terms of points, where a point is one
 * seventy-second of an inch.

This conversion especially causes problems on a printer, where the DPI is usually much higher than on a display.    Theoretically you should be able to just say:

fontData.setHeight(font.getSize());

Matthew

----------------------------------------------------------------------

>Comment By: Henry Proudhon (nenry)
Date: 2008-07-01 13:54

Message:
Logged In: YES 
user_id=1379312
Originator: NO

awt font always assume a 72 dpi resolution so we need stuff like this to
convert to swt:
int height = (int) Math.round(font.getSize() * 72.0 / device.getDPI().y);

For instance my screen is 121 dpi. So SWT fonts of size 18 are huge in
comparison to awt ones.
What is strange is doing the conversion makes the SWT fonts to small (this
I don't understand).

so to explain a bit more, in the TimeSeriesDemo, the awt title font is 18,
converting to SWT gives me size 11 but after the ensureSameSize hack, I end
up with size 14 which render correctly.
I'm not sure I can do more here...

----------------------------------------------------------------------

Comment By: Henry Proudhon (nenry)
Date: 2008-07-01 13:00

Message:
Logged In: YES 
user_id=1379312
Originator: NO

sorry for the confusion, just passing the awt font size produce much
larger results with swt.
since in a chart the space (for axis, legend...) is computed from the awt
entities, I think we should match as closely as possible the size of the
awt font. hence the the of the ensureSameSize boolean.
Comments are welcome

----------------------------------------------------------------------

Comment By: Henry Proudhon (nenry)
Date: 2008-07-01 12:50

Message:
Logged In: YES 
user_id=1379312
Originator: NO

In principle you're right Matthew but I included this code because just
passing font.getSize() produced noticely smaller font with SWT.
Now we should ask the question, do we want to match the size of awt fonts
on screen or not ?
David maybe you've got an idea on this ?

----------------------------------------------------------------------

Comment By: Henry Proudhon (nenry)
Date: 2008-06-18 01:02

Message:
Logged In: YES 
user_id=1379312
Originator: NO

Thanks for your input Matthew, I recall I had to use this to work around a
problem by just converting the font using points. This probably need to be
revisited now.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=115494&aid=1995345&group_id=15494