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

Half-Hidden tab getting close while not clicking on close button #8431

Open
MikaelDesharnais-ST opened this issue Aug 25, 2020 · 1 comment
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help ui/ux issues related to user interface / user experience

Comments

@MikaelDesharnais-ST
Copy link

Bug Description:

When you are clicking at the right most part of a half-hidden tab, the tab in question gets closed (Zone highlighted in red) :
image

Steps to Reproduce:

  1. Open many files
  2. Scroll your tabs to have on tab half hidden to the right of the tabbar
  3. click on the right most part of the tab

Additional Information

This error comes from PhosphorJS which is now abandonned as discussed in this thread :
#6501
In phosphorJS tabbar when the mousedown event is received, the tab that is clicked becomes the default one, so the tabbar gets centered on this tab.
In the mouse up event, there is a check if the click was made on the close icon. This check returns true because now the tabbar was recentered on the tab clicked and so the click calls the close tab.
I checked with VSCode and in this case, the tab only gets selected : not closed

  • Operating System: Any
  • Theia Version: Latest
@vince-fugnitto vince-fugnitto added 🤔 needs more info issues that require more info from the author ui/ux issues related to user interface / user experience and removed 🤔 needs more info issues that require more info from the author labels Aug 25, 2020
@vince-fugnitto
Copy link
Member

@MikaelDesharnais-ST it really depends on where the tab is clicked, as the closing of the tab does not occur reliably (for example, it never occurs when clicking the beginning of the tab which is the more natural place). In any case, I think its a minor bug (it will likely be much better when we drop phosphor).

@akosyakov akosyakov added bug bugs found in the application help wanted issues meant to be picked up, require help labels Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help ui/ux issues related to user interface / user experience
Projects
None yet
Development

No branches or pull requests

3 participants