#### Brief overview of PR changes/additions
Ran clang-format on all 134 CPP files in src/ using the project's
.clang-format config
#### Motivation for adding to Mudlet
Ensures consistent code formatting across the codebase.
#### Other info (issues closed, discussion etc)
None
**Test case:** Build the project and verify it compiles successfully.
<!-- Keep the title short & concise so anyone non-technical can
understand it,
the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
pre_guard.h and post_guard.h snuck back in during #8239, but they aren't
required any more as per #8378
#### Motivation for adding to Mudlet
code readability
#### Other info (issues closed, discussion etc)
<!-- Keep the title short & concise so anyone non-technical can
understand it,
the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Reduce blank pattern widget lines to only show used ones + 1. New
trigger start with two.
When you delete the last one, it reduces back to always used + 1 blank
Works good with import triggers and legacy multiple lines triggers.,
improves keyboard navigation:
modified order of tab navigation to go from name to patterns, then
continue,
add control+f to go first pattern line, and control+L last.. hint under
pattern lines
Up/down arrows to move between pattern lines
#### Motivation for adding to Mudlet
Enhanced tab navigation on the trigger editor menu , priority to useful
displays, etc.
better experiencie for screen readers users , less objects to navigate.
#### Other info (issues closed, discussion etc)
replace #8215
<img width="1860" height="681" alt="image"
src="https://github.com/user-attachments/assets/9bcc9cb4-dc52-4ff9-b4a1-0ad65dbb2e48"
/>
<img width="1825" height="825" alt="image"
src="https://github.com/user-attachments/assets/abc393f2-4ad3-4eac-9bdb-848c27dbc9e9"
/>
https://github.com/user-attachments/assets/69acf69e-15a2-4439-93da-ea98c39958c6
---------
Signed-off-by: SammerPetria <sammerpetria@gmail.com>
<!-- Keep the title short & concise so anyone non-technical can
understand it,
the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Remove support for leak detection using MSVC - this functionality hasn't
been in use in a decade, and we're switching to to using
[LeakSanitizer](https://github.com/google/sanitizers/wiki/AddressSanitizerLeakSanitizer)
instead.
#### Motivation for adding to Mudlet
Cleaner code.
#### Other info (issues closed, discussion etc)
---------
Co-authored-by: Claude <noreply@anthropic.com>
<!-- Keep the title short & concise so anyone non-technical can
understand it,
the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Fix font issues in trigger pattern editor.
#### Motivation for adding to Mudlet
Better user experience.
#### Other info (issues closed, discussion etc)
https://discord.com/channels/283581582550237184/427919962561052673/1318678483295146064
---------
Co-authored-by: Vadim Peretokin <vadi2@users.noreply.github.com>
This reverts commit c1295a8655.
<!-- Keep the title short & concise so anyone non-technical can
understand it,
the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Revert regex highlighting added in
https://github.com/Mudlet/Mudlet/pull/7523
#### Motivation for adding to Mudlet
User feedback in
https://discord.com/channels/283581582550237184/1318678483295146064 -
getting the text and line height proper on all platforms is tough
#### Other info (issues closed, discussion etc)
We'll add it back later after the release.
Co-authored-by: Vadim Peretokin <vadi2@users.noreply.github.com>
#### Brief overview of PR changes/additions
make variable name more readable
#### Motivation for adding to Mudlet
Improve readability
#### Other info (issues closed, discussion etc)
Fix#6375
Co-authored-by: Stephen Lyons <slysven@virginmedia.com>
There is also some minor cosmetic changes, mainly with members initialised
within the body of the constructors. Also some bogus uses of `nullptr`
**pointer** value instead of a `QColor()` default value for a couple of
`QColors` in the `dlgRoomSybol` class.
I have steered away from using `{}` to initialise simple, POD data types in
favour of explicitly stating what their default values are.
Signed-off-by: Stephen Lyons <slysven@virginmedia.com>
We recently removed the foreground colour coding of text entered into the
trigger item text line because it could render some types unreadable with
some themes/styling. That was not universally popular because the colour
was also a sub-conscious reminder to some users as to the item type.
This PR adds a coloured icon to the trigger type selection `QComboBox` but
- given that it is a little way from the point at which any text is entered
I have also arranged for the coloured (they could also be pictorial icons
if someone wanted to design a set) icons to be duplicated at the start
of the `QLineEdit` where the text is entered.
The colours largely duplicate the colours previously used for those with
text entry and I have chosen a pair of contrasting colours for the two
(colour and prompt) triggers that do not show the text entry widget.
This also properly wraps the trigger type texts in `tr(`...`)` so they
will finally be included in the translation process - from which they
were previously not!
Signed-off-by: Stephen Lyons <slysven@virginmedia.com>
This also allows for memory leak checking on MSVC by having include
surrounding Qt headers, which break if you have a macro define new
to be something which tracks line information.
