Code Monkey home page Code Monkey logo

Comments (21)

GerbilSoft avatar GerbilSoft commented on May 26, 2024

The original file has a correct aspect ratio: http://rpdb.gerbilsoft.com/ps2/cover/SLES/SLES-55103.jpg

This is definitely a Windows rendering issue. I'll need to figure out why it's being rendered incorrectly.

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

This is also affecting Wii U boxart, and probably others. (Tested on Windows 7.)

It should be creating a squared thumbnail, but obviously something is wrong...

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

Watch, it's going to be something stupid like Windows changing how thumbnails work.... again

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

It seems the "squaring" function that's needed for icons to work properly somehow caused long boxes to get squished. I'm pretty sure it worked before...

Disabled squaring on everything except icons and it seems to work now. Tested on Windows 7 with IThumbnailProvider. Will test IExtractImage later to ensure it isn't broken, but IExtractImage is generally only used on Windows XP and other older software.

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

As soon as the build pops out of appveyor I will test on my end to make sure everything is good

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

@GerbilSoft Aspect Ratio is still wrong
image
image

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

Did you clear the thumbnail cache? (rp-config has a convenient button to do this.)

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

Yep, unregistered the old dll, used rp-config to clean both rp's cache and the system cache, checked some isos to make sure that they weren't showing anything other than the default icon, and then registered the new dll

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

I'll need to test it on my Win10 VM then. (It seems to have fixed it for Win7, at least...)

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

I'm not able to reproduce this on Windows 10 v1809. What version of Windows 10 (or 11) are you using?

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

image

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

Will set up a Windows 11 22H2 VM tomorrow then. This will probably be the last bugfix I'll look at before releasing v2.2.1.

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

I have good news. Turns out that Windows and its ability to be dumb never ends, and I was actually still running the old build even though it said that it had unregistered it. New DLL generates box arts correctly!
image

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

Unless you can think of anything else you want me to test or last minute fixes for something, I think everything is good on my end now.

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

Something I have noticed is that initially it still generates as the squished thumbnail and then seemingly makes a second pass and fixes the aspect ratio.
You can see what I am talking about here, the top is generating the correct icon, but the bottom is squished. Eventually, once they catch up with each other, they will all snap to the correct ratio. It's almost like there are two threads trying to generate them, one with the fix and one without

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

Reopening, just in case you want to look into that. If not, feel free to close, it seems like the correct one gets saved to the cache in the end so it just ends up being a visual quirk

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

That's the icon, which appears to be squished. (If you view file properties, you'll see the squished icon.) I'm not sure why it's getting squished, since the squaring is supposed to add columns but not change the aspect ratio...

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

It seems removing squaring for icons fixes it. I know I had problems with non-square icons before, but that might have been on Windows XP.

I'll change it to only enable squared icons on Windows XP and earlier. (Not sure about IExtractImage; will need to test that later.)

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

Icon squaring is now disabled on Vista and later. My WinXP VM isn't working properly due to some weird interaction with VirtualBox and SMAP, and reporting a bug to VirtualBox requires registering an Oracle account. I could probably work around this by disabling SMAP, but I'd rather not do that...

from rom-properties.

Masamune3210 avatar Masamune3210 commented on May 26, 2024

I truly feel for you, I don't think anybody willingly wants to deal with oracle lol

from rom-properties.

GerbilSoft avatar GerbilSoft commented on May 26, 2024

Closing since this should be fixed in Windows Vista and later. Will need to test XP again sometime...

from rom-properties.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.