Code Inspections not highlighting code on double click

[daFreeMan]

Rubberduck version information

Version 2.4.1.5182
OS: Microsoft Windows NT 10.0.17134.0, x64
Host Product: Microsoft Office 2016 x64
Host Version: 16.0.4912.1000
Host Executable: EXCEL.EXE

Description
Double clicking an inspection in the Code Inspections window causes a very brief flicker of highlight, but leaves the offending code unhighlighted.

To Reproduce
Steps to reproduce the behavior:

1. Run a parse
2. Click on a code inspection
3. '....'
4. Watch it flicker

Expected behavior
This is a regression somewhere along the line. Double-clicking an inspection should leave the impacted code highlighted.

Screenshots
This was recorded in build .5170, but I did test & duplicate in .5182 - I just didn't re-record it.

Logfile
RubberduckLog.txt

Additional context
Loads of inspections in this project... Just confirmed the same behavior in Access. Wasn't there last time I looked at inspections, but that has been a... while...

Additional, additional context
This happens the vast majority of the time, but it's not a 100% failure rate. On occasion, it will leave the code highlighted. It's probably a 95+% failure rate, though.

[IvenBach]

@daFreeMan Is it just flickering because the Code Inspections window is getting focus and not the IDE? I had similar thoughts till I realized which had focus. Clicking the title bar of the IDE to give it focus shows that the correct inspection is selected.

[daFreeMan]

Yup, you are correct - when I click the IDE title bar, the code does appear highlighted. However, this is new behavior. Previous versions (not sure how previous) would show the code highlighted whether I manually clicked the VBE or not. I suppose they left the VBE as the focused window instead of the CI tool window (which I do not use docked).

[retailcoder]

That's a VBIDE quirk, nothing appears to be selected unless the code pane has focus. Presumably the behavior returns to normal when the toolwindow is docked? Confirmed: docking the toolwindow fixes the focusing issue.

I don't think this is something we can fix.

[daFreeMan]

I'd think that is fixable, since it seems to have been broken (recently-ish). This is not something I'd ever had issue with before. I tried using the CI window docked once, hated it and it's been undocked ever since. I know I've used the CI in this fashion before - many times before, and the code's always been highlighted because the VBE has gotten the focus from the tool window after the double-click. I'm not sure how or why it was broken, but it used to work.

I'll be happy to go back through my collection of pre-release builds and install & test until I find one that works, if that would help.

[bclothier]

Given that it recently got a horizontal scrollbar, it would be easy to download the version before that merge and see if it behaves or not.

[IvenBach]

I’ve noticed this behavior for quite some time. Before the scroll bars for certain.

[daFreeMan]

Build .4993: Works
Build .5033: Works
Build .5081 (got the "new pre-release available" pop-up, that surprised me!): Works
Build .5094: Works
Build .5096: Works
Build .5096: Works
Build .5097: Works (and found new issues Yay! or is that Boo... ?)
Build .5105: Works (and I went back to the previously grouped list of issues?)
Build .5115: Works
Build .5130: Works (that "new pre-release" pop up is gettin' kinda naggy at this point... :/)
Build .5131: Works
Build .5132: BROKEN
Build .5170: BORKEN
Build .5182: BORKEN
Build .5196: OBRKEN

Looks like 5132 broke things.

_{I'll investigate the 5096->5097->5105 changes and report as a separate issue if necessary}

[MDoerner]

Probably, the following event handler is to blame.
https://github.com/glowingrunes/Rubberduck/blob/302122d512cbfd7e1f4491f77ddda1cba9f3f038/Rubberduck.Core/UI/Inspections/InspectionResultsControl.xaml.cs#L15

It was introduces to avoid that the event reaches the scroll viewer. However, that seems to also have swallowed the focus change.

[Tanarri]

Hi,

for me the double click doesn't work for several versions. Today I tried the lastes version and still the double click in Code Inspections doesn't work.
Can I help you with further information?

Version 2.5.2.5936
Betriebssystem: Microsoft Windows NT 10.0.19042.0, x64
Hostprodukt: Microsoft Office 2013 x86
Hostversion: 15.0.5267.1000
Hostprogramm: MSACCESS.EXE

[Tanarri]

I have more Infos and a workaround:
a) When I double-click on a code-inspection the module will be shown and the correct module lines will be shown in which the inspection is located.
b) Then when I click "outside" the lines which are affected by the code-inspection, none codelines will be selected.
b) Then when I llooonnnggggg click "inside" the lines which are affected by the code-inspection, the correct codelines will be selected (this is what a double-click in the window "code inspection" should do)
b) Then when I short click "inside" the lines which are affected by the code-inspection, none codelines will be selected.

This video shows what I mean

Hope it helps

Version 2.5.2.5984
Betriebssystem: Microsoft Windows NT 10.0.19043.0, x64
Hostprodukt: Microsoft Office 2013 x86
Hostversion: 15.0.5349.1000
Hostprogramm: MSACCESS.EXE