-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Right panel shows old info in case all is deleted in the left-hand tree #3293
Comments
The problem seems to be that the last tab is closed. This should not be the case. Previously the last tab was uncloseable. |
By previously you mean pre- #3274? |
added video show vid-.2024-09-29.10-57-52-579.mp4 |
Yes, our latest official release shows the following, if the assembly list is empty: There is a tab "New tab" with no content. From @greenozon's first screenshot, we can see that in the current master, there can be zero open tabs, which is not desired, because ILSpy is essentially an SDI application, with the possibility to view multiple documents. if necessary. |
@tom-englert I can still reproduce this using the latest master (ILSpy version 9.0.0.7802-preview2+7f46aab34493d01edf9801df6b86fc14d5c8ae62):
I also noticed that switching the current assembly list while a decompilation is in progress, it's not cancelled. When switching tree nodes the old decompilation is cancelled. I would expect the same to happen, when switching the assembly list mid-progress. I will try to look into this, but ever since the refactoring I am no longer sure how the code is supposed to work/be organized, so actually fixing stuff is a lot harder. The code is organized better, but I still have to get used to it. |
@siegfriedpammer I'll have a look, too |
Steps to reproduce
Error message shown
eg:
Details
.NET version 8.0.8+08338fcaa5c9b9a8190abb99222fed12aaba956c
The text was updated successfully, but these errors were encountered: