mirror of
https://github.com/coollabsio/coolify.git
synced 2026-09-26 01:10:30 -04:00
fix(ui): weld the deployment log sidebar to the main sidebar
Remove :flush="true" from the deployment show configuration sidebar so it stays fixed beside the main nav instead of sitting flush in the page layout.
This commit is contained in:
@@ -6,7 +6,7 @@
|
||||
<livewire:project.application.heading :application="$application" wire:key="application-heading-deployment-show" />
|
||||
<section class="application-settings-workspace mt-4 flex min-h-0 w-full max-w-none flex-1 lg:mt-0">
|
||||
<div class="grid min-h-0 min-w-0 flex-1 gap-8 xl:grid-cols-[210px_minmax(0,1fr)] xl:gap-8">
|
||||
<x-application.configuration-sidebar :application="$application" :flush="true"
|
||||
<x-application.configuration-sidebar :application="$application"
|
||||
current-route="project.application.deployment.show" />
|
||||
<div class="flex min-h-0 min-w-0 flex-col gap-4 xl:h-full xl:pt-px">
|
||||
<livewire:project.application.deployment.index :embedded="true"
|
||||
|
||||
@@ -519,14 +519,13 @@ it('builds application sidebar routes independently of the current request route
|
||||
->toContain("'application_uuid' => \$application->uuid");
|
||||
});
|
||||
|
||||
it('keeps the deployment log sidebar fixed in the layout without a top gap', function () {
|
||||
it('welds the deployment log sidebar to the main sidebar', function () {
|
||||
$deployment = file_get_contents(resource_path('views/livewire/project/application/deployment/show.blade.php'));
|
||||
$css = file_get_contents(resource_path('css/app.css'));
|
||||
|
||||
expect($deployment)->toContain(':flush="true"')
|
||||
->and($css)->toContain('.application-settings-navigation.is-flush')
|
||||
->and($css)->toContain('position: static;')
|
||||
->and($css)->toContain('overflow: visible;');
|
||||
expect($deployment)->not->toContain(':flush="true"')
|
||||
->and($css)->toContain('left: var(--sidebar-w, 14rem);')
|
||||
->and($css)->toContain('position: fixed;');
|
||||
});
|
||||
|
||||
it('uses the same mobile heading gap on deployment pages as application settings', function () {
|
||||
|
||||
Reference in New Issue
Block a user