Skip to content

Commit

Permalink
fix: remove animations form tooltip docs and fix placement example
Browse files Browse the repository at this point in the history
  • Loading branch information
cwoolum committed Aug 1, 2024
1 parent e0ea1e2 commit 74dd6cd
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ export default component$(() => {
<option value="left">Left</option>
</select>

<Tooltip.Root gutter={4} flip placement={placement.value}>
<Tooltip.Root key={placement.value} gutter={4} flip placement={placement.value}>
<Tooltip.Trigger>Hover or Focus me</Tooltip.Trigger>
<Tooltip.Panel
aria-label={`Tooltip content on the ${placement.value}`}
Expand Down
9 changes: 6 additions & 3 deletions apps/website/src/routes/docs/headless/tooltip/index.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -118,11 +118,14 @@ Use the `data-open` and `data-closed` attributes on the `<Tooltip.Panel>` compon

### Cross-browser Animations

Entry and exit animations have often been a frustrating experience in web development, especially trying to animate between `display: none`, a discrete property.
{/\* Entry and exit animations have often been a frustrating experience in web development, especially trying to animate between `display: none`, a discrete property.

Luckily, the Qwik UI team has done an excellent job of managing animations on polyfill browsers for you using the `data-*` attributes.

<CodeSnippet name="animation.css" />
*/}

> Animations are not currently available in the Tooltip component. We are working on a solution to provide a more seamless experience. In the meantime, you can still use transitions.
## Tooltip Behavior

Expand Down Expand Up @@ -170,7 +173,7 @@ If Tailwind is the framework of choice, then styles can be added using the [arbi

> The arbitrary variant can be customized/abstracted even further by [adding a variant](https://tailwindcss.com/docs/plugins#adding-variants) as a plugin in the tailwind config.
## Animations
{/\* ## Animations

<Showcase name="animation" />

Expand All @@ -182,7 +185,7 @@ To use an animation, add the following CSS classes to the component.

Here's the CSS imported from the example:

<CodeSnippet name="animation.css" />
<CodeSnippet name="animation.css" /> */}

### Transition declarations

Expand Down

0 comments on commit 74dd6cd

Please sign in to comment.