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

[BUG] - Tricky problem about focus #40

Closed
tramhao opened this issue Jul 20, 2022 · 1 comment · Fixed by #41
Closed

[BUG] - Tricky problem about focus #40

tramhao opened this issue Jul 20, 2022 · 1 comment · Fixed by #41
Assignees
Labels
bug Something isn't working

Comments

@tramhao
Copy link
Contributor

tramhao commented Jul 20, 2022

I found a trick problem about focus as below:

  1. I open a popup named tageditor from treeview.
  2. In the popup, I do some operation, that will lead to a reload of treeview. And because of the reload, treeview is no longer in focus stack.
  3. After I close the tageditor popup, I don't have focus.

Currently I'm focusing treeview manually. But it could get worse. For example, from the popup, there could be error popup mounted. The error message could be showing before or after closing tageditor popup. And now it's very hard to know where to focus.

Is it possible that a remount will not change focus stack?

@tramhao tramhao added the question Further information is requested label Jul 20, 2022
@veeso veeso added the bug Something isn't working label Jul 20, 2022
@veeso veeso removed the question Further information is requested label Aug 3, 2022
@veeso veeso changed the title [QUESTION] - Tricky problem about focus [BUG] - Tricky problem about focus Aug 3, 2022
@veeso veeso linked a pull request Aug 3, 2022 that will close this issue
14 tasks
@veeso veeso closed this as completed in #41 Aug 3, 2022
@veeso
Copy link
Owner

veeso commented Aug 3, 2022

Fixed in 1.7.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants