-
Notifications
You must be signed in to change notification settings - Fork 446
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
Dev/renderer neutral #220
Dev/renderer neutral #220
Conversation
felixse
commented
Mar 5, 2019
- Improved architecture to support different renderers
- both input and output buffer is now kept in utf8 for the most part, less conversions that might delay
- possibly fixed a race condition, could be Not working properly #206 and similar
I'm not entirely sure how it relates to the changes in this PR (quite a lot has changed so it's hard to pinpoint for me 😅) but with this PR I'm experiencing a few problems that make Fluent Terminal almost unusable.
That being said, I think this may have actually solved #206 🎉 (I occasionally had the problem described there, but relaunching usually solved it) |
Cannot reproduce this. Maybe the shell process is already dead at this point?
I'll have a look at these, thanks for testing 👍
I think it's still alive. But we are in a better position to finally catch this now. Next step is to finally add some logging 😅 |
I got one more problem (found this as part of #225), the InputDialog is not responding at all. Typing something just inputs it into the shell, and clicking the buttons does nothing.
You mean as I described in my second point? Not so, other than those UI elements everything works just fine. Maybe the .gif below helps explain my point.
Too bad... 😕 |
@ericcornelissen can you checkout the latest master? I found an issue with the websocket sometimes being called to early which brought us in some funny states. Can you have a look if your issues are still occuring? #216 could be really solved now. |
@felixse the input dialog from #216 does indeed work, although I did not test that one before, I was referring to the dialog for changing the tab name. The And lastly, the UI elements in MainView not responding properly (as seen in the .gif above) is still a problem. |
Never had this UI not responding issue on my side. What version of Windows are you experiencing this with? And you can reproduce #159 after I fixed it? 😟 |
The above comment is based on 4e83e66 and my Windows version is 1803 OS Build 17134.590 (home) |
Alright, gonna try this out tomorrow on a 1803 VM. Did you see anything interesting in the new log files? |
Not really (now running on 6b64d39), this is the logs from the point that I close the second terminal onwards:
Also, I'm not 100% sure on how to reproduce this but if Fluent Terminal crashes when I open from the system tray icon it always outputs the error below. It seems this crash happens when I first reproduce #159 (open new window, close it, have tabs in original window closed), then close the Fluent Terminal window, and finally try to launch from the system tray.
|
I can reproduce this on 1803. Must be something about focusing the WebView. Both the edit title dialog and the searchbox are instantly unfocused and it returns to the webview |
Ok, I removed something that looked reasonable but was in fact unnecessary and caused issues on Windows 10 1803. Can you check if it is behaving properly again on your side? |
Solved the problem on my end as well 👍 Thanks! The #159 problem remains though... |
Hmm 🤔 From my latest testing it seems you are right, more generally whenever the second windows has more than one tab open when it is closed, the tabs (regardless of how many) in the first window are closed but the window stays open. |
I think I fixed it. But don't ask me how exactly 😦 |
Doesn't seem to be the case on my end 🤔 Note that I just updated Windows 1809 (OS build 17763.379). Here is clean log of a run that caused the problem. In this case I had just one tab, opened a new window, closed it with only a single tab, and the tab in the first window also closed; but I also tried it with multiple tabs and the same happened.
|
I did some debugging and it seems that the fix you added in 4e83e66 is not working properly, |
Yeah, noticed it's not completely fixed. Occurs to me in around 1 of 10 tries. Does it happen to you all the time? I have no idea whats going on, we seem to end up on the wrong ui thread somehow sometimes |
For me it happens consistently as long as the number of tabs in the second window |
No problem, thanks for helping me with this 😄 |
Hmm 😕 Now the tabs closing problem is no longer consistent as you described two comments ago. However, now I have the following problem...: start Fluent Terminal, open new window, close the new window (everything is fine), open new window, close the new window. both windows close. Here is a clean log for that
|
I know I keep saying this, but I think I found the issue now 😌 Can you give it another try? |
Seems like it, good job! 😃 |