Skip to content

Commit 7837d5f

Browse files
authored
Merge pull request #120 from farrah-deriv/P2PS-2883/disable-ad-menu
farrah/P2PS-2883/fix: disable advert menu
2 parents ed8e8df + 0e23d4c commit 7837d5f

File tree

2 files changed

+13
-10
lines changed

2 files changed

+13
-10
lines changed

src/components/PopoverDropdown/PopoverDropdown.tsx

+11-7
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,17 @@ const PopoverDropdown = ({ dropdownList, onClick, tooltipMessage }: TPopoverDrop
2525

2626
return (
2727
<div className='popover-dropdown' ref={ref}>
28-
<TooltipMenuIcon
29-
as='button'
30-
onClick={() => setVisible(prevState => !prevState)}
31-
tooltipContent={tooltipMessage}
32-
>
33-
<LabelPairedEllipsisVerticalLgBoldIcon data-testid='dt_popover_dropdown_icon' />
34-
</TooltipMenuIcon>
28+
{isAdvertiserBarred ? (
29+
<LabelPairedEllipsisVerticalLgBoldIcon data-testid='dt_popover_dropdown_icon' fill='#999999' />
30+
) : (
31+
<TooltipMenuIcon
32+
as='button'
33+
onClick={() => setVisible(prevState => !prevState)}
34+
tooltipContent={tooltipMessage}
35+
>
36+
<LabelPairedEllipsisVerticalLgBoldIcon data-testid='dt_popover_dropdown_icon' />
37+
</TooltipMenuIcon>
38+
)}
3539
{visible && (
3640
<div className='popover-dropdown__list'>
3741
{dropdownList.map(item => (

src/components/PopoverDropdown/__tests__/PopoverDropdown.spec.tsx

+2-3
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,9 @@ describe('PopoverDropdown', () => {
4848
expect(mockProps.onClick).toHaveBeenCalledWith('value 1');
4949
});
5050

51-
it('should disable the button if advertiser is barred', async () => {
51+
it('should disable the icon if advertiser is barred', async () => {
5252
mockUseIsAdvertiserBarred.mockReturnValue(true);
5353
render(<PopoverDropdown {...mockProps} />);
54-
await userEvent.click(screen.getByTestId('dt_popover_dropdown_icon'));
55-
expect(screen.getByRole('button', { name: 'label 1' })).toBeDisabled();
54+
expect(screen.getByTestId('dt_popover_dropdown_icon')).not.toHaveProperty('onClick');
5655
});
5756
});

0 commit comments

Comments
 (0)