From bb4558a46b032a6f7c99aa09d7214b94c23f10fc Mon Sep 17 00:00:00 2001 From: jeremiG Date: Fri, 4 Jan 2019 11:04:06 -0500 Subject: [PATCH] fix(ui): Remove rounded borders on button class Inconsistent UX can cause problems and unhappy users Ref: #1198 --- src/app/ui/styles-guide/styleGuideDavid.js | 32 ++++++++++------------ 1 file changed, 14 insertions(+), 18 deletions(-) diff --git a/src/app/ui/styles-guide/styleGuideDavid.js b/src/app/ui/styles-guide/styleGuideDavid.js index 849cf8ab94..f223fad0da 100644 --- a/src/app/ui/styles-guide/styleGuideDavid.js +++ b/src/app/ui/styles-guide/styleGuideDavid.js @@ -60,7 +60,6 @@ function styleGuideDavid () { BORDERS ------------------------------------------------------ */ borders: { - primary_borderRadius: '3px', secondary_borderRadius: '5px' } } @@ -283,7 +282,6 @@ function styleGuideDavid () { background-color : ${opts.BackgroundColor}; border : 1px solid ${opts.BorderColor}; color : ${opts.Color}; - border-radius : ${cssProperties.borders.primary_borderRadius}; font-size : 12px; padding : 10px 15px; line-height : 20px; @@ -345,22 +343,20 @@ function styleGuideDavid () { `, button: (opts = {}) => ` - margin : 1px; - background-color : ${opts.BackgroundColor}; - border : .3px solid ${opts.BorderColor}; - color : ${opts.Color}; - display : flex; - align-items : center; - justify-content : center; - border-radius : 3px; - cursor : pointer; - min-height : 25px; - max-height : 25px; - width : 70px; - min-width : 70px; - font-size : 12px; - overflow : hidden; - word-break : normal; + margin : 1px; + background-color : ${opts.BackgroundColor}; + color : ${opts.Color}; + display : flex; + align-items : center; + justify-content : center; + cursor : pointer; + min-height : 25px; + max-height : 25px; + width : 70px; + min-width : 70px; + font-size : 12px; + overflow : hidden; + word-break : normal; ` } }