Fix alignment of Plugin Updates in Horizontal Mode #2495
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
Restores left-aligned layout for the update-info widget in horizontal mode and tidies up the HTML structure.
- Replaced
align-items-centerwithalign-items-starton the main container to align items at the top. - Consolidated multi-line attributes into single lines for clickable
<div>and<a>elements. - Added HTML comments before each service block to help identify sections.
Comments suppressed due to low confidence (3)
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.html:38
- [nitpick] The comment
<!-- Homebridge -->appears in both Docker and non-Docker sections and may be ambiguous. Consider specifying<!-- Docker Homebridge -->and<!-- Standalone Homebridge -->for clarity.
<!-- Homebridge -->
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.html:9
- The inline style
cursor: pointeris redundant since thecursor-pointerclass already sets the pointer cursor. Consider removing the inline style for consistency.
<div class="d-flex ps-3 py-1 align-items-center cursor-pointer" (click)="toggleDockerExpand()"
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.html:8
- Multiple elements use the same inline
min-widthand flex styles. Extract these into a reusable CSS class to reduce duplication and improve maintainability.
<div class="hb-status-item d-flex flex-column mb-1" style="min-width: max(25%, 200px)">
bwp91
approved these changes
Jul 15, 2025
bwp91
pushed a commit
that referenced
this pull request
Jul 15, 2025
bwp91
pushed a commit
that referenced
this pull request
Jul 16, 2025
bwp91
pushed a commit
that referenced
this pull request
Jul 17, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change was to put back align-items-start
#2478
To fake running in Docker, add these environment variables