Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(button): prevent tooltip from showing on incorrect hover #6349

Merged
merged 4 commits into from
Jun 29, 2020

Conversation

tw15egan
Copy link
Collaborator

Closes #6346

Prevents mouse hover on the invisible caret to trigger the icon-only button tooltip from showing.

Changelog

New

  • Styles to prevent tooltip from showing when hovering over the invisible caret. This also prevents clicking on the tooltip to click the button

Testing / Reviewing

Ensure icon-only button still renders properly, and that you can not trigger the tooltip by hovering slightly below the button (where the caret is)

@tw15egan tw15egan requested a review from a team as a code owner June 25, 2020 19:00
@ghost ghost requested review from aledavila and dakahn June 25, 2020 19:00
@netlify
Copy link

netlify bot commented Jun 25, 2020

Deploy preview for carbon-elements ready!

Built with commit b408a27

https://deploy-preview-6349--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Jun 25, 2020

Deploy preview for carbon-components-react ready!

Built with commit b408a27

https://deploy-preview-6349--carbon-components-react.netlify.app

@tw15egan tw15egan merged commit d0770ed into carbon-design-system:master Jun 29, 2020
gracelo added a commit to gracelo/carbon that referenced this pull request Jun 30, 2020
* origin/master: (77 commits)
  feat(pictograms): add systems devops category and pictograms (carbon-design-system#6332)
  fix(button): prevent tooltip from showing on incorrect hover (carbon-design-system#6349)
  feat(pictograms): add coronavirus pictogram in Life Science category (carbon-design-system#6351)
  chore(npmignore): update npmignore file (carbon-design-system#6350)
  chore(project): sync generated files
  fix(pagination): set correct width on ghost buttons (carbon-design-system#6344)
  docs(text-area): rename story file to use camel case in component name (carbon-design-system#6334)
  chore(release): v10.15.0-rc.0 (carbon-design-system#6343)
  feat(pictograms): New add device pictogram (carbon-design-system#6333)
  fix(Dropdown): add downshiftProps handling (carbon-design-system#6341)
  feat(pictograms): add supply chain category with new pictograms (carbon-design-system#6305)
  feat(pictograms): add gift pictogram in retail (carbon-design-system#6336)
  feat(pictograms): add blender pictogram in lifestyle category (carbon-design-system#6318)
  fix(dropdowns): pass user ID's to Downshift (carbon-design-system#6326)
  fix(carbon): revert pr 6185 (carbon-design-system#6331)
  feat(react): remove unsafe lifecycle methods from DataTable and DatePicker (carbon-design-system#6307)
  docs(handbook): add icon docs to developer-handbook (carbon-design-system#6300)
  feat(pictograms): add planning analytics pictogram to cloud category (carbon-design-system#6320)
  chore(project): sync generated files
  fix(loading): use `$overlay-01` token for overlay (carbon-design-system#6323)
  ...

# Conflicts:
#	packages/components/src/components/tabs/_tabs.scss
@tw15egan tw15egan deleted the buttonTooltipFix branch April 28, 2021 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[React Button] Tooltip of React icon-only button appears outside of button, is clickable and triggers onClick
3 participants