Skip to content

Commit

Permalink
fix(FEC-11510): No default font size selected in cvaa (#639)
Browse files Browse the repository at this point in the history
Use fontSize instead of fontScale.
Fixes FEC-11510.

Related PRs:
kaltura/playkit-js#603
kaltura/kaltura-player-js#491
  • Loading branch information
SivanA-Kaltura committed Sep 13, 2021
1 parent 77c70c7 commit c9ad30c
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/components/cvaa-overlay/custom-captions-window.js
Expand Up @@ -61,10 +61,10 @@ class CustomCaptionsWindow extends Component {
const edgeStyles = player.TextStyle.EdgeStyles;
const standardColors = player.TextStyle.StandardColors;

const fontScaleOptions = player.TextStyle.FontSizes.map(scale => ({
value: scale.value,
label: scale.label,
active: props.customTextStyle.fontScale === scale.value
const fontSizeOptions = player.TextStyle.FontSizes.map(size => ({
value: size.label,
label: size.label,
active: props.customTextStyle.fontSize === size.label
}));

const fontColorOptions = Object.keys(standardColors).map(key => ({
Expand Down Expand Up @@ -97,9 +97,9 @@ class CustomCaptionsWindow extends Component {
<DropDownCaptionsStyle
addAccessibleChild={props.addAccessibleChild}
labelId="cvaa.size_label"
options={fontScaleOptions}
options={fontSizeOptions}
classNames={[style.formGroupRow, style.fontSize]}
styleName="fontScale"
styleName="fontSize"
changeCustomStyle={props.changeCustomStyle}
/>
<DropDownCaptionsStyle
Expand Down

0 comments on commit c9ad30c

Please sign in to comment.