Bug 581025 - [Git History] The diff view do sometimes not display the diff content
Summary: [Git History] The diff view do sometimes not display the diff content
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: 6.4   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-11-10 04:40 EST by Wolfgang Schramm CLA
Modified: 2022-11-11 03:26 EST (History)
2 users (show)

See Also:

Attachments
Diff content in diff view (102.21 KB, video/mp4)
2022-11-10 04:40 EST, Wolfgang Schramm CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Wolfgang Schramm CLA 2022-11-10 04:40:53 EST 
Created attachment 288827 [details]
Diff content in diff view

When a file is selected in the commit history, then sometimes the diff view do not show the diff content, only a one liner. After moving the vertical splitter (UI is resized), then the content is also displayed
Comment 1 Eclipse Genie CLA 2022-11-10 15:08:28 EST 
New Gerrit change created: https://git.eclipse.org/r/c/egit/egit/+/196869
Comment 2 Thomas Wolf CLA 2022-11-10 15:12:49 EST 
Yes, I see that in one of my Eclipses, too. Have been looking for a cause a few times already, to no avail. But your screengrab with the large history view finally set me on the right track.

If the text to be shown is small enough to fit the existing view, no layout of the text viewers occurred, so sometimes they were too small. As a result one might not see the whole commit message; often only the first line with the commit ID.

The manual resize then triggers a re-layout, after which the texts are fully visible.

(In reply to Eclipse Genie from comment #1)
> New Gerrit change created: https://git.eclipse.org/r/c/egit/egit/+/196869

This should fix this problem.
Comment 4 Wolfgang Schramm CLA 2022-11-11 01:08:49 EST 
(In reply to Thomas Wolf from comment #2)

> The manual resize then triggers a re-layout, after which the texts are fully
> visible.
> 

I've seen this problem since a very long time, but yesterday I discovered the "resize-fix"
Comment 5 Matthias Sohn CLA 2022-11-11 03:26:13 EST 
you can install the fixed version from the nightly build's update site
https://download.eclipse.org/egit/updates-nightly/