Skip to content

Commit ea842ee

Browse files
author
Krzysztof Pniak
committed
feat(*): renames forId to widgetId
1 parent 97a1b07 commit ea842ee

File tree

8 files changed

+18
-19
lines changed

8 files changed

+18
-19
lines changed

packages/angular-widget/src/HeadwayWidget.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -97,14 +97,14 @@ export class HeadwayWidget implements OnInit, OnDestroy {
9797
@Component({
9898
selector: "headway-widget-trigger",
9999
template: `<div
100-
class="{{ headwayWidgetTriggerClassName + '_' + forId }}"
100+
class="{{ headwayWidgetTriggerClassName + '_' + widgetId }}"
101101
style="position: relative; display: inline-block"
102102
>
103103
<ng-content></ng-content>
104104
</div>`,
105105
})
106106
export class HeadwayWidgetTrigger {
107-
@Input() forId: string = "widget-1";
107+
@Input() widgetId: string = "widget-1";
108108
headwayWidgetTriggerClassName = HeadwayWidgetTriggerClassName;
109109
}
110110

packages/angular-widget/stories/widget.stories.mdx

+3-3
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ class AppComponent {}
129129

130130
## External trigger
131131

132-
Widget can be triggered by any other element on the page. To achieve that use HeadwayWidgetTrigger and set its `forId`
132+
Widget can be triggered by any other element on the page. To achieve that use HeadwayWidgetTrigger and set its `widgetId`
133133
prop to corresponding widget `id`.
134134

135135
<Preview withSource="none">
@@ -139,7 +139,7 @@ prop to corresponding widget `id`.
139139
<headway-widget account="J1LKG7" id="idOfYourChoice">
140140
<logo></logo>
141141
</headway-widget>
142-
<headway-widget-trigger forId="idOfYourChoice">
142+
<headway-widget-trigger widgetId="idOfYourChoice">
143143
<button>Trigger</button>
144144
</headway-widget-trigger>
145145
</div>`,
@@ -156,7 +156,7 @@ prop to corresponding widget `id`.
156156
<headway-widget account="J1LKG7" id="idOfYourChoice">
157157
Logo
158158
</headway-widget>
159-
<headway-widget-trigger forId="idOfYourChoice">
159+
<headway-widget-trigger widgetId="idOfYourChoice">
160160
<button>Trigger</button>
161161
</headway-widget-trigger>
162162
</div>

packages/react-widget/package.json

-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
"build": "rollup -c",
88
"dev": "start-storybook -p 6006",
99
"build-storybook": "build-storybook",
10-
"pack:rename": "(cd output/ && mv \"$(npm pack ../ | tail -n 1)\" package.tgz)"
1110
},
1211
"files": [
1312
"dist/*"

packages/react-widget/src/HeadwayWidget.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,15 @@ const HeadwayWidgetSelector = "." + HeadwayWidgetClassName;
66
const HeadwayWidgetTriggerClassName = "HW_trigger";
77
const HeadwayWidgetTriggerSelector = "." + HeadwayWidgetTriggerClassName;
88

9-
const HeadwayWidgetTrigger = ({ forId, component, children }) => {
9+
const HeadwayWidgetTrigger = ({ widgetId, component, children }) => {
1010
const Component = component || "div";
11-
const className = HeadwayWidgetTriggerClassName + `_${forId}`;
11+
const className = HeadwayWidgetTriggerClassName + `_${widgetId}`;
1212

1313
return <Component className={className}>{children}</Component>;
1414
};
1515

1616
HeadwayWidgetTrigger.defaultProps = {
17-
forId: "widget-1",
17+
widgetId: "widget-1",
1818
};
1919

2020
const parsePosition = (positionText) => {

packages/react-widget/stories/widget.stories.mdx

+3-3
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ Widget can be triggered by its children.
9494

9595
## External trigger
9696

97-
Widget can be triggered by any other element on the page. To achieve that use HeadwayWidgetTrigger and set its `forId`
97+
Widget can be triggered by any other element on the page. To achieve that use HeadwayWidgetTrigger and set its `widgetId`
9898
prop to corresponding widget `id`.
9999

100100
<Preview withSource="none">
@@ -103,7 +103,7 @@ prop to corresponding widget `id`.
103103
<HeadwayWidget account="J1LKG7" id="widget-external-trigger">
104104
<Logo />
105105
</HeadwayWidget>
106-
<HeadwayWidgetTrigger forId="widget-external-trigger">
106+
<HeadwayWidgetTrigger widgetId="widget-external-trigger">
107107
<button>Trigger</button>
108108
</HeadwayWidgetTrigger>
109109
</div>
@@ -115,7 +115,7 @@ prop to corresponding widget `id`.
115115
<HeadwayWidget account="J1LKG7" id="widget-external-trigger">
116116
<Logo />
117117
</HeadwayWidget>
118-
<HeadwayWidgetTrigger forId="widget-external-trigger">
118+
<HeadwayWidgetTrigger widgetId="widget-external-trigger">
119119
<button>Trigger</button>
120120
</HeadwayWidgetTrigger>
121121
</div>

packages/vanilla-widget/stories/widget.stories.mdx

+2-2
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ You can change widget position to one of 6 possible values. Check all of them in
5959

6060
## External trigger
6161

62-
Widget can be triggered by any other element on the page. To achieve that use HeadwayWidgetTrigger and set its `forId`
62+
Widget can be triggered by any other element on the page. To achieve that use HeadwayWidgetTrigger and set its `widgetId`
6363
prop to corresponding widget `id`.
6464

6565
<Preview>
@@ -69,7 +69,7 @@ prop to corresponding widget `id`.
6969
<headway-widget account="J1LKG7" id="4">
7070
<logo></logo>
7171
</headway-widget>
72-
<headway-widget-trigger forId="4">
72+
<headway-widget-trigger widgetId="4">
7373
<button>Trigger</button>
7474
</headway-widget-trigger>
7575
</div>`,

packages/vue-widget/src/HeadwayWidgetTrigger.vue

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ const HeadwayWidgetTriggerSelector = "." + HeadwayWidgetTriggerClassName;
1818
export default {
1919
name: "headway-widget-trigger",
2020
props: {
21-
forId: {
21+
widgetId: {
2222
type: String,
2323
default: 'widget-1'
2424
},
@@ -28,7 +28,7 @@ export default {
2828
},
2929
},
3030
data: (self) => ({
31-
cName: HeadwayWidgetTriggerClassName + `_${self.forId}`,
31+
cName: HeadwayWidgetTriggerClassName + `_${self.widgetId}`,
3232
}),
3333
};
3434
</script>

packages/vue-widget/stories/widget.stories.mdx

+3-3
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ You can change widget position to one of 6 possible values. Check all of them in
119119

120120
## External trigger
121121

122-
Widget can be triggered by any other element on the page. To achieve that use HeadwayWidgetTrigger and set its `forId`
122+
Widget can be triggered by any other element on the page. To achieve that use HeadwayWidgetTrigger and set its `widgetId`
123123
prop to corresponding widget `id`.
124124

125125
<Preview withSource="none">
@@ -129,7 +129,7 @@ prop to corresponding widget `id`.
129129
<headway-widget account="J1LKG7" id="4">
130130
<logo></logo>
131131
</headway-widget>
132-
<headway-widget-trigger forId="4">
132+
<headway-widget-trigger widgetId="4">
133133
<button>Trigger</button>
134134
</headway-widget-trigger>
135135
</div>`,
@@ -143,7 +143,7 @@ prop to corresponding widget `id`.
143143
<headway-widget account="J1LKG7" position="center-left" id="idOfYourChoice"
144144
>Logo</headway-widget
145145
>
146-
<headway-widget-trigger forId="idOfYourChoice">
146+
<headway-widget-trigger widgetId="idOfYourChoice">
147147
<button>Trigger</button>
148148
</headway-widget-trigger>
149149
</div>

0 commit comments

Comments
 (0)