Skip to content

Commit

Permalink
🐛 Fixed cramped filter columns in QuickFilters (#587)
Browse files Browse the repository at this point in the history
* Fixed cramped filter rows in QuickFilters

* Bumped package versions

* Updated hover-color for quick-filters
  • Loading branch information
kjellhaaland authored Feb 15, 2024
1 parent 88a5cc0 commit 2b506fd
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 11 deletions.
2 changes: 1 addition & 1 deletion packages/power-bi/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@equinor/workspace-powerbi",
"version": "1.0.15",
"version": "1.0.16",
"type": "module",
"sideEffects": false,
"license": "MIT",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import { ActiveFilter, PowerBiFilter, PowerBiFilterItem } from '../../types';
import { getFilterHeaderText } from '../../utils/getFilterHeader';
import { FilterController } from '../Filter/Filter';
import { PowerBiGroupPopoverMenu } from '../groupPopoverMenu/GroupPopoverMenu';
import { StyledFilterGroupWrapper } from './quickFilterGroup.styles';
import { StyledFilterGroupContent, StyledFilterGroupWrapper } from './quickFilterGroup.styles';

interface PowerBiFilterGroupProps {
controller: FilterController;
Expand All @@ -28,15 +28,17 @@ export const PowerBiFilterGroup = ({
return (
<>
<StyledFilterGroupWrapper onClick={() => setIsOpen((s) => !s)} ref={anchorEl}>
<div>
{getFilterHeaderText(
isAllChecked,
group.type,
activeFilters.map((s) => s?.toString() ?? '(Blank)')
)}
</div>
<StyledFilterGroupContent>
<div>
{getFilterHeaderText(
isAllChecked,
group.type,
activeFilters.map((s) => s?.toString() ?? '(Blank)')
)}
</div>

<Icon color={tokens.colors.text.static_icons__tertiary.hex} name="chevron_down" />
<Icon color={tokens.colors.text.static_icons__tertiary.hex} name="chevron_down" />
</StyledFilterGroupContent>
</StyledFilterGroupWrapper>
{isOpen && (
<PowerBiGroupPopoverMenu
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,13 @@ export const StyledFilterGroupWrapper = styled.div`
width: min-content;
display: flex;
align-items: center;
`;

export const StyledFilterGroupContent = styled.div`
height: auto;
width: min-content;
display: flex;
align-items: center;
cursor: pointer;
justify-content: center;
overflow: hidden;
Expand Down
2 changes: 1 addition & 1 deletion packages/workspace-fusion/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@equinor/workspace-fusion",
"version": "7.0.5",
"version": "7.0.6",
"type": "module",
"sideEffects": false,
"license": "MIT",
Expand Down

0 comments on commit 2b506fd

Please sign in to comment.