Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Focus always moving to the assembly tree view when changing tabs #3284

Closed
siegfriedpammer opened this issue Sep 22, 2024 · 2 comments · Fixed by #3285
Closed

Focus always moving to the assembly tree view when changing tabs #3284

siegfriedpammer opened this issue Sep 22, 2024 · 2 comments · Fixed by #3285
Labels

Comments

@siegfriedpammer
Copy link
Member

siegfriedpammer commented Sep 22, 2024

Using the latest released version the focus behavior was as shown in the following gif:

focus1

Using latest master the behavior is as follows:

focus2

I have yet to determine the change that causes this deviation. For me this is only a minor distraction, because switching between tabs and tree nodes makes the screen flicker a bit more than usual. However, as we have seen previously, there are people who depend on accessibility software to use ILSpy, and I can image that this change causes some frustration, because the focus move is unanticipated.

While creating the above screen recordings, I noticed that the label of the assembly tree view content (previously labelled Assemblies) is now labelled ILSpy <version> - <assembly list name>, which was previously in the window title, not sure if this has some negative impact on accessibility...

/cc @tom-englert

@siegfriedpammer
Copy link
Member Author

Note: looking at the recordings, it seems to me that the new behavior is that switching between tabs triggers a refresh/decompilation run.

@tom-englert
Copy link
Contributor

I'll have a look on this

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 a pull request may close this issue.

2 participants