fixes for hover, regenerate button known regression

codecov · Sep 4, 2024 · e55f3e9 · e55f3e9
1 parent 53027a1
commit e55f3e9
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 6 deletions.
diff --git a/src/pages/LoginPage/LoginButton.jsx b/src/pages/LoginPage/LoginButton.jsx
@@ -18,7 +18,7 @@ function LoginButton({ provider }) {
 
  return (
  <a
- className="flex h-14 items-center rounded-sm border border-ds-gray-quaternary bg-ds-gray-primary font-semibold shadow hover:bg-ds-gray-secondary dark:hover:bg-ds-gray-quaternary"
+ className="flex h-14 items-center rounded-sm border border-ds-gray-quaternary bg-ds-gray-primary font-semibold shadow hover:bg-ds-gray-secondary"
  href={signIn.path({ to, provider })}
  data-cy={'login-button'}
  >

diff --git a/src/pages/SyncProviderPage/SyncButton.tsx b/src/pages/SyncProviderPage/SyncButton.tsx
@@ -12,7 +12,7 @@ const SyncButton: React.FC<SyncButtonProps> = ({ provider }) => {
  return (
  <div className="flex h-14 items-center rounded-sm border border-ds-gray-quaternary bg-ds-gray-primary text-left shadow">
  <a
- className="flex h-full grow items-center font-semibold hover:bg-ds-gray-secondary dark:hover:bg-ds-gray-quaternary"
+ className="flex h-full grow items-center font-semibold hover:bg-ds-gray-secondary"
  href={signIn.path({ to, provider })}
  data-cy={'login-button'}
  >

diff --git a/src/ui/Button/Button.jsx b/src/ui/Button/Button.jsx
@@ -25,7 +25,6 @@ const variantClasses = {
  border-solid border shadow
 
  hover:bg-ds-gray-secondary
- dark:hover:bg-ds-gray-quaternary
  `,
  primary: `
  justify-center font-semibold

diff --git a/src/ui/ContextSwitcher/ContextSwitcher.jsx b/src/ui/ContextSwitcher/ContextSwitcher.jsx
@@ -40,7 +40,7 @@ function ContextItem({ context, defaultOrgUsername, setToggle, owner }) {
 
  return (
  <li
- className="cursor-pointer select-none py-2 text-gray-900 hover:bg-ds-gray-secondary dark:hover:bg-ds-gray-quaternary"
+ className="cursor-pointer select-none py-2 text-gray-900 hover:bg-ds-gray-secondary"
  id="listbox-option-0"
  >
  <Button

diff --git a/src/ui/Dropdown/Dropdown.tsx b/src/ui/Dropdown/Dropdown.tsx
@@ -120,7 +120,7 @@ const Content = React.forwardRef<
 Content.displayName = 'Dropdown.Content'
 
 const item = cva(
- 'focus:bg-accent focus:text-accent-foreground relative flex cursor-default select-none items-center rounded-sm px-2 py-1.5 text-sm text-ds-gray-octonary outline-none transition-colors hover:cursor-pointer hover:bg-ds-gray-secondary dark:hover:bg-ds-gray-quaternary data-[disabled]:pointer-events-none data-[disabled]:opacity-50'
+ 'focus:bg-accent focus:text-accent-foreground relative flex cursor-default select-none items-center rounded-sm px-2 py-1.5 text-sm text-ds-gray-octonary outline-none transition-colors hover:cursor-pointer hover:bg-ds-gray-secondary data-[disabled]:pointer-events-none data-[disabled]:opacity-50'
 )
 
 interface ItemProps

diff --git a/src/ui/Select/Select.jsx b/src/ui/Select/Select.jsx
@@ -15,7 +15,7 @@ const SelectClasses = {
  root: 'relative',
  item: 'block cursor-pointer py-1 px-3 text-sm font-normal',
  button:
- 'flex justify-between items-center w-full rounded bg-ds-background text-left whitespace-nowrap disabled:text-ds-gray-quaternary disabled:bg-ds-gray-primary disabled:border-ds-gray-tertiary focus:outline-1',
+ 'flex justify-between items-center w-full rounded bg-ds-container text-left whitespace-nowrap disabled:text-ds-gray-quaternary disabled:bg-ds-gray-primary disabled:border-ds-gray-tertiary focus:outline-1',
  ul: 'overflow-hidden rounded-bl rounded-br bg-ds-background border-ds-gray-tertiary absolute w-full z-40 max-h-80 min-w-fit',
  loadMoreTrigger: 'relative top-[-65px] invisible block leading-[0]',
 }