Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

When cold starting on Win10 with multi monitors, starts basically invisible #46

Closed
wants to merge 3 commits into from

Conversation

tig
Copy link

@tig tig commented Apr 10, 2018

I found on two different Win10 machines with multi-mon, the first time I ran winfile.exe the window was basically just the title bar (zero height client area) and infinitely wide (actually 0x80000000)).

This one line I removed is the cause and I can't see why it would be needed. SystemParametersInfo(SPI_GETWORKAREA) returns the correct RECT.

Did not test on any version of Windows other than Win10...

@craigwi
Copy link
Contributor

craigwi commented Apr 10, 2018

Hey Charlie, I thought I tested that, but glad it was a simple fix.

Can you revert the change to .gitignore? I'll merge your request after that.

@tig
Copy link
Author

tig commented Apr 10, 2018

Sorry, my git skillz are lame. I couldn't figure out how to revert the .gitignore change (I actually thought I had reverted it before the pull request). I just copied the original over my edited version and added/commited that.

@deadbaed
Copy link

@tig to revert a commit, from the command line, you run git revert xxx, where xxx is the commit id, you can find it when you run git log.

hope it helps in the future :)

craigwi added a commit that referenced this pull request Apr 11, 2018
@craigwi
Copy link
Contributor

craigwi commented Apr 11, 2018

Right fix is more complicated. See my change. Thanks Charlie!

@craigwi craigwi closed this Apr 11, 2018
@craigwi craigwi added the fixed differently Spirit of PR was taken if not all of the details; credit given to original author. label Apr 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
fixed differently Spirit of PR was taken if not all of the details; credit given to original author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants