Skip to content

fix: restore full-width disk space bar and fix icon tooltip clipping off-screen - #44

Merged
konradmichalik merged 1 commit into
mainfrom
fix/disk-space-icon-tooltip
Sep 7, 2026
Merged

fix: restore full-width disk space bar and fix icon tooltip clipping off-screen#44
konradmichalik merged 1 commit into
mainfrom
fix/disk-space-icon-tooltip

Conversation

@konradmichalik

Copy link
Copy Markdown
Contributor

Summary

  • Restore a full-width, thin status bar at the top of the feature-branch overview page (filled proportionally to disk usage, colored by status), which the icon-only version from the previous release had dropped
  • Replace the icon's tooltip mechanism: it relied on Pico CSS's built-in data-tooltip, which always opens upward - placed a few pixels from the top of the viewport, that tooltip rendered off-screen and was never visible
  • Give the icon a more polished look: stroke-based database glyph, subtle shadow/ring, hover and keyboard-focus feedback

Changes

  • deployer/feature/dist/index/src/Service/TemplateService.php - renderDiskSpace() now renders both the full-width bar and the icon; new custom data-info attribute instead of Pico's data-tooltip; new stroke-style icon markup
  • deployer/feature/dist/index/assets/css/style.css - .disk-space-bar/.disk-space-bar-fill for the full-width bar; custom ::after-based tooltip on .disk-space-icon that always opens downward, gated animation behind prefers-reduced-motion

Test Plan

  • Deploy the feature-branch index page and confirm the full-width bar shows at the very top, filled/colored by current disk usage
  • Hover and tab-focus the icon and confirm the tooltip appears below it, fully visible, with percentage/free/total GB and the low-space warning when applicable

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • deployer/feature/dist/index/assets/css/style.css is excluded by !**/dist/**
  • deployer/feature/dist/index/src/Service/TemplateService.php is excluded by !**/dist/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 91b03440-413a-4a55-919f-ce34d72dd646

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@konradmichalik
konradmichalik merged commit 321dc0a into main Sep 7, 2026
2 checks passed
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.

1 participant