Bug 533555 - Double click on a file in staged changes will make the other files disappear
Summary: Double click on a file in staged changes will make the other files disappear
Status: NEW
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 4.9   Edit
Hardware: PC Windows 10
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 540383 540726 543970 (view as bug list)
Depends on: 376936 458015
Blocks:
  Show dependency tree
 
Reported: 2018-04-13 12:03 EDT by chidveer chinthakuntla CLA
Modified: 2021-06-13 08:02 EDT (History)
5 users (show)

See Also:


Attachments
screenshot for double click on a file in staged changes (55.78 KB, image/png)
2018-04-13 12:06 EDT, chidveer chinthakuntla CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description chidveer chinthakuntla CLA 2018-04-13 12:03:59 EDT
Eclipse Dark Mode ->
Go to -> git staging ->click on double click on any file in the staged changes list -> the files will disappear.
If you change the focus or Hover yours mouse on the hidden files - they will re-appear.

-- Configuration Details --
Product: Eclipse 4.7.3.20180405-1200 (org.eclipse.epp.package.jee.product)Installed Features:
 org.eclipse.egit 4.9.2.201712150930-r
Comment 1 chidveer chinthakuntla CLA 2018-04-13 12:06:43 EDT
Created attachment 273597 [details]
screenshot for double click on a file in staged changes
Comment 2 Thomas Wolf CLA 2018-04-13 14:26:38 EDT
Most likely a duplicate of or related to bug 458015, which is a bug in the Eclipse platform, not in EGit.
Comment 3 chidveer chinthakuntla CLA 2018-04-18 20:36:23 EDT
Comment on attachment 273597 [details]
screenshot for double click on a file in staged changes

Will Replace.
Comment 4 Thomas Wolf CLA 2018-05-31 03:40:54 EDT
This is a really annoying bug. I can reproduce it on Windows easily enough:

1. Have more than one unstaged changes, staging view open.
2. Select one file in the unstaged viewer.
3. Select a file in the package explorer -> staging view loses focus, select
   file in unstaged viewer gets gray (inactive selection) background.
4. Click again on the same file in the unstaged viewer: it gets the blue
   (active selection) background, but all other files and the viewer's border
   disappear. The files re-appear when moused over.

Happens only on Windows.
Comment 5 chidveer chinthakuntla CLA 2018-06-09 09:05:37 EDT
Hmm
I wana try a fix that i posted in this link
https://bugs.eclipse.org/bugs/show_bug.cgi?id=428697#c24

