-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[prebuild] Fix view when coming from prefix #11119
Conversation
started the job as gitpod-build-laushinka-prebuild-11079.2 because the annotations in the pull request description changed |
f53d25e
to
899c887
Compare
899c887
to
6df428a
Compare
@@ -123,17 +123,19 @@ export default function PrebuildLogs(props: PrebuildLogsProps) { | |||
}, [logsEmitter, props.workspaceId, workspaceInstance?.id, workspaceInstance?.status.phase]); | |||
|
|||
return ( | |||
<div className="rounded-xl overflow-hidden bg-gray-100 dark:bg-gray-800 flex flex-col"> | |||
<div className="rounded-xl overflow-hidden bg-gray-100 dark:bg-gray-800 flex flex-col mb-8"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ended up adding the margin here - tested all three views. Also I think moving forward we should make the logs be centered in the screen. It's too off down there at the bottom.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and works! ✔️
Description
This fixes the broken layout, but doesn't add yet the "Skip Prebuild" button as we intended in a recent epic. Skipping prebuilds will be covered in a follow-up issue.
Related Issue(s)
Fixes #11079
How to test
https://laushinka-be81110388.preview.gitpod-dev.com/#prebuild/[YOUR_REPO]
Release Notes
Documentation
Werft options: