frontend: kraken: InstalledExtensionCard: Fix style of cards in different pages sizes#3903
Merged
Williangalvani merged 1 commit intobluerobotics:masterfrom Apr 28, 2026
Conversation
Reviewer's GuideAdjusts the InstalledExtensionCard layout to behave better across different page widths by making the title area flex-safe, truncating long text, and making the Update button responsive without fixed widths or narrow-screen media overrides. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue, and left some high level feedback:
- The
>>>combinator used in.card-update-button >>> .v-btn__contentis deprecated in newer Vue tooling; consider replacing it with::v-deep(or an equivalent approach supported by your build setup) to keep the deep selector forward-compatible.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The `>>>` combinator used in `.card-update-button >>> .v-btn__content` is deprecated in newer Vue tooling; consider replacing it with `::v-deep` (or an equivalent approach supported by your build setup) to keep the deep selector forward-compatible.
## Individual Comments
### Comment 1
<location path="core/frontend/src/components/kraken/cards/InstalledExtensionCard.vue" line_range="32-41" />
<code_context>
- <div>
- <div>{{ extension.name }}</div>
+ <div class="title-text">
+ <div class="text-truncate">
+ {{ extension.name }}
+ </div>
<span
</code_context>
<issue_to_address>
**suggestion:** Truncating the name without any fallback might hurt discoverability and accessibility.
Because `text-truncate` hides overflow, long names become partially inaccessible, unlike the button which now has a tooltip. Please add a tooltip or similar affordance on the truncated name/tag so users can still see the full value when it’s cut off.
```suggestion
</v-avatar>
<div class="title-text">
<div
class="text-truncate"
v-tooltip="extension.name"
>
{{ extension.name }}
</div>
<span
class="d-block text-truncate"
style="color: grey;"
v-tooltip="extension.tag"
>
{{ extension.tag }}
</div>
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
…rent pages sizes Signed-off-by: Patrick José Pereira <patrickelectric@gmail.com>
3450970 to
9735107
Compare
Williangalvani
approved these changes
Apr 28, 2026
Williangalvani
approved these changes
Apr 28, 2026
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
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.
Old:


New:
Old:


New:
Summary by Sourcery
Improve the layout and responsiveness of the InstalledExtensionCard header and update button across different page sizes.
Bug Fixes:
Enhancements: