-
Notifications
You must be signed in to change notification settings - Fork 6.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(form-field): remove 6.0.0 deletion targets #10164
refactor(form-field): remove 6.0.0 deletion targets #10164
Conversation
@deletion-target 6.0.0 | ||
--> | ||
<span class="mat-form-field-label-wrapper mat-input-placeholder-wrapper mat-form-field-placeholder-wrapper"> | ||
<span class="mat-form-field-label-wrapper"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a @deletion-target
while you're here to remove all of the mat-input-*
classes? We can do the actual remove in a different PR because I know for sure it's going to be a nightmare to merge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. I've set it to 7.0.0, but I can bump it down to 6.0.0 if you want.
14a0de5
to
a6384f3
Compare
Can you change the breaking notes to match that earlier PR where applicable (i.e. "Xxx, which was deprecated in 5.0.0, has been removed. Use yyy instead")? |
a6384f3
to
049918e
Compare
Done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Caretaker note: we need to remove google usages of the old API. This might require a cleanup before syncing this PR. |
049918e
to
119f082
Compare
small piece of angular#10164, split out for presubmit reasons BREAKING CHANGES: - dividerColor which was deprecated in 5.0.0 has been removed. Use color instead. - floatPlaceholder which was deprecated in 5.0.0 has been removed. Use floatLabel instead.
Removes the targets marked for 6.0.0 deletion from the `material/form-field`. BREAKING CHANGES: * `dividerColor` which was deprecated in 5.0.0 has been removed. Use `color` instead. * `floatPlaceholder` which was deprecated in 5.0.0 has been removed. Use `floatLabel` instead. * `MatFormFieldControl.shouldPlaceholderFloat` which was deprecated in 5.0.0 has been removed. Use `shouldLabelFloat` instead. * `FloatPlaceholderType` which was deprecated in 5.0.0 has been removed. Use `FloatLabelType` instead. * `PlaceholderOptions` which was deprecated in 5.0.0 has been removed. Use `LabelOptions` instead. * `MAT_PLACEHOLDER_GLOBAL_OPTIONS` which was deprecated in 5.0.0 has been removed. Use `MAT_LABEL_GLOBAL_OPTIONS` instead. * The `mat-input-placeholder-wrapper` and `mat-form-field-placeholder-wrapper` CSS classes which were deprecated in 5.0.0 have bee removed. The same element can be targeted using the `mat-form-field-label-wrapper` class. * The `mat-input-placeholder` and `mat-form-field-placeholder` CSS classes which were deprecated in 5.0.0 have bee removed. The same element can be targeted using the `mat-form-field-label` class.
119f082
to
98a7d90
Compare
small piece of #10164, split out for presubmit reasons BREAKING CHANGES: - dividerColor which was deprecated in 5.0.0 has been removed. Use color instead. - floatPlaceholder which was deprecated in 5.0.0 has been removed. Use floatLabel instead.
closing since all shard PRs have been merged |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Removes the targets marked for 6.0.0 deletion from the
material/form-field
.BREAKING CHANGES:
dividerColor
which was deprecated in 5.0.0 has been removed. Usecolor
instead.floatPlaceholder
which was deprecated in 5.0.0 has been removed. UsefloatLabel
instead.MatFormFieldControl.shouldPlaceholderFloat
which was deprecated in 5.0.0 has been removed. UseshouldLabelFloat
instead.FloatPlaceholderType
which was deprecated in 5.0.0 has been removed. UseFloatLabelType
instead.PlaceholderOptions
which was deprecated in 5.0.0 has been removed. UseLabelOptions
instead.MAT_PLACEHOLDER_GLOBAL_OPTIONS
which was deprecated in 5.0.0 has been removed. UseMAT_LABEL_GLOBAL_OPTIONS
instead.mat-input-placeholder-wrapper
andmat-form-field-placeholder-wrapper
CSS classes which were deprecated in 5.0.0 have bee removed. The same element can be targeted using themat-form-field-label-wrapper
class.mat-input-placeholder
andmat-form-field-placeholder
CSS classes which were deprecated in 5.0.0 have bee removed. The same element can be targeted using themat-form-field-label
class.