Skip to content

fix(app): #154 Fixed Window Size when restoring window - #196

Merged
lulzsun merged 3 commits into
lulzsun:mainfrom
sshcrack:fix-restore
Apr 25, 2024
Merged

lulzsun merged 3 commits into
lulzsun:mainfrom
sshcrack:fix-restore

Conversation

@sshcrack

Copy link
Copy Markdown
Contributor

Added a _PrevSize Variable to Program in order to restore the previous size

Comment thread RePlays.csproj Outdated
<NugetTools>$(PkgNuGet_CommandLine)\tools</NugetTools>
<SquirrelTools>$(PkgClowd_Squirrel)\tools</SquirrelTools>
<Version>1.0.0</Version>
<Version>1.3.1</Version>

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it is necessary to change the version here. Version is handled during the dotnet publish where we specifically pass in a version number.

Comment thread RePlays.sln
VisualStudioVersion = 17.9.34728.123
MinimumVisualStudioVersion = 10.0.40219.1
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "RePlays", "RePlays.csproj", "{10BA385C-0777-4293-BED2-C85223FC2CF8}"
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "RePlays", "RePlays.csproj", "{10BA385C-0777-4293-BED2-C85223FC2CF8}"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting that using a newer Visual Studio Version changes the project GUID. It does not seem like it affects anything really so I guess we can keep the changes.

Not a necessary change, but let's keep some things to how it used to be

@lulzsun lulzsun left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested it, looks and works as expected. Thanks! Ready to merge.

I do want to mention that this pull request did contain some unrelated changes to files to the following files:

  • Program.cs
  • RePlays.csproj
  • RePlays.sln

While these changes were minor and/or most likely auto-generated, I recommend to be mindful of committing them in a pull request if they aren't related to the main changes of a PR just so we can keep commit history relevant.

Since they don't affect the project's functionality, I'm okay with overlooking them this time. Thanks again for the pull request.

@lulzsun
lulzsun merged commit 5ec1b02 into lulzsun:main Apr 25, 2024
@sshcrack

Copy link
Copy Markdown
Contributor Author

Alright, will be careful not to commit unrelated changes. Thanks for reviewing!

@sshcrack
sshcrack deleted the fix-restore branch April 25, 2024 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants