Skip to content

Conversation

@spliffone
Copy link
Member

@spliffone spliffone commented Jan 15, 2026

Describe in detail what your merge request does and why. Add relevant
screenshots and reference related issues via Closes #XY or Related to #XY.


@spliffone spliffone requested a review from a team as a code owner January 15, 2026 13:33
@gemini-code-assist
Copy link

Summary of Changes

Hello @spliffone, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request significantly refactors the si-collapsible-panel component by migrating its expand and collapse animations from Angular's animation module to native CSS. This change aims to streamline the component's implementation by reducing TypeScript-side animation logic and delegating control to CSS properties like grid-template-rows and @starting-style, potentially leading to a more performant and maintainable animation system.

Highlights

  • Animation Refactor: The accordion's expand/collapse animation has been refactored to use native CSS animations instead of Angular's @angular/animations module.
  • CSS-driven Collapse: The animation now leverages CSS grid-template-rows for height transitions and @starting-style for defining the initial state of the animation, providing a more native feel.
  • Simplified Component Logic: The si-collapsible-panel.component.ts file has been simplified by removing Angular animation imports and the animations metadata from the component decorator.
  • Conditional Content Rendering: The collapsible content div is now conditionally rendered using an @if block, which works in conjunction with the new CSS animation approach for managing its presence in the DOM.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the collapsible panel to use native CSS animations instead of Angular's animation library, which is a great modernization. The implementation leverages display: grid and animates grid-template-rows for a smooth collapse/expand effect, which is a robust technique. The code is cleaner and has fewer dependencies. I've found one critical issue regarding the restoration of the scroll position which I've commented on. Otherwise, this is a solid improvement.

@spliffone
Copy link
Member Author

@dr-itz Can you take a look? We had an issue 1183 where you adjusted the scroll behaviour but I couldn't figure out what was the problem.

@spliffone spliffone force-pushed the refactor/collapsible-drop-angular-animation branch from 065ce75 to a9ff0bf Compare January 15, 2026 13:46
@github-actions
Copy link

@dr-itz
Copy link
Contributor

dr-itz commented Jan 15, 2026

So the intention is that scrolling position is restored upon re-opening an accordion panel. In the example here: https://element.siemens.io/element-examples/#/overview/si-accordion/si-accordion-full-height?q=ac

you can make the screen small enough that the first panel will scroll. Opening another panel and then re-opening the first one restores the position. Currently broken in this MR

@spliffone spliffone force-pushed the refactor/collapsible-drop-angular-animation branch from a9ff0bf to 1156bbb Compare January 15, 2026 14:59
@spliffone spliffone force-pushed the refactor/collapsible-drop-angular-animation branch from 1156bbb to aa2106d Compare January 15, 2026 20:19
@spliffone spliffone requested a review from a team as a code owner January 15, 2026 20:19
@spliffone spliffone force-pushed the refactor/collapsible-drop-angular-animation branch 2 times, most recently from cda71ad to fdbedf3 Compare January 19, 2026 10:55
@spliffone spliffone force-pushed the refactor/collapsible-drop-angular-animation branch from fdbedf3 to 7f9d070 Compare January 20, 2026 08:22
@github-actions
Copy link

Code Coverage

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.

3 participants