Skip to content

Commit 321fa39

Browse files
committed
fix: feedback Bob
1 parent 8fe1adc commit 321fa39

File tree

5 files changed

+31
-33
lines changed

5 files changed

+31
-33
lines changed

src/components/cc-plan-item/cc-plan-item.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ export class CcPlanItem extends LitElement {
6060
}
6161

6262
/**
63-
*
6463
* @param {PlanDetails} detail
6564
* @returns TemplateResult
6665
*/

src/components/cc-plan-picker/cc-plan-picker.js

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -234,22 +234,18 @@ export class CcPlanPicker extends CcFormControlElement {
234234
}
235235
236236
.plan-legend-text {
237-
--ct-form-label-font-family: 'Source Sans 3';
238-
--ct-form-label-font-size: 1.625em;
239-
--ct-form-label-font-weight: 500;
240-
--ct-form-input-font-size: 1.25em;
241-
242237
color: var(--cc-color-text-primary-strongest);
243-
font-family: var(--ct-form-label-font-family), sans-serif;
244-
font-size: var(--ct-form-label-font-size);
245-
font-weight: var(--ct-form-label-font-weight);
238+
font-family: var(--cc-ff-form-legend), inherit;
239+
font-size: 1.625em;
240+
font-weight: 500;
246241
}
247242
248243
.form-controls {
249244
display: grid;
250245
gap: 1em;
251-
grid-template-columns: repeat(auto-fill, minmax(12.5em, 1fr));
252-
margin: 1em 2em;
246+
grid-template-columns: repeat(auto-fill, minmax(13em, 1fr));
247+
margin-block-start: 0.5em;
248+
margin-inline-start: 34px;
253249
}
254250
`,
255251
];

src/components/cc-plan-picker/cc-plan-picker.stories.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export const defaultStory = makeStory(conf, {
2727
/** @type {Array<Partial<CcPlanPicker>>} */
2828
items: [
2929
{
30-
name: 'foobar',
30+
name: 'plan',
3131
plans: DEFAULT_PLANS,
3232
value: 'plan_12',
3333
},
@@ -38,6 +38,7 @@ export const readonly = makeStory(conf, {
3838
/** @type {Array<Partial<CcPlanPicker>>} */
3939
items: [
4040
{
41+
name: 'plan',
4142
plans: DEFAULT_PLANS,
4243
readonly: true,
4344
value: 'plan_12',
@@ -49,6 +50,7 @@ export const sampleDatabase = makeStory(conf, {
4950
/** @type {Array<Partial<CcPlanPicker>>} **/
5051
items: [
5152
{
53+
name: 'plan',
5254
plans: [
5355
{
5456
name: 'S',
@@ -215,6 +217,7 @@ export const thirdParty = makeStory(conf, {
215217
/** @type {Array<Partial<CcPlanPicker>>} */
216218
items: [
217219
{
220+
name: 'plan',
218221
plans: THIRD_PARTY,
219222
value: 'third_party_1',
220223
},
@@ -225,6 +228,7 @@ export const thirdPartyReadonly = makeStory(conf, {
225228
/** @type {Array<Partial<CcPlanPicker>>} */
226229
items: [
227230
{
231+
name: 'plan',
228232
plans: THIRD_PARTY,
229233
readonly: true,
230234
value: 'third_party_1',

src/stories/fixtures/plans.js

Lines changed: 19 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,6 @@ export const DEFAULT_PLANS = [
220220
},
221221
],
222222
},
223-
224223
{
225224
name: 'L',
226225
id: 'plan_5',
@@ -363,23 +362,23 @@ export const DEFAULT_PLANS = [
363362

364363
/** @type {PlanItem[]} **/
365364
export const THIRD_PARTY = [{
366-
id: 'third_party_1',
367-
name: 'Third party 1',
368-
details: [],
369-
},
370-
{
371-
id: 'third_party_2',
372-
name: 'Third party 2',
373-
details: [],
374-
},
375-
{
376-
id: 'third_party_3',
377-
name: 'Third party 3',
378-
details: [],
379-
},
380-
{
381-
id: 'third_party_4',
382-
name: 'Third party 4',
383-
details: [],
384-
},
365+
id: 'third_party_1',
366+
name: 'Third party 1',
367+
details: [],
368+
},
369+
{
370+
id: 'third_party_2',
371+
name: 'Third party 2',
372+
details: [],
373+
},
374+
{
375+
id: 'third_party_3',
376+
name: 'Third party 3',
377+
details: [],
378+
},
379+
{
380+
id: 'third_party_4',
381+
name: 'Third party 4',
382+
details: [],
383+
},
385384
];

src/translations/translations.fr.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1066,7 +1066,7 @@ export const translations = {
10661066
//#endregion
10671067
//#region cc-plan-picker
10681068
'cc-plan-picker.legend': `Sélectionnez votre plan`,
1069-
'cc-plan-picker.legend.customize': `Customisez votre plan`,
1069+
'cc-plan-picker.legend.customize': `Personnalisez votre plan`,
10701070
//#endregion
10711071
//#region cc-pricing-estimation
10721072
'cc-pricing-estimation.count.label': /** @param {{productCount: number}} _ */ ({ productCount }) =>

0 commit comments

Comments
 (0)