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.
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 if specific font sizes are needed, 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 t o 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 size might be set other than setFont, can you also review that ? Tim Boudreau can provide information also on general issue. Issue will be 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 and Nevada. Specific calls to setFont seen in source code: setFont(new Font("Monospaced", Font.PLAIN, 12)); output.setFont(new Font("Monospaced", Font.PLAIN, 12)); tipsBrowser.setFont(new java.awt.Font("Dialog", 0, 11));
Ken, If you've analyzed the source code, please provide the specific classes where this problem occurs - java source file & line number would be ideal - preferable one issue for each instance of the problem, so it can be assigned to the developer responsible for that class. Also, doesn't the umbrella task to fix this cover this issue? I suggest closing this issue as invalid and providing individual issues (P2 if you want) to block the umbrella task issue for fixing this globally.
Created attachment 10487 [details] patch for font sizes
attached patch should solve fontsize problems for core. I will test it more and commit later.
Created attachment 10510 [details] refined patch, fully tested
I'm taking this issue, hope you agree Marek.
fixed in main trunk. Cvs output: /cvs/openide/src/org/openide/text/CloneableEditorSupport.java,v <-- CloneableEditorSupport.java new revision: 1.87; previous revision: 1.86 /cvs/openide/api/examples/org/netbeans/examples/modules/globalactions/TestAction.java,v <-- TestAction.java new revision: 1.11; previous revision: 1.10
If verification should be in ide itself, can you tell us what actual windows relate to the code lines from original bug that were fixed ? ken.frank@sun.com
Answer to Ken: Editor windows which use default font (I don't know which they are :-( and window opened in example TestAction (you have to run the example to see it).
Fix merged into release35 branch: /cvs/openide/api/examples/org/netbeans/examples/modules/globalactions/TestAction.java,v <-- TestAction.java new revision: 1.10.4.1; previous revision: 1.10 /cvs/openide/src/org/openide/text/CloneableEditorSupport.java,v <-- CloneableEditorSupport.java new revision: 1.81.2.3; previous revision: 1.81.2.2
verified [030616_1] OutputPanel NotifyExceptionPanel WelcomePanel