Skip to content

Fix deploy manifest being cut off#636

Open
tejhan wants to merge 1 commit into
Azure:mainfrom
tejhan:deployManifestCutoff
Open

Fix deploy manifest being cut off#636
tejhan wants to merge 1 commit into
Azure:mainfrom
tejhan:deployManifestCutoff

Conversation

@tejhan
Copy link
Copy Markdown
Collaborator

@tejhan tejhan commented May 4, 2026

Description

The manifest editor on the Deploy step was hardcoded to 60vh, larger than the 55vh component it resided in, resulting in final content being cut off.

Switched to flex layout & modified height to ensure editor content is fully expanded & contained within the view.

Fixes issue:

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Code refactoring
  • CI/CD changes
  • Other: **___**

Related Issues

Closes #625

Changes Made

  • Changed MonacoEditor height from 60vh -> 100%
  • Added flex:1, minHeight:0 & display:flex properties to parent Box

Testing

  • Unit tests pass
  • Integration tests pass
  • Manual testing completed
  • Performance tested (if applicable)
  • Accessibility tested (if applicable)

Test Cases

Manual Testing:

  1. Go through the deployment wizard, until the final step.
  2. Observe the deployment manifest is fully visible.

Screenshots/Videos

If applicable, add screenshots or videos to demonstrate the changes.

Loading
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.

[BUG] Deployment manifest cut off

2 participants