fix: setting pe-btn* focus indicator

Pearson-Higher-Ed / elements-sdk

Base Styling and Atomic JavaScript Components

MIT License

13 stars 13 forks source link

fix: setting pe-btn* focus indicator #153

Closed bldunn-dev closed 5 years ago

bldunn-dev commented 5 years ago

Setting the outline for pe-btn*:focus to match what is for the base

antelopeb commented 5 years ago

Hey @briandunnpearson , we have deliberately disabled that outline state, as we control the focus with a pseudo selector instead. This is because you can't set the border-radius on focus outlines, but you can set them on the pseudo selector. This matches the design intent.

I'm going to close this one for now. Let me know if there is a specific use case that needs solved for here.