search

Automattic / dops-components

Shared Calypso-style components for non-Calypso projects.
GNU General Public License v2.0
9 stars 6 forks source link

Fixed the banner plan class colors to match the icon. #100

Closed zinigor closed 7 years ago

zinigor commented 7 years ago

Before only yearly plans would properly activate border colors, but now monthly plans do the same.

eliorivero commented 7 years ago

Your changes are ok 👍

Added a commit so the top border is visible on hover

Before

captura de pantalla 2017-03-15 a las 11 06 33

After

captura de pantalla 2017-03-15 a las 11 05 28

I didn't want to use margin-top: 1px to avoid positioning issues. For now this is ok for the current usage of all banners. If there comes a time where the banner is next to a element with a higher z-index we might have to position it correctly but it would be very specific.