Skip to content

Second set of changes to enable high DPI in Service Bus Explorer#800

Draft
jmoralesv wants to merge 10 commits intopaolosalvatori:mainfrom
jmoralesv:second-set-changes
Draft

Second set of changes to enable high DPI in Service Bus Explorer#800
jmoralesv wants to merge 10 commits intopaolosalvatori:mainfrom
jmoralesv:second-set-changes

Conversation

@jmoralesv
Copy link
Contributor

This pull request continues the work started in pull-request: #797 by using TableLayoutPanels across several forms and user controls, to improve the rendering and resizing of the inner controls in a high DPI monitor, so there is no need for custom paint and resize logic in some cases.

  • Adding a TableLayoutPanel in the TimeSpanControl, to improve the scaling of the controls inside, so they can be displayed better. This also allows the control to have a bigger size in a container, and the controls can expand their size according to the container size.
  • Updating the HandleTopicControl to use the updated TimeSpanControl.
  • Changes for the layout, sizes and position of the controls in the HandleTopicControl, HandleSubscriptionControl and HandleRuleControl. The last one has some temporary changes, because the resize of the controls is messy without a TableLayoutPanel.
  • Adding a TableLayoutPanel to the HandleRuleControl, to improve the scaling and resizing of the child controls.
  • Changes for showing the grouperCorrelationFilter control in the HandleRuleControl.
  • Adding a TableLayoutPanel for the grouperCorrelationFilter control in the HandleRuleControl.
  • Aligning the layout of the rest of Handle controls (HandleConsumerGroup, HandleEventHub, HandleNotificationHub, HandleQueue, HandleRelay, HandleSubscription, HandleTopic), so the groupers and others are in similar positions.
  • Adding calls to SuspendDrawing, SuspendLayout, ResumeDrawing and ResumeLayout when rendering the groupers in the HandleRuleControl.
  • Changes for rendering the headers of the list views in the Handle controls, including HandlePartition, ListenerControl, PartitionListenerControl, TestEventHub, TestQueue, TestRelay, TestSubscription, and TestTopic, by calling e.DrawText.
  • Changes in the HandleNotificationHubControl, to use a table layout panel to improve the layout when rendering the form in a high DPI monitor. Because of this, the HandleNotificationHubControl_Resize method is no longer needed.
  • Removing the HandleRuleControl_Resize method in the HandleRuleControl.

…ing of the controls inside, so they can be displayed better. This also allows the control to have a bigger size in a container, and the controls can expand their size according to the container size. Updating the HandleTopicControl to use the updated TimeSpanControl.
…dleTopicControl, HandleSubscriptionControl and HandleRuleControl. The last one has some temporary changes, because the resize of the controls is messy without a TableLayoutPanel.
…aling and resizing of the child controls. There is code pending for the CorrelationFilter grouper.
…nel to improve the layout when rendering the form in a high DPI monitor.
@jmoralesv jmoralesv changed the title First set changes to enable high DPI in Service Bus Explorer Second set of changes to enable high DPI in Service Bus Explorer Aug 5, 2024
@ErikMogensen
Copy link
Collaborator

I want to wait with this PR until we have a solution for the blurriness for text in some scenarios as mentioned in #797.

@stale stale bot added the wontfix label Jan 31, 2025
Repository owner deleted a comment from stale bot Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants