Skip to content

Commit 03f60a4

Browse files
committed
fix: add ripples to clickable cards
Adds ripples to the various clickable cards to make them feel more "Material-ey".
1 parent b79ba88 commit 03f60a4

File tree

6 files changed

+12
-6
lines changed

6 files changed

+12
-6
lines changed

material.angular.io/material.angular.io/src/app/pages/component-category-list/component-category-list.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<a *ngFor="let component of docItems.getItems(section)"
88
class="docs-component-category-list-item"
99
[routerLink]="'/' + section + '/' + component.id">
10-
<div class="docs-component-category-list-card">
10+
<div class="docs-component-category-list-card" matRipple>
1111
<img *ngIf="section === 'components'"
1212
class="docs-component-category-list-card-image-wrapper"
1313
[src]="'../../../assets/screenshots/' + component.id + '.scene.png'"

material.angular.io/material.angular.io/src/app/pages/component-category-list/component-category-list.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import {CommonModule, NgIf, NgFor, AsyncPipe} from '@angular/common';
22
import {Component, NgModule, OnDestroy, OnInit} from '@angular/core';
33
import {MatCardModule} from '@angular/material/card';
44
import {ActivatedRoute, Params, RouterModule, RouterLink} from '@angular/router';
5+
import {MatRipple} from '@angular/material/core';
56
import {combineLatest, Observable, Subscription} from 'rxjs';
67

78
import {
@@ -17,7 +18,7 @@ import {ComponentPageTitle} from '../page-title/page-title';
1718
templateUrl: './component-category-list.html',
1819
styleUrls: ['./component-category-list.scss'],
1920
standalone: true,
20-
imports: [NavigationFocus, NgIf, NgFor, RouterLink, AsyncPipe]
21+
imports: [NavigationFocus, NgIf, NgFor, RouterLink, AsyncPipe, MatRipple]
2122
})
2223
export class ComponentCategoryList implements OnInit, OnDestroy {
2324
params: Observable<Params> | undefined;

material.angular.io/material.angular.io/src/app/pages/guide-list/guide-list.html

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
<main focusOnNavigation aria-label="Guide list" id="guide-list" class="docs-guide">
22
<div class="docs-guide-list">
3-
<a class='docs-guide-item' *ngFor="let guide of guideItems.getAllItems()"
4-
routerLink='/guide/{{guide.id}}'>
3+
<a class="docs-guide-item" *ngFor="let guide of guideItems.getAllItems()"
4+
routerLink="/guide/{{guide.id}}"
5+
matRipple>
56
<mat-card class="docs-guide-card" appearance="outlined">
67
<mat-card-title>{{guide.name}}</mat-card-title>
78
<mat-card-content class="docs-guide-card-summary">

material.angular.io/material.angular.io/src/app/pages/guide-list/guide-list.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,14 @@ import {NgFor} from '@angular/common';
66
import {NavigationFocus} from '../../shared/navigation-focus/navigation-focus';
77
import {ComponentPageTitle} from '../page-title/page-title';
88
import {MatCardModule} from '@angular/material/card';
9+
import {MatRipple} from '@angular/material/core';
910

1011
@Component({
1112
selector: 'app-guides',
1213
templateUrl: './guide-list.html',
1314
styleUrls: ['./guide-list.scss'],
1415
standalone: true,
15-
imports: [NavigationFocus, NgFor, RouterLink, MatCardModule, Footer]
16+
imports: [NavigationFocus, NgFor, RouterLink, MatCardModule, Footer, MatRipple]
1617
})
1718
export class GuideList implements OnInit {
1819
@HostBinding('class.main-content') readonly mainContentClass = true;

material.angular.io/material.angular.io/src/app/pages/homepage/homepage.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,8 @@ <h2>Guides</h2>
7777
<app-carousel aria-label="Guides">
7878
<a carousel-item class="carousel-item docs-homepage-guides-carousel-item"
7979
*ngFor="let guide of guideItems.getAllItems()"
80-
routerLink="/guide/{{guide.id}}">
80+
routerLink="/guide/{{guide.id}}"
81+
matRipple>
8182
<mat-card class="docs-homepage-guides-card" appearance="outlined">
8283
<mat-card-title>{{guide.name}}</mat-card-title>
8384
<mat-card-content class="docs-component-category-list-card-summary">{{guide.overview}}</mat-card-content>

material.angular.io/material.angular.io/src/app/pages/homepage/homepage.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import {Component, HostBinding, Inject, OnInit, Optional} from '@angular/core';
22
import {ANIMATION_MODULE_TYPE} from '@angular/platform-browser/animations';
33

44
import {MatButtonModule} from '@angular/material/button';
5+
import {MatRipple} from '@angular/material/core';
56
import {Footer} from '../../shared/footer/footer';
67
import {RouterLink} from '@angular/router';
78
import {ComponentPageTitle} from '../page-title/page-title';
@@ -36,6 +37,7 @@ const TOP_COMPONENTS = ['datepicker', 'input', 'slide-toggle', 'slider', 'button
3637
Support,
3738
Footer,
3839
AppLogo,
40+
MatRipple,
3941
],
4042
})
4143
export class Homepage implements OnInit {

0 commit comments

Comments
 (0)