react icon indicating copy to clipboard operation
react copied to clipboard

Using `<Tooltip>` within `<ButtonGroup>` affects styling

Open tmelliottjr opened this issue 2 years ago • 6 comments

Description

Using <Tooltip> within <ButtonGroup> affects the group's styling.

Expected:

Image showing expected output

Actual:

Image showing actual output

Steps to reproduce

<ButtonGroup>
 <Tooltip aria-label="Some text">
  <Button>😜</Button>
 </Tooltip>
 <Tooltip aria-label="Some more text">
  <Button>😳</Button>
 </Tooltip>
 <Tooltip aria-label="Even more text">
  <Button>😭</Button>
 </Tooltip>
</ButtonGroup>

Version

v36.5.0

Browser

Chrome

tmelliottjr avatar Jan 08 '24 21:01 tmelliottjr

Uh oh! @tmelliottjr, the image you shared is missing helpful alt text. Check your issue body.

Alt text is an invisible description that helps screen readers describe images to blind or low-vision users. If you are using markdown to display images, add your alt text inside the brackets of the markdown image.

Learn more about alt text at Basic writing and formatting syntax: images on GitHub Docs.

github-actions[bot] avatar Jan 08 '24 21:01 github-actions[bot]

Uh oh! @tmelliottjr, the image you shared is missing helpful alt text. Check your issue body.

Alt text is an invisible description that helps screen readers describe images to blind or low-vision users. If you are using markdown to display images, add your alt text inside the brackets of the markdown image.

Learn more about alt text at Basic writing and formatting syntax: images on GitHub Docs.

github-actions[bot] avatar Jan 08 '24 21:01 github-actions[bot]

Ah, yes. This is definitely a bug! I have added this to the agenda for the next prioritisation call (every monday)

@broccolinisoup FYI, something to take care of in https://github.com/primer/react/pull/4046 as well!

siddharthkp avatar Jan 09 '24 12:01 siddharthkp

Thanks @siddharthkp for the ping! When the button group have text buttons with tooltip, there is a style issue there too. This time, the tooltip style is broken; doesn't have the rounded borders. I suspect this is the issue in the tooltip itself. I added this as a task under https://github.com/github/primer/issues/1244 and will look into it.

Button group that includes three text buttons. The first one has tooltip and the tooltip appears without any radius on the borders.

broccolinisoup avatar Jan 12 '24 03:01 broccolinisoup

Hi! This issue has been marked as stale because it has been open with no activity for 180 days. You can comment on the issue or remove the stale label to keep it open. If you do nothing, this issue will be closed in 7 days.

github-actions[bot] avatar Aug 10 '24 00:08 github-actions[bot]

not stale

broccolinisoup avatar Aug 12 '24 02:08 broccolinisoup