Looking for direction, u got any ?
Comment 6 Thomas Wolf CLA 2018-06-09 13:53:14 EDT
(In reply to chidveer chinthakuntla from comment #5)
> Hmm
> I wana try a fix that i posted in this link
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=428697#c24
> 
> Looking for direction, u got any ?

Not if you think it's a CSS problem. I don't know that part of Eclipse at all.
Comment 7 Thomas Wolf CLA 2018-10-23 05:47:06 EDT
*** Bug 540383 has been marked as a duplicate of this bug. ***
Comment 8 Thomas Wolf CLA 2018-11-02 10:45:24 EDT
*** Bug 540726 has been marked as a duplicate of this bug. ***
Comment 9 Eclipse Genie CLA 2018-11-03 07:09:10 EDT
New Gerrit change created: https://git.eclipse.org/r/131894
Comment 10 Andrey Loskutov CLA 2018-11-03 07:42:51 EDT
(In reply to Thomas Wolf from comment #4)
> This is a really annoying bug. I can reproduce it on Windows easily enough:
> 
> 1. Have more than one unstaged changes, staging view open.
> 2. Select one file in the unstaged viewer.
> 3. Select a file in the package explorer -> staging view loses focus, select
>    file in unstaged viewer gets gray (inactive selection) background.
> 4. Click again on the same file in the unstaged viewer: it gets the blue
>    (active selection) background, but all other files and the viewer's border
>    disappear. The files re-appear when moused over.
> 
> Happens only on Windows.

Doesn't work for me, I mean: I don't see the bug at all, neither with the steps above nor with any other work in the Staging view, I have *never* seen this actually on Windows. I'm on 4.10 recent nightly build and up-to-date Windows 10 installation.

However, I've switched CSS Themes in Eclipse off long time ago, and I see in the screenshots that they are used.

(In reply to Eclipse Genie from comment #9)
> New Gerrit change created: https://git.eclipse.org/r/131894

Michael, I assume the patch fixes the problem described above? Or it can be reproduced differently too?

@All on CC: which Eclipse / Windows are you running? All have Eclipse "Themes" enabled?
Comment 11 Andrey Loskutov CLA 2018-11-03 07:51:17 EDT
(In reply to Andrey Loskutov from comment #10)
> However, I've switched CSS Themes in Eclipse off long time ago

Switched it on again and I see this issue! OMG. Just learned another reason to avoid this mess.
Comment 12 Thomas Wolf CLA 2018-11-03 07:58:22 EDT
(In reply to Andrey Loskutov from comment #11)
> (In reply to Andrey Loskutov from comment #10)
> > However, I've switched CSS Themes in Eclipse off long time ago
> 
> Switched it on again and I see this issue! OMG. Just learned another reason
> to avoid this mess.

That points to a either a bug in e4 themeing, or some CSS problem?

So we could also try instead of redrawing to call viewer.getControl.reskin(SWT.ALL);

Even if the bug gets found and fixed, we'd still have to have some work-around for older Eclipses.
Comment 13 Andrey Loskutov CLA 2018-11-03 08:04:38 EDT
(In reply to Thomas Wolf from comment #12)
> (In reply to Andrey Loskutov from comment #11)
> > (In reply to Andrey Loskutov from comment #10)
> > > However, I've switched CSS Themes in Eclipse off long time ago
> > 
> > Switched it on again and I see this issue! OMG. Just learned another reason
> > to avoid this mess.
> 
> That points to a either a bug in e4 themeing, or some CSS problem?

I'm totally ignoring all e4 themeing issues as the only solution to fix it is to disable it. It is such an UI overhead on top of the "usual" UI overhead, plus extra bugs.. No, sorry, no idea how deep it is in e4 theming.
 
> Even if the bug gets found and fixed, we'd still have to have some
> work-around for older Eclipses.

This is true. I wondering however that so far I haven't heard (listening on platform UI) about recent similar issues outside of EGit staging view. And yes, this bug would drive me mad too. So could it be our staging view has a bug, breaking (or pausing) somehow rendering of the tree during "set focus/set selection"?
Comment 14 Thomas Wolf CLA 2018-11-03 08:26:54 EDT
(In reply to Andrey Loskutov from comment #13)
> This is true. I wondering however that so far I haven't heard (listening on
> platform UI) about recent similar issues outside of EGit staging view. And
> yes, this bug would drive me mad too. So could it be our staging view has a
> bug, breaking (or pausing) somehow rendering of the tree during "set
> focus/set selection"?

Could be, of course. As I wrote in the duplicates, I have tried debugging this several times, but got nowhere.

But compare the linked bug in platform: bug 458015, and then 435377 (reporting this very issue back in 2014!), and the supposedly fixed bug 433858.
Comment 15 Andrey Loskutov CLA 2018-11-03 19:38:24 EDT
(In reply to Thomas Wolf from comment #14)
> the supposedly fixed bug 433858.

Now reading this, for me this is still bug 433858. Just switched back to no-themes and enjoy again flicker free experience.
Comment 16 Eclipse Genie CLA 2018-11-04 10:47:17 EST
Gerrit change https://git.eclipse.org/r/131894 was merged to [master].
Commit: http://git.eclipse.org/c/egit/egit.git/commit/?id=98fd2ee606047d72e0a9d9263f23bdafcba90e7d
Comment 17 Lars Vogel CLA 2018-11-06 03:36:18 EST
(In reply to Eclipse Genie from comment #16)
> Gerrit change https://git.eclipse.org/r/131894 was merged to [master].
> Commit:
> http://git.eclipse.org/c/egit/egit.git/commit/?id=98fd2ee606047d72e0a9d9263f23bdafcba90e7d
> 

I still see "empty entries" during DnD to the "Staged Changes" part in the staging view.
Comment 18 Thomas Wolf CLA 2018-11-06 10:49:14 EST
(In reply to Lars Vogel from comment #17)
> (In reply to Eclipse Genie from comment #16)
> > Gerrit change https://git.eclipse.org/r/131894 was merged to [master].
> > Commit:
> > http://git.eclipse.org/c/egit/egit.git/commit/?id=98fd2ee606047d72e0a9d9263f23bdafcba90e7d
> > 
> 
> I still see "empty entries" during DnD to the "Staged Changes" part in the
> staging view.

@Lars, please see comment 12 to comment 15. Looks like this is some e4 themeing bug. It is highly doubtful we can do much about in EGit.
Comment 19 Thomas Wolf CLA 2019-01-30 06:51:57 EST
*** Bug 543970 has been marked as a duplicate of this bug. ***