Skip to content

Fix deployment logs overlap with indicator #7580

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.

Sign up for GitHub

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

Merged
andrasbacsai merged 2 commits into from
Dec 11, 2025
Merged

Fix deployment logs overlap with indicator #7580

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
911cfa0
Fix deployment logs overlap with indicator and improve visibility
andrasbacsai Dec 11, 2025
c6a8908
Refactor deployment indicator to use server-side route detection
andrasbacsai Dec 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Refactor deployment indicator to use server-side route detection 
Replace client-side JavaScript URL checking with Laravel's routeIs() for determining when to reduce indicator opacity. This simplifies the code and uses route names as the source of truth.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
  • Loading branch information
@andrasbacsai @claude
andrasbacsai and claude committed Dec 11, 2025
commit c6a89087c52412d64679ba9c4ed4d0dee94c2948
6 changes: 6 additions & 0 deletions app/Livewire/DeploymentsIndicator.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,12 @@ public function deploymentCount()
return $this->deployments->count();
}

#[Computed]
public function shouldReduceOpacity(): bool
{
return request()->routeIs('project.application.deployment.*');
}

public function toggleExpanded()
{
$this->expanded = ! $this->expanded;
Expand Down
11 changes: 2 additions & 9 deletions resources/views/livewire/deployments-indicator.blade.php
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,17 +1,10 @@
<div wire:poll.3000ms x-data="{
expanded: @entangle('expanded'),
isDeploymentPage: false,
checkPage() {
this.isDeploymentPage = window.location.pathname.includes('/deployment');
},
init() {
this.checkPage();
document.addEventListener('livewire:navigated', () => this.checkPage());
}
reduceOpacity: @js($this->shouldReduceOpacity)
}" class="fixed bottom-0 z-60 mb-4 left-0 lg:left-56 ml-4">
@if ($this->deploymentCount > 0)
<div class="relative transition-opacity duration-200"
:class="{ 'opacity-100': expanded || !isDeploymentPage, 'opacity-60 hover:opacity-100': isDeploymentPage && !expanded }">
:class="{ 'opacity-100': expanded || !reduceOpacity, 'opacity-60 hover:opacity-100': reduceOpacity && !expanded }">
<!-- Indicator Button -->
<button @click="expanded = !expanded"
class="flex items-center gap-2 px-4 py-2 rounded-lg shadow-lg transition-all duration-200 dark:bg-coolgray-100 bg-white dark:border dark:border-coolgray-200 hover:shadow-xl">
Expand Down