Fix #1004

Merged
ZS
3mo ago
+4-0

What does this PR aim to accomplish?:

Fix #1004

How does this PR accomplish the above?:

Add height style for #CompareElement .monaco-merge-host in L4973-L4975


By submitting this pull request, I confirm the following:

  1. I have read and understood the contributor's guide, as well as this entire template. I understand which branch to base my commits and Pull Requests against.
  2. I have commented on my proposed changes within the code.
  3. I have tested my changes.
  4. I am willing to help maintain this change if there are issues with it later.
  5. It is compatible with the GNU General Public License v3.0
  6. I have squashed any insignificant commits. (git rebase)
  7. I have checked that another pull request for this purpose does not exist.
  8. I have considered and confirmed that this submission will be valuable to others.
  9. I accept that this submission may not be used, and the pull request can be closed at the will of the maintainer.
  10. I give this submission freely and claim no ownership to its content.
  11. I have verified that my changes work correctly in both the new UI and the old/classic UI.

  • I have read the above and my PR is ready for review. Check this box to confirm

Summary by Sourcery

Enhancements:

  • Inject a style override so that the #CompareElement .monaco-merge-host element uses a fixed relative height within the compare view.

Summary by cubic

Sets a fixed height for the Monaco merge editor in the compare view to prevent clipping and empty space, fixing #1004. Injects a style that sets #CompareElement .monaco-merge-host { height: 95%; }.