Go to definition, but with Ctrl + Click

[QuangNguyenMinh123]

As the title suggests

[epasveer]

Just to mention, Ctrl+Click is open for use. I just want to make you aware that Ctrl+DoubleClick is taken already by the source editor widgets.

Keys
----
The easiest method for logging a variable to the Variable Logger is to double-click the variable text in the source. When doing this, a key modifier (shift and/or ctrl) can be used to prepend a '*' and/or '&' to the variable. This will better handle pointers and references.
      Double-Click
      Modifier      Description                           Example
      ------------  ------------------------------------  -----------------------
      None          No modifier is prepended to variable  'argc'
      Shift Key     Prepend '&' to variable               '&argc'
      Ctrl Key      Prepend '*' to variable               '*argc'
      Ctrl+Shift    Prepend '*&' to variable              '*&argc'

[QuangNguyenMinh123]

this should be merged after #402

[epasveer]

@QuangNguyenMinh123

After merging #402 , it looks like this MR has conflicts.

[QuangNguyenMinh123]

@QuangNguyenMinh123

After merging #402 , it looks like this MR has conflicts.

I'll check it

[QuangNguyenMinh123]

@epasveer
Please give this MR a try.
It might work, but it still needs a lot of improvement

[epasveer]

Thanks. Will do.

[QuangNguyenMinh123]

I fixed the issue, please review

[epasveer]

Looks good and works. I've updated the help screen for the SeerEditorManagerWidget.

If there's nothing else, I think I can pull it.

[QuangNguyenMinh123]

That's all

[epasveer]

@QuangNguyenMinh123

Just to mention, Ctrl+Click is open for use. I just want to make you aware that Ctrl+DoubleClick is taken already by the source editor widgets.

I may have merged it too soon.

It appears your use of Ctrl+Click breaks the feature of adding variables to the Variable logger using Double Ctrl+Click.

Can we change your feature to use Alt+Click instead?

[QuangNguyenMinh123]

change your feature to use Alt+Click instead

Disagree. Alt+Click seems counterintuitive, I don't think users will love it
BTW, I see this on CHANGELOG.md:

## [2.3] - 2023-11-19
* In the margins of the source windows, allow CTRL+DoubleClick to do a quick RunToLine or RunToAddress.

I tried using this feature, but it only works if I Ctrl + double-click the grey area, right? Could you please confirm this?

If so, Ctrl + Click (Go to Definition) doesn't affect Ctrl + Double-click, because Ctrl + Click is only applied to SeerEditorWidgetSourceArea::mousePressEvent, while Ctrl + double-click is only applied to SeerEditorWidgetSourceLineNumberArea::mouseDoubleClickEvent