Skip to content

Commit c7c0207

Browse files
authored
chore: edits for professionalism of our comments (#18346)
1 parent 8865e78 commit c7c0207

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

src/cdk/a11y/interactivity-checker/interactivity-checker.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -390,7 +390,7 @@ describe('InteractivityChecker', () => {
390390

391391
appendElements([objectEl]);
392392

393-
// This is a hack to create an empty contentDocument for the frame element.
393+
// This creates an empty contentDocument for the frame element.
394394
objectEl.type = 'text/html';
395395
objectEl.contentDocument!.body.appendChild(button);
396396

src/material/datepicker/datepicker-content.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ $mat-datepicker-touch-max-height: 788px;
3333

3434
.mat-datepicker-content-touch {
3535
display: block;
36-
// make sure the dialog scrolls rather than being cropped on ludicrously small screens
36+
// Make sure the dialog scrolls rather than being cropped on ludicrously small screens
3737
max-height: 80vh;
3838
overflow: auto;
3939

40-
// TODO(mmalerba): hack to offset the padding of the dialog. Can be removed when we switch away
41-
// from using dialog.
40+
// Offsets the padding of the dialog.
41+
// TODO(mmalerba): Remove when we switch away from using dialog.
4242
margin: -24px;
4343

4444
.mat-calendar {

src/material/form-field/form-field-outline.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ $mat-form-field-outline-subscript-padding:
8181
}
8282

8383
.mat-form-field-outline-gap {
84-
// hack for Chrome's treatment of borders with non-integer (scaled) widths
84+
// Workaround for Chrome's treatment of borders with non-integer (scaled) widths
8585
// refer to https://github.com/angular/components/issues/10710
8686
border-radius: 0.000001px;
8787

src/material/form-field/form-field.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -251,8 +251,8 @@ export class MatFormField extends _MatFormFieldMixinBase
251251
@ContentChild(MatFormFieldControl) _controlNonStatic: MatFormFieldControl<any>;
252252
@ContentChild(MatFormFieldControl, {static: true}) _controlStatic: MatFormFieldControl<any>;
253253
get _control() {
254-
// TODO(crisbeto): we need this hacky workaround in order to support both Ivy
255-
// and ViewEngine. We should clean this up once Ivy is the default renderer.
254+
// TODO(crisbeto): we need this workaround in order to support both Ivy and ViewEngine.
255+
// We should clean this up once Ivy is the default renderer.
256256
return this._explicitFormFieldControl || this._controlNonStatic || this._controlStatic;
257257
}
258258
set _control(value) {

0 commit comments

Comments
 (0)