gdx-backend-jtransc icon indicating copy to clipboard operation
gdx-backend-jtransc copied to clipboard

Graphics support

Open czyzby opened this issue 9 years ago • 3 comments

  • [x] Icons seem to be missing. It would be nice if icons from LwjglApplicationConfiguration were applied to the Lime game window.
  • [ ] The application window (CPP/Lime on Linux) is bigger than the original (Java/LWJGL). Application window width and height settings don't seem to be honored.
  • [x] PPI/PPC (pixel per inch/centimeter) and density return mock-up values. The values are way off - I suggest using GWT fixed PPI (96f), PPC (96f/2.54f) and density (96f/160f) values until they are implemented properly.
  • [ ] Cursor image cannot be changed.
  • [x] Fullscreen mode does not seem to be correctly handled. Returned monitors and displayed modes are mock-ups.

czyzby avatar May 26 '16 20:05 czyzby

I'm using this for density: https://github.com/openfl/lime/blob/develop/lime/system/Display.hx

Not sure if it is right. So PPC are redundant? Since centimeters can be calculated from inches? Density is redundant too?

There is a GraphicsAdaptor so I can extend it and get that common stuff already? Or there is just the interface?

soywiz avatar May 26 '16 20:05 soywiz

Well, I think density can be read directly with native Android APIs, and LibGDX Graphics simply emulate this functionality using PPI on platforms where is not supported. I wouldn't be surprised if it comes down to the same exact implementation in Android native APIs as well. PPC is just out there for your convenience, I guess. See AndroidGraphics and LwjglGraphics.

There is no abstract class as far as I know. Some platforms actually report these values "natively", so I guess this is the reason.

czyzby avatar May 26 '16 20:05 czyzby

You should check out if the reported values are OK, though. I think DPI reported by Lime might actually match density rather than PPI. In other words, check if getDensity() returns about 96/160 on desktop - if not, you probably have to return getDensity() * 160f as PPI and native Haxe DPI as density.

czyzby avatar May 26 '16 20:05 czyzby