Skip to content

Commit 3bbf529

Browse files
committed
Telemetry + flex align
1 parent d500cbb commit 3bbf529

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

src/controls/carousel/Carousel.module.scss

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@
5050

5151
.buttonsContainer {
5252
display: flex;
53+
align-items: center;
5354
background-color: transparent;
5455
cursor: pointer;
5556
}

src/controls/carousel/Carousel.tsx

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,13 +10,16 @@ import { css, ICssInput } from "@uifabric/utilities/lib";
1010
import { ProcessingState } from "./ICarouselState";
1111
import { Spinner } from "office-ui-fabric-react/lib/Spinner";
1212
import { isArray } from "@pnp/common";
13+
import * as telemetry from '../../common/telemetry';
1314

1415
export class Carousel extends React.Component<ICarouselProps, ICarouselState> {
1516
constructor(props: ICarouselProps) {
1617
super(props);
1718

1819
const currentIndex = props.startIndex ? props.startIndex : 0;
1920

21+
telemetry.track('ReactCarousel', {});
22+
2023
this.state = {
2124
currentIndex,
2225
processingState: ProcessingState.idle
@@ -58,7 +61,8 @@ export class Carousel extends React.Component<ICarouselProps, ICarouselState> {
5861

5962
return (
6063
<div className={this.getMergedStyles(styles.container, containerStyles)}>
61-
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)} onClick={() => { if (!prevButtonDisabled) { this.onCarouselButtonClicked(false); } }} >
64+
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)}
65+
onClick={() => { if (!prevButtonDisabled) { this.onCarouselButtonClicked(false); } }} >
6266
<IconButton
6367
className={this.getMergedStyles(this.getButtonStyles(false), prevButtonStyles)}
6468
iconProps={{ iconName: prevButtonIconName }}
@@ -85,7 +89,8 @@ export class Carousel extends React.Component<ICarouselProps, ICarouselState> {
8589
}
8690
</div>
8791

88-
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)} onClick={() => { if (!nextButtonDisabled) { this.onCarouselButtonClicked(true); } }}>
92+
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)}
93+
onClick={() => { if (!nextButtonDisabled) { this.onCarouselButtonClicked(true); } }}>
8994
<IconButton
9095
className={this.getMergedStyles(this.getButtonStyles(true), nextButtonStyles)}
9196
iconProps={{ iconName: nextButtonIconName }}
@@ -99,7 +104,7 @@ export class Carousel extends React.Component<ICarouselProps, ICarouselState> {
99104
/**
100105
* Return merged styles for Button containers.
101106
*/
102-
private getButtonContainerStyles = (): string => {
107+
private getButtonContainerStyles(): string {
103108
const buttonsDisplayMode = this.props.buttonsDisplay ? this.props.buttonsDisplay : CarouselButtonsDisplay.block;
104109
let buttonDisplayModeCss = "";
105110
switch (buttonsDisplayMode) {

0 commit comments

Comments
 (0)