fix: restore border to ghost button and fix discover slider visibility toggle position (#3226)

pull/3743/head
Ryan Cohen 2 years ago committed by Mike Kao
parent f95bb82a8f
commit 4716f24c02

@ -46,7 +46,7 @@ function Button<P extends ElementTypes = 'button'>(
ref?: React.Ref<Element<P>> ref?: React.Ref<Element<P>>
): JSX.Element { ): JSX.Element {
const buttonStyle = [ const buttonStyle = [
'inline-flex items-center justify-center border border-transparent leading-5 font-medium rounded-md focus:outline-none transition ease-in-out duration-150 cursor-pointer disabled:opacity-50 whitespace-nowrap', 'inline-flex items-center justify-center border leading-5 font-medium rounded-md focus:outline-none transition ease-in-out duration-150 cursor-pointer disabled:opacity-50 whitespace-nowrap',
]; ];
switch (buttonType) { switch (buttonType) {
case 'primary': case 'primary':
@ -71,7 +71,7 @@ function Button<P extends ElementTypes = 'button'>(
break; break;
case 'ghost': case 'ghost':
buttonStyle.push( buttonStyle.push(
'text-white bg-transaprent border-gray-600 hover:border-gray-200 focus:border-gray-100 active:border-gray-100' 'text-white bg-transparent border-gray-600 hover:border-gray-200 focus:border-gray-100 active:border-gray-100'
); );
break; break;
default: default:

@ -184,10 +184,13 @@ const DiscoverSliderEdit = ({
/> />
)} )}
<div className="flex w-full flex-col rounded-t-lg border-t border-l border-r border-gray-800 bg-gray-900 p-4 text-gray-400 md:flex-row md:items-center md:space-x-2"> <div className="flex w-full flex-col rounded-t-lg border-t border-l border-r border-gray-800 bg-gray-900 p-4 text-gray-400 md:flex-row md:items-center md:space-x-2">
<div className="mb-4 flex space-x-2 md:mb-0"> <div
className={`${slider.data ? 'mb-4' : 'mb-0'} flex space-x-2 md:mb-0`}
>
<Bars3Icon className="h-6 w-6" /> <Bars3Icon className="h-6 w-6" />
<div>{getSliderTitle(slider)}</div> <div>{getSliderTitle(slider)}</div>
</div> </div>
{slider.data && (
<div className="pointer-events-none mb-4 flex-1 md:mb-0"> <div className="pointer-events-none mb-4 flex-1 md:mb-0">
{(slider.type === DiscoverSliderType.TMDB_MOVIE_KEYWORD || {(slider.type === DiscoverSliderType.TMDB_MOVIE_KEYWORD ||
slider.type === DiscoverSliderType.TMDB_TV_KEYWORD) && ( slider.type === DiscoverSliderType.TMDB_TV_KEYWORD) && (
@ -226,6 +229,7 @@ const DiscoverSliderEdit = ({
<Tag iconSvg={<MagnifyingGlassIcon />}>{slider.data}</Tag> <Tag iconSvg={<MagnifyingGlassIcon />}>{slider.data}</Tag>
)} )}
</div> </div>
)}
<div className="flex items-center space-x-2"> <div className="flex items-center space-x-2">
{!slider.isBuiltIn && ( {!slider.isBuiltIn && (
<> <>
@ -265,7 +269,7 @@ const DiscoverSliderEdit = ({
</Button> </Button>
</> </>
)} )}
<div className="flex-1 pl-4 text-right"> <div className="absolute top-4 right-4 flex-1 pl-4 text-right md:relative md:top-0 md:right-0">
<Tooltip content={intl.formatMessage(messages.enable)}> <Tooltip content={intl.formatMessage(messages.enable)}>
<div> <div>
<SlideCheckbox <SlideCheckbox

Loading…
Cancel
Save