feat(sbb-bar-container): initial implementation#4533
Conversation
Summary of ChangesHello @TomMenga, 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 delivers the initial implementation of the Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe 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
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 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. New component takes flight, Red band leads, a guiding light, Code reviewed, all bright. Footnotes
|
DavideMininni-Fincons
left a comment
There was a problem hiding this comment.
Looks good, added some ideas for improvement
# Conflicts: # tools/web-test-runner/preload-icons.ts
jeripeierSBB
left a comment
There was a problem hiding this comment.
Nice work. I think we should discuss with @mcilurzo and @kyubisation whether we should make it easy to align the card on the grid (on sbb.ch I see various example where they have a smaller grid area for the card and link). Right now, when placing sbb-grid class on the sbb-bar-container (like a consumer could do), it needs a lot of manual fixes (but it would be possible). Maybe we should apply the grid as well, so it would be easier to just re-define the grid-area. Although with the current design, without a slot for the link, it makes it a little complicated as there are two grid rows to be specified. With a dedicated slot we could additional just define the vertical gap of the grid instead of the slotted rules.
jeripeierSBB
left a comment
There was a problem hiding this comment.
LGTM! Will ask @mcilurzo for a UX review
# Conflicts: # src/elements/core/styles/core.scss
|
We postpone this feature approx. until autumn, as we would like to merge it with the sbb-lean-container. |

Preflight Checklist
Issue
This PR Closes #3431
Pull request checklist
Please check if your PR fulfills the following requirements:
See Review Guidelines for more information on what is checked during the review process.
Changes
Changes in this pull request:
Browsers
I tested the build on the following browsers:
Screen readers
I tested the build on the following browsers:
Pull request type
Please check the type of change your PR introduces:
Does this introduce a breaking change?
Other information