Skip to content

Show filename in Quick Diff progress job label#3931

Merged
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:quickdiff-job-label-filename
Apr 24, 2026
Merged

Show filename in Quick Diff progress job label#3931
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:quickdiff-job-label-filename

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented Apr 24, 2026

The DocumentLineDiffer background job is currently labelled "Initializing Quick Diff" for every editor. The name is misleading because the job is not a one-time initialization (it also re-runs on every save and whenever the reference document changes), and identical labels give no hint which editor each Progress view entry belongs to.

This change passes the editor input name into the differ so the job appears as "Quick Diff: comparing " when a name is available, falling back to the generic label otherwise. DocumentLineDiffer is internal, so the new setter is not API; IEditorInput.getName() is existing API.

The DocumentLineDiffer background job is currently labelled
"Initializing Quick Diff" for every editor, which is misleading:
the job also re-runs on every save and on reference changes, and
identical labels give no hint which editor a given entry belongs to.

Use the editor input name to produce a more informative label
"Quick Diff: comparing <filename>" when available; fall back to the
generic label otherwise.
@github-actions
Copy link
Copy Markdown
Contributor

Test Results

   852 files  ±0     852 suites  ±0   56m 4s ⏱️ + 5m 14s
 7 913 tests ±0   7 670 ✅ +1  243 💤 ±0  0 ❌  - 1 
20 241 runs  ±0  19 586 ✅ +1  655 💤 ±0  0 ❌  - 1 

Results for commit 3cdf444. ± Comparison against base commit b2f2d9b.

@vogella vogella merged commit b39314c into eclipse-platform:master Apr 24, 2026
18 checks passed
@vogella vogella deleted the quickdiff-job-label-filename branch April 24, 2026 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant