> > > From the history of the PR: It breaks existing code/tests, so is > basicly not mergeable. > > One observation about High DPI/antialias changes: Work should be done > to fix Swing Upstream to provide sensual defaults, its not as if > netbeans is the only Swing Application. If it is not done on the JDK > level, future changes in the JDK might cause further problems. >
No, no, 10000 times no. Been there, done that. You do not wait for someone else to fix your problem. What's the logic there? Well, who cares if nobody uses our product - it's not my job to fix it, it's their fault, it's their problem. How well do you think that actually works? One of the biggest handicaps NetBeans had, up through 2002, was that, post Sun-acquisition, we were forced into using Metal Look & Feel regardless of it looking alien on desktops of the day. Aesthetics matter. Readability matters. It is NOT a workable solution to just hope someone else solves the problem. As for High DPI, many monitors and video cards *do not accurately report resolution or physical size.* In the case of going through things like projectors, HDMI to DVI adapter boxes and such, the information simply does not exist. No change in Swing is ever going to fix that. No auto-detection will ever work for that. Asking the user on start needs to be coordinated with the other startup > dialogs (Import Settings, Anonymous data sending). We already have an > appearence tab to choose the LAF, why not add options there? > So, show the UI once with microscopic fonts and no antialiasing, and say "Hey user, this looks awful, doesn't it? How about we fix it and then restart?". We *could*, but why? It's certainly not better user experience to pop up something ugly, then pop something up in front of it to fix it, then tear it all down and put it back again. This isn't a "Oh, some users might want to change this sometime, so let's add an option" thing. It's a "This will make most users run it once and uninstall it" thing. The only thing that should be needed for tests to pass is for a variable to be set in netbeans.conf *only when tests are run* that tells the startup script the configuration is already set. -Tim
