Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blurry Image with installer #4442

Closed
RoiEXLab opened this issue Dec 18, 2018 · 32 comments
Closed

Blurry Image with installer #4442

RoiEXLab opened this issue Dec 18, 2018 · 32 comments
Labels
Problem A problem, bug, defect - something to fix

Comments

@RoiEXLab
Copy link
Member

Noticing this for a while now, but I wanted to resurrect #746 again.
Basically the installer somehow doesn't work really well with a zoom setting that isn't 100% on windows:
Installer 100% Zoom:
100% Zoom Installer
Installer 125% Zoom:
125% Zoom Installer
Running from source 125% Zoom:
125% Zoom Source

Haven't tried out java 9 or something that's supposed to fix a couple of HiDPI issues (or a more recent TripleA version with a newer version of Install4j)
But this is definitely something we should report if its still present in more recent versions.
Thoughts?

@Cernelius
Copy link
Contributor

I said this about 2 years ago. Anyway, it has always been so since I started using the installer. Haven't tested it in some time, but if you, instead, use the "all platforms", there is not this behaviour (at least last time I did).

The other thing that I reported is that Windows zoom zooms the custom cursor too, making it pixelated and very bad looking. Moreover, it zooms the image but not the hotspot, so you can see, for example in Pact of Steel, that the target will be off.

The other issue was that the purchase etc. Windows were sized incorrectly, providing scroll bars as default, but that has been corrected by @ron-murhammer some time ago.

The above are the reasons why, in the past, I suggested to keep the all platforms available (and with the "exe" inside!) for any users to download, like it was during veq-old times.

I've no idea how windows users, with the commonly used zoom, can guess anything else but TripleA being a seriously bad looking blurred game, and that's it, after they install it (it looks really bad to me; try it at 150% zoom...).

@DanVanAtta DanVanAtta added the Problem A problem, bug, defect - something to fix label Jan 4, 2019
@stale
Copy link

stale bot commented Sep 15, 2019

The TripleA team regrets this issue was not solved sooner. To keep focus on latest issues and the most pressing tasks, this issue has been automatically marked as stale because it has not had recent activity. If the issue may be closed, please do so. If there are remaining items, we encourage for those items to be resubmitted as new, independent, concise tasks. The strategy is divide and conquer. Thank you for your contributions.

@stale stale bot added the Stale label Sep 15, 2019
@DanVanAtta
Copy link
Member

Does this problem happen when you install with zoom turned on, or does this happen whenever the game is launched and zoom is turned on?

This is an odd one as in theory there is little different between the installer and stand-alone jar.

@Cernelius do you remember if this was a problem on the 1.7 series and before?

@stale stale bot removed the Stale label Sep 16, 2019
@Cernelius
Copy link
Contributor

This never happened to me before 1.9, as long as I remember, but I suppose this was due to the fact that I always used the all platforms zip, instead of any installers, and I recall this problem happened using the installer but not using the all platforms, early in 1.9. Practically, what I remember happened was that the all platforms had TripleA working pixel by pixel, thus looking good, while the installer used to expand TripleA by the specified Windows zoom, thus looking blurred.

I've not tested this issue again in more than 1 year, as it is a long time I just keep the general windows zoom at 100%, also since that causes issues with the custom cursors too.

@DanVanAtta
Copy link
Member

@Cernelius which (exact) version of java do you have installed?

@DanVanAtta
Copy link
Member

I'm thinking we might have a JDK problem, there are not many differences between the all-platforms version compared to installer. Something like this could be the issue: https://bugs.openjdk.java.net/browse/JDK-8156149 "Blurry rendering on Windows 7 at 125% screen setting"

@RoiEXLab
Copy link
Member Author

@DanVanAtta I recall having this issue too in the past on my laptop.
My best guess at the moment is that it's caused by the windows zoom setting (125% on my laptop). Not sure if this changed in java 11 though, there were some minor improvements to swing I noticed (sideway scrolling now fires a normal mousewheel event with shift down mask it seems)

@RoiEXLab
Copy link
Member Author

Nevermind, you googled faster than I could type

@DanVanAtta
Copy link
Member

@Cernelius or @RoiEXLab can you confirm if this is still a problem?

@RoiEXLab
Copy link
Member Author

@DanVanAtta If @Cernelius doesn't check until Friday I'll have enought time to check myself.

From the linked JDK issue it seems this problem was fixed in JDK 9

@RoiEXLab
Copy link
Member Author

Also I didn't realize I opened this issue in the first place, but it makes sense now that I knew what the underlying problem was ^^

@Cernelius
Copy link
Contributor

I could test, but, then, it would be still up to @RoiEXLab, so I guess it makes more sense for me to wait, and eventually double check what he checked, in case it gets fixed.

@DanVanAtta
Copy link
Member

@Cernelius that does not quite make sense; if you're going to double check anyways, why not go ahead? 😁

There's nothing special in terms of this verification, anyone that had the original problem can verify if it's gone with the latest prerelease. We fixed nothing, the latest prerelease has a more recent version of java compared to 1.9

@RoiEXLab
Copy link
Member Author

Yup, I can 100% confirm this is still an issue

@Cernelius
Copy link
Contributor

