Skip to content

Conversation

@JammingBen
Copy link
Contributor

@JammingBen JammingBen self-assigned this Sep 16, 2025
Copilot AI review requested due to automatic review settings September 16, 2025 12:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes avatar alignment and z-index issues in the design system by adjusting CSS classes for better visual presentation. The changes ensure proper spacing between stacked avatars and correct z-index behavior during hover interactions.

  • Updated avatar stacking margin from -ml-4 to -ml-3 with :not-first selector
  • Fixed z-index hover effect by adding ! important modifier
  • Increased timeline padding from pl-5 to pl-8 for better alignment

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/design-system/src/components/OcAvatars/OcAvatars.vue Updated avatar stacking CSS classes to improve spacing and z-index behavior
packages/design-system/src/components/OcList/OcList.vue Increased left padding for timeline list items
packages/web-app-files/tests/unit/components/SideBar/snapshots/ActivitiesPanel.spec.ts.snap Updated test snapshots to reflect the new CSS class changes

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@kulmann kulmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@kulmann kulmann merged commit 06e3498 into main Sep 16, 2025
29 checks passed
@kulmann kulmann deleted the fix/activities-panel-alignment branch September 16, 2025 13:14
openclouders pushed a commit that referenced this pull request Sep 16, 2025
@openclouders openclouders mentioned this pull request Sep 16, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants