All the modern browsers support display: flex and other related properties. We don't need -webkit-flex anymore.
Ref: https://caniuse.com/flexbox
.entity-moderation-form {
display: -webkit-flex; /* Safari */
display: flex;
-webkit-flex-wrap: wrap; /* Safari */
flex-wrap: wrap;
-webkit-align-items: flex-start; /* Safari */
align-items: flex-start;
list-style: none;
}
.entity-moderation-form__item:last-child {
-webkit-align-self: flex-end; /* Safari */
align-self: flex-end;
margin-right: 0;
}
Proposed resolution:
Remove these properties:
.entity-moderation-form {
- display: -webkit-flex; /* Safari */
display: flex;
- -webkit-flex-wrap: wrap; /* Safari */
flex-wrap: wrap;
- -webkit-align-items: flex-start; /* Safari */
align-items: flex-start;
list-style: none;
}
.entity-moderation-form__item:last-child {
-webkit-align-self: flex-end; /* Safari */
align-self: flex-end;
margin-right: 0;
}
Comments
Comment #2
gauravvvv commentedI have provided a patch for same. Please review.
Comment #3
rinku jacob 13 commentedHi @Gauravvv , Verified and tested patch #2 on drupal version 10.1.x. The patch was successfully removed webkit properties from content_moderation.module.css file. Need RTBC +1
Comment #4
smustgrave commentedSeems like a simple cleanup.
Good job!
Comment #5
lauriiiClosed #3338478: Stable: Remove webkit-specific display flex from CSS as a duplicate of this. We should implement the stable changes here.
Comment #6
gauravvvv commentedRemoved the webkit properties from stable theme as well. Here is updated patch with interdiff please review
Comment #7
smustgrave commentedLooks good!
Comment #8
longwave@Rinku Jacob 13 there is no need to post screenshots of patches being applied, DrupalCI will tell us whether this is successful.
Committed and pushed to 10.1.x, thanks!