This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 33702 - I18N - hardcoded font sizes in source code not relative to global font
Summary: I18N - hardcoded font sizes in source code not relative to global font
Status: VERIFIED FIXED
Alias: None
Product: obsolete
Classification: Unclassified
Component: vcsgeneric (show other bugs)
Version: 3.x
Hardware: Sun Solaris
: P2 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords: I18N
Depends on:
Blocks: 33246
  Show dependency tree
 
Reported: 2003-05-18 17:51 UTC by Ken Frank
Modified: 2003-07-22 17:14 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ken Frank 2003-05-18 17:51:32 UTC
Analysis of source code of current product shows
some some calls to setFont that hardcode a certain
font
size that isn't relative to global font size.

Usually, fonts are obtained from the global font
size
value of the product. But sometimes specific font
sizes
are wanted, and if so, they should be assigned
based on
a relative offset from the default font size
instead of
as a specific size.

This is because the localized products will soon
be able
to set the default font size needed for proper
appearance
of their character sets, and hardcoded fonts may
be too
small for meeting those requirements.

If there are other parts of code where hardcoded
font siz
e
might be set other than setFont, can you also
review it?
If the code shown below is not part of
product,please
close issue.

Issue marked with FONTSIZE keyword which means it
will
not be placed on dashboards for 3.5 or nevada as
needing waiving but fixes still need to be for the
ML
release of 3.5/Nevada.

Also will mark as blocking 33246, general task on
resizing of windows, which deals also with font
size
setting.

Specific calls to setFont seen in source code:
taRevLog.setFont(new java.awt.Font("Default", 0,
11));
Comment 1 Martin Entlicher 2003-05-19 09:54:41 UTC
You probably mean file
vcsgeneric/profilessupport/src/org/netbeans/modules/vcs/profilessupport/commands/log/LogInfoPanel.java

The module "profilessupport" is currently in no build. Thus this is
not a problem of Nevada nor NetBeans.

I'll fix it anyway so that it's not introduced later to the standard
distribution.
Comment 2 Martin Entlicher 2003-05-19 12:31:56 UTC
Fixed in the main trunk:
/cvs/vcsgeneric/profilessupport/src/org/netbeans/modules/vcs/profilessupport/commands/log/LogInfoPanel.form,v 
<--  LogInfoPanel.form
new revision: 1.2; previous revision: 1.1
/cvs/vcsgeneric/profilessupport/src/org/netbeans/modules/vcs/profilessupport/commands/log/LogInfoPanel.java,v 
<--  LogInfoPanel.java
new revision: 1.4; previous revision: 1.3

And in the projects branch:
/cvs/vcsgeneric/profilessupport/src/org/netbeans/modules/vcs/profilessupport/commands/log/LogInfoPanel.form,v 
<--  LogInfoPanel.form
new revision: 1.1.36.1; previous revision: 1.1
/cvs/vcsgeneric/profilessupport/src/org/netbeans/modules/vcs/profilessupport/commands/log/LogInfoPanel.java,v 
<--  LogInfoPanel.java
new revision: 1.3.20.1; previous revision: 1.3

The Target Milestone "future" does not make much sense, but there's
not anything better. It's just fixed in the current code base.
Comment 3 Jan Chalupa 2003-06-17 08:26:44 UTC
The code is not part of the S1S 5 codebase -> NOS1S5.
Comment 4 dmladek 2003-07-21 18:22:14 UTC
Looking into code ... it seems fixed
for prj40_prototype branch

only location path ov LogInfoPanel has changed....
Comment 5 dmladek 2003-07-22 17:14:09 UTC
BTW: this bug sounds like a dup of issue #33672

but it was fixed and the origin one too... so I won't make any changes
of this bug resolution