Skip to content

Fixed scrolling inside the XMI tab of the E4WorkbenchModelEditor#2321

Merged
merks merged 1 commit intoeclipse-pde:masterfrom
r-mennig:fixed_xmi_tab_scrolling
Apr 28, 2026
Merged

Fixed scrolling inside the XMI tab of the E4WorkbenchModelEditor#2321
merks merged 1 commit intoeclipse-pde:masterfrom
r-mennig:fixed_xmi_tab_scrolling

Conversation

@r-mennig
Copy link
Copy Markdown
Contributor

Fixes #2299

This PR removes the disablement of the source viewer in the XMI tab of the E4WorkbenchModelEditor. This allows scrolling inside the viewer while still keeping the contents unmodifiable.

This also enables copying of the XMI contents which might be useful e.g. for copying elementIds or other parts.

There's currently no test plugin for the XMI tab, but if #2320 gets merged, I can create a follow-up PR adding tests for the scrollbar too.

@github-actions
Copy link
Copy Markdown

Test Results

  126 files  ±0    126 suites  ±0   34m 0s ⏱️ - 3m 20s
3 502 tests ±0  3 406 ✅  - 42   54 💤 ±0  42 ❌ +42 
9 321 runs  ±0  9 148 ✅  - 43  130 💤 ±0  43 ❌ +43 

For more details on these failures, see this check.

Results for commit 761cfeb. ± Comparison against base commit 5e3ba32.

@merks
Copy link
Copy Markdown
Contributor

merks commented Apr 28, 2026

I think the warning is this one which is an infrastructure issue not related to the PR:

image

Copy link
Copy Markdown
Contributor

@merks merks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice to use ModelEditorPreferences.TAB_FORM_SEARCH_SHOW instead of a copy of the string literal!

@merks
Copy link
Copy Markdown
Contributor

merks commented Apr 28, 2026

I restarted two of the jobs to get more green verifications.

@merks merks merged commit 71658f5 into eclipse-pde:master Apr 28, 2026
22 of 24 checks passed
@merks
Copy link
Copy Markdown
Contributor

merks commented Apr 28, 2026

Thanks @r-mennig ❣️

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.

Scrolling not possible in XMI tab of E4WorkbenchModelEditor

2 participants