Skip to content

Conversation

AndisGrossteins
Copy link
Contributor

PR Classification

Code cleanup and modernization. Should fix #476.

PR Summary

This pull request refactors and modernizes the codebase for improved readability, maintainability, and robustness, while enhancing logging and UI consistency.

  • RegexHelperDialog.cs: Replaced comboBox item management with encapsulated history lists for better data handling.
  • SearchDialog.Designer.cs: Refactored UI code to remove redundant this references, modernized control properties, and streamlined form setup. (mostly automatic shanges by Visual Studio designer mode)
  • Program.cs: Improved exception logging to log exception instead of event args.
  • PluginRegistry.cs: Moved logging into if statement to reduce erroneous log messages.

Set SearchDialog.buttonRegexHelper.Autosize
Insert new items into comboBox dataSource - _expressionHistoryList and _testtextHistoryList instead of directly into comboBox.Items.
Moved _logger.Info call into if statement where it should be
Replaced the `Button` control `buttonRegexHelper` with a `LinkLabel` control `linkLabelRegexHelper` to improve the UI for the search dialog. Updated `TabIndex` values across multiple controls to ensure proper tab navigation order. Adjusted properties of `linkLabelRegexHelper` for accessibility and alignment with the new design.

Modified `groupBoxOptions` and `groupBoxDirection` controls to reflect the new layout, including size and `TabIndex` adjustments. Removed the declaration of `buttonRegexHelper` and added `linkLabelRegexHelper` in the private fields section.
…UnhandledExceptionEventArgs"

This reverts commit 79d59d9.
@AndisGrossteins AndisGrossteins marked this pull request as ready for review October 10, 2025 23:15
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.

ArgumentException in RegexHelperDialog

2 participants