feat(radio): align with 2018 material design spec #12299
Conversation
googlebot
added
the
cla: yes
| .mat-radio-inner-circle, | ||
| .mat-radio-ripple .mat-ripple-element:not(.mat-radio-persistent-ripple), | ||
| &.mat-radio-checked .mat-radio-persistent-ripple, | ||
| &:active .mat-radio-persistent-ripple { |
There was a problem hiding this comment.
A comment with the different states would be probably nice here, but not required 😄
devversion
added
pr: lgtm
action: merge
|
I see that you just added the If you want your PR to be merged, it has to pass all the CI checks. If you can't get the PR to a green state due to flakes or broken master, please try rebasing to master and/or restarting the CI job. If that fails and you believe that the issue is not due to your change, please contact the caretaker and ask for help. |
crisbeto
force-pushed
the
radio-spec-alignment
branch
from
6ed07c9 to
b52cf1c
Compare
crisbeto
force-pushed
the
radio-spec-alignment
branch
from
b52cf1c to
0e2dd1b
Compare
crisbeto
changed the title
jelbourn
added
the
P2
|
Hi @crisbeto! This PR has merge conflicts due to recent upstream merges. |
Aligns the radio button with the latest iteration of the Material spec. The radio itself was mostly aligned, but these changes introduce ripples hover, as well as varying opacity for the focused and active ripples.
crisbeto
force-pushed
the
radio-spec-alignment
branch
from
0e2dd1b to
7ffa194
Compare
|
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. |
Aligns the radio button with the latest iteration of the Material spec. The radio itself was mostly aligned, but these changes introduce ripples hover, as well as varying opacity for the focused and active ripples.