I meant, if this gets fixed, I'll see if it's fixed for me too. Maybe consider the custom cursors too, they get zoomed as much, and in a very raw way. It would be good both the map and the custom cursor (meaning the skin in general) are not affected.

@RoiEXLab
Copy link
Member Author

Well the problem is not really on us.
Something is wrong with install4j that causes this, because if I run the game from source code directly, I don't have any problems whatsoever.
I mean workaround is to go to your windows settings and set thr zoom level of your display to 100% when playing.

@DanVanAtta Might be worth upgrading install4j to see if that fixes the issue

@Cernelius
Copy link
Contributor

Or still providing an all platform (zipped without installer) version, with the exe and all included, for easy of use, and advising Windows users to preferably use that over the Windows installer.

Not like this is the first time I'm suggesting keeping the all platform available, by the way. Not sure if one of the various release options here is like the old all platform, but that's definitely not for the common user.

@DanVanAtta
Copy link
Member

DanVanAtta commented Sep 22, 2019

@RoiEXLab see if you can launch the game with the same JRE bundled with install4j. The key difference is that, once you have the files deployed on system, the fact it was deployed with install4j should be immaterial.

advising Windows users to preferably use that over the Windows installer.

@Cernelius - No. Otherwise what's the point of an installer at all? Why even create or have it at all?

Warning users if they have a zoom over 100% would be useful though, but better we just fix this problem.

@RoiEXLab
Copy link
Member Author

Launching triplea using cli directly (with just the bundled JRE) works completely fine.
Only when using the exe, the image is blurry.

The exe is also responsible for the splash screen, so that might have something to do with it

@DanVanAtta
Copy link
Member

Okay, thanks for checking. If the problem is in the EXE, we should then confirm if latest install4j has the same issue and try a variety of the JDK options to rule them all out or find one that has no issues.

@Cernelius
Copy link
Contributor

Only when using the exe, the image is blurry.

What do you mean? You mean that if we would have an all-platform zipped with an exe inside it would still have the same problem? Did you create such a non-installed-but-with-exe package and test it?

@Cernelius
Copy link
Contributor

Cause I remember than when the all platform was still available it worked fine also by just pasting the exe into it (after having unzipped it), to run it. Only the installed version had this problem, but, again, years since then.

@Cernelius
Copy link
Contributor

Cernelius commented Sep 23, 2019

Warning users if they have a zoom over 100% would be useful though, but better we just fix this problem.

By now, it is years that Windows users (I assume all of them using a non 100% zoom) have this problem. I reported it a long time ago, for a time keeping using the all platform, then, myself, just switched the zoom to 100%. It's by now more than 2 years that I keep Windows at 100% zoom.

I'm actually surprised nobody complained. I assume that people that have this issue can do nothing else but assume that TripleA just looks bad like that normally; so not much to say.

Also, please, don't forget about custom cursors, that are affected by this too.

Side note, there were also resize issues with Windows zoom for the game's windows, like the purchase screen, but @ron-murhammer fixed those. Not sure if that may be of any help for fixing this matter too.

@DanVanAtta
Copy link
Member

For context, are we talking text zoom, or zoom of everything? Can someone detail where the setting is exactly, to be sure we are all looking at the same thing.

@RoiEXLab
Copy link
Member Author

Windows isn't overly specific on this. It says "Text, Apps and other elements"
Screenshot

@RoiEXLab
Copy link
Member Author

@Cernelius

What do you mean?

Exactly what I was saying. If I use the exe generated by install4j, the image is blurry, if I don't use it it's clear.

@Cernelius
Copy link
Contributor

@Cernelius

What do you mean?

Exactly what I was saying. If I use the exe generated by install4j, the image is blurry, if I don't use it it's clear.

How about if you paste the TripleA package without installing it (like in the old all platform), then add an exe file to run it, still without using any installers?

From what you said, it sounded like the problem was running it through an exe, while I remember there was no problem back then we had a zipped all-platform package with the exe included.

@Cernelius
Copy link
Contributor

@DanVanAtta I believe the general Windows zoom is supposed to zoom everything. If it does it with absolute consistence, I cannot say, since I'm not using it in years (actually stopped using it because of this issue here).

@ssoloff
Copy link
Member

ssoloff commented Sep 23, 2019

The above are the reasons why, in the past, I suggested to keep the all platforms available (and with the "exe" inside!) for any users to download, like it was during veq-old times.

@Cernelius Note that the "all platforms zip" is still available, but it was renamed to triplea-game-headed-<version>.zip a year or so ago. After downloading it, unzip it, open a command prompt in the folder to which you unzipped it, and then run java -jar bin\triplea-game-headed-<version>.jar to start TripleA, replacing <version> appropriately (or just use Tab completion after typing the bin\).

@DanVanAtta
Copy link
Member

@Cernelius / @RoiEXLab would either of you mind checking the latest? The installer software has been updated to a new major version and the bundled JRE is now more recent too.

@RoiEXLab
Copy link
Member Author

Thanks for the reminder, will check later this evening.
Will also review the PRs then

@RoiEXLab
Copy link
Member Author

Good news, looks like they fixed this issue 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Problem A problem, bug, defect - something to fix
Projects
None yet
Development

No branches or pull requests

4 participants