-
Notifications
You must be signed in to change notification settings - Fork 9
release/15_10_2025 #444
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
release/15_10_2025 #444
Changes from 22 commits
d52b3b1
956f02b
c8ace7e
38ca2d3
e97641f
9329dcd
0445dc3
e0a2694
5fccbda
e957263
18a4c96
097903d
44ba780
102f365
c04b42c
6f217ab
564f72d
7d2e0d2
1bc2613
2764f48
0323183
e7281ca
4b86026
169c1cd
6013a04
097b62d
1a57dc8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,16 +1,17 @@ | ||
| <a [routerLink]="['/', article().slug] | alLocalize"> | ||
| <article | ||
| class="h-full rounded-lg bg-cover bg-no-repeat transition-transform hover:scale-105 motion-reduce:transition-none motion-reduce:hover:scale-100" | ||
| [style.background-image]=" | ||
| 'url(' + | ||
| (article().featuredImageUrl || 'assets/article-placeholder.webp') + | ||
| ')' | ||
| " | ||
| class="relative h-full overflow-hidden rounded-lg bg-black bg-cover bg-no-repeat transition-transform hover:scale-105 motion-reduce:transition-none motion-reduce:hover:scale-100" | ||
coderabbitai[bot] marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| [attr.aria-labelledby]="article().slug" | ||
| > | ||
| <div | ||
| class="bg-al-background relative flex h-full flex-col justify-between opacity-85" | ||
| > | ||
| <img | ||
| alt="" | ||
| class="absolute h-auto w-full bg-contain opacity-20" | ||
| [ngSrc]="article().featuredImageUrl || 'assets/article-placeholder.webp'" | ||
| [priority]="imagePriority()" | ||
| width="1215" | ||
| height="750" | ||
| /> | ||
| <div class="relative flex h-full flex-col justify-between text-[#fff]"> | ||
|
||
| <div | ||
| class="hover:bg-al-bottom-radial-gradient absolute h-full w-full" | ||
| ></div> | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,6 +1,6 @@ | ||||||
| <a [routerLink]="['/', article().slug] | alLocalize"> | ||||||
| <article | ||||||
| class="bg-al-card md:hover:shadow-al-primary group relative h-full w-full rounded-lg border border-transparent shadow-none transition-transform motion-reduce:transition-none lg:hover:scale-105 lg:motion-reduce:hover:scale-100" | ||||||
| class="bg-al-card md:hover:shadow-al-primary group relative h-full w-full rounded-lg border shadow-none transition-transform motion-reduce:transition-none lg:hover:scale-105 lg:motion-reduce:hover:scale-100 dark:border-transparent" | ||||||
|
||||||
| class="bg-al-card md:hover:shadow-al-primary group relative h-full w-full rounded-lg border shadow-none transition-transform motion-reduce:transition-none lg:hover:scale-105 lg:motion-reduce:hover:scale-100 dark:border-transparent" | |
| class="bg-al-card md:hover:shadow-al-primary group relative h-full w-full rounded-lg border-al-border shadow-none transition-transform motion-reduce:transition-none lg:hover:scale-105 lg:motion-reduce:hover:scale-100" |
🤖 Prompt for AI Agents
In
libs/blog/articles/ui-article-card/src/lib/components/article-regular-card/article-regular-card.component.html
around line 3, the element uses Tailwind's generic border class plus
dark:border-transparent which conflicts with the theme-scoped border variable;
replace the hard-coded border classes with the theme-aware border variable
approach (use the --border CSS variable for border color/width via an
appropriate utility or inline style and remove the dark: override) so both light
and dark themes rely on the main.scss --border value; ensure the element no
longer uses the plain "border" and "dark:border-transparent" classes and instead
applies the border using the --border variable consistent with the rest of the
app.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can be private