-
Notifications
You must be signed in to change notification settings - Fork 586
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
First set changes to enable high DPI in Service Bus Explorer #797
First set changes to enable high DPI in Service Bus Explorer #797
Conversation
jmoralesv
commented
Jul 28, 2024
- Adding support for .NET Framework 4.7.2 and using the settings in the app.manifest file for enabling high DPI support, and for taking advantage of some new helper methods the framework offers for high DPI related calculations, such as LogicalToDeviceUnits and ScaleBitmapLogicalToDevice
- Changes for making the ConnectForm and MainForm work with high DPI, by fixing how the height of some controls are calculated, such as HeaderPanel or HandleTopicControl.
- Whitespace formatting in the FilterForm.
- Fixes for the message dialogs ChangeStatusForm, ClipboardForm and DeleteForm, so the message text is displayed completely.
- Changes in the ParameterForm and the MainForm to display the ParameterForm properly in high DPI.
- Changes in the NewVersionAvailableForm, for showing the labels correctly in high DPI, besides removing some event handlers no longer called by the form.
- Small improvement in the GitHubReleaseProvider class, to have one static readonly HttpClient instance created.
- Fixing the height of the header box in the Grouper control, besides replacing tabs with spaces.
- Addressing some warnings in the Common project related to the app.config mapping and binding redirects.
…ying high DPI support.
…anifest file instead.
…xing how the height of some controls are calculated, such as HeaderPanel or HandleTopicControl. Updating to .NET Framework 4.7.2 to take advantage of some new helper methods the framework offers for high DPI related calculations.
…the message text is displayed completely. Changes in the ParameterForm and the MainForm to display the ParameterForm properly in high dpi.
…tly in high DPI. Small improvement in the GitHubReleaseProvider, to have one HttpClient instance created.
Nice. I only took a brief look, but it looks good to me. I don't think upgrading to .NET Framework 4.7.2 is a problem. We'll release it in a new major version once this PR is merged. |
+1 for gradually releasing. Less chances of something going awfully bad. +1 for .NET 4.7.2 bump. I wish we'd go with the last version, 4.8.x but older and still supported Server OS don't support it IIRC. Thank you, @jmoralesv |
@ErikMogensen @sean-feldman-sh thank you! In that case, I will move this pull request as ready for review, do we expect more approvals? How is the merge process? On the other hand, .NET Framework 4.7.2. is still supported and doesn't have an end date yet, so it is perfectly fine to use it. For .NET Framework 4.8, it looks like for Windows Server 2019 it has to be installed (4.7.2 is included by default); whereas Windows Server 2022 already includes 4.8. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great and thanks for the cleanups!
I see that Windows Server 2012 supports .NET 4.8. @SeanFeldman, is there a reason besides using the latest version that you wish we' use 4.8.x? |
.NET 4.8.x is the last version that Microsoft will release and maintain for the full framework. That's why. |
Thanks @ErikMogensen @SeanFeldman for your approvals! Glad to see this PR merged 😄 I'll publish a second one with more forms and code updated these days. |
.NET 4.8 is supported on Windows Server 2016. However, that article mentions that it is not supported on some older Windows 10 versions so let's stick to 4.7.2 for now. |
Also, as mentioned, some Windows versions require an installation to get to .NET 4.8 which may be a problem. |
Windows 2012/R2 what I meant, not 2016. |
Not that it really matters but .NET 4.8 is supported on Windows Server 2012 and on Windows Server 2012 R2. |
Great work @jmoralesv! However when running the current main branch, which contains this PR I see that the tree view has gotten more compressed as shown in the figure below. The left one is running on an older commit. It may be my personal view, but I liked the old one better. What do you think? Also, the text is a bit blurry on the right side. This is on a screen with 100% Scale. On a screen using 150% Scale the text looks good. I am not sure if it is the Scale setting though. Do you know what causes this? |
Hi @ErikMogensen that's correct, I can see a similar situation in Windows 10: I have at least one commit that can resolve this in my original commits, in a separate branch, which I can cherry pick from and add it to this PR: #800 I will review my original commits and see if that commit and others can be added, but it will increase the number of changes in PR 800. |
Excellent! Does this include the fixing the blurriness on some screens? |
@ErikMogensen that's something I can't really see in the machine I have for testing, could you please share more details about the screen you're using to test with? |
When changing the scaling to 150% on the same computer then it is the other way around. |