dbeaver/pro#6557 [CB] Dropdown for roles is too small (#3663)

* dbeaver/pro#6557 refactor: remove redundant width: 100%

There are a few reasons why we don't need this property:
0. It's already present in base Select styles
1. Select component is flex container by default, i.e. takes all available place
2. No evidences that it is needed

* dbeaver/pro#6557 fix: user filters selects

Simplifies layout and set min width for selects

---------

Co-authored-by: Daria Marutkina <125263541+dariamarutkina@users.noreply.github.com>
This commit is contained in:
Sychev Andrey
2025-08-19 18:01:41 +08:00
committed by GitHub
co-authored by Daria Marutkina
parent 62ad5a61c9
commit 4ff45220ee
2 changed files with 5 additions and 6 deletions
@@ -36,7 +36,6 @@
--dbv-kit-select-border-radius: 3px;
--dbv-kit-control-outline-offset: -2px;
width: 100%;
}
.dbv-kit-select {
@@ -8,7 +8,7 @@
import { observer } from 'mobx-react-lite';
import { AuthRolesResource } from '@cloudbeaver/core-authentication';
import { Select, Group, useResource, useTranslate } from '@cloudbeaver/core-blocks';
import { Select, useResource, useTranslate } from '@cloudbeaver/core-blocks';
import { type IUserFilters, USER_ROLE_ALL, USER_STATUSES } from './useUsersTableFilters.js';
@@ -21,22 +21,22 @@ export const UsersTableFiltersDetails = observer<Props>(function UsersTableFilte
const authRolesResource = useResource(UsersTableFiltersDetails, AuthRolesResource, undefined);
return (
<Group box gap>
<div className="tw:flex tw:gap-4">
<Select
value={filters.status}
items={USER_STATUSES}
valueSelector={value => translate(value.label)}
keySelector={value => value.value}
keepSize
className="tw:min-w-[172px]"
onSelect={filters.setStatus}
>
{translate('authentication_user_status')}
</Select>
{!!authRolesResource.data.length && (
<Select items={[...authRolesResource.data, USER_ROLE_ALL]} value={filters.role} keepSize onSelect={filters.setRole}>
<Select items={[...authRolesResource.data, USER_ROLE_ALL]} value={filters.role} className="tw:min-w-[172px]" onSelect={filters.setRole}>
{translate('authentication_user_role')}
</Select>
)}
</Group>
</div>
);
});