Skip to content

Commit 4a0cbec

Browse files
committed
Fix: Address feedback
1 parent a95ff63 commit 4a0cbec

File tree

3 files changed

+8
-10
lines changed

3 files changed

+8
-10
lines changed

packages/components/src/components/hds/code-block/index.ts

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,9 @@ export default class HdsCodeBlock extends Component<HdsCodeBlockSignature> {
100100
});
101101

102102
private _setUpCodeBlockCode = modifier((element: HTMLElement) => {
103-
this._isExpanded = false; // reset expanded state on updates
103+
if (this.showFooter) {
104+
this._isExpanded = false; // reset expanded state on updates
105+
}
104106
this.setPrismCode(element);
105107
return () => {};
106108
});
@@ -175,10 +177,10 @@ export default class HdsCodeBlock extends Component<HdsCodeBlockSignature> {
175177
}
176178

177179
// Existing line numbers must be removed in order to be updated correctly
178-
if (element.querySelector('.line-numbers-rows')) {
179-
const lineNumbers = element.querySelector(
180-
'.line-numbers-rows'
181-
) as HTMLElement;
180+
const lineNumbers = element.querySelector(
181+
'.line-numbers-rows'
182+
) as HTMLElement;
183+
if (lineNumbers) {
182184
element.removeChild(lineNumbers);
183185
}
184186

showcase/app/styles/showcase-pages/code-block.scss

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,4 @@ body.components-code-block {
1313
left: 0;
1414
}
1515
}
16-
17-
.shw-component-code-block-demo-btn {
18-
margin-top: 12px;
19-
}
2016
}

showcase/app/templates/components/code-block.hbs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -727,7 +727,7 @@ func main() {
727727
type="button"
728728
@text="Update"
729729
@isInline={{true}}
730-
class="shw-component-code-block-demo-btn"
730+
{{style marginTop="12px"}}
731731
{{on "click" this.onUpdateClickDemo1}}
732732
/>
733733
</SF.Item>

0 commit comments

Comments
 (0)