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.

Sign up for GitHub

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

Jump to bottom

NPopover doesn't display when triggered using NFloatButton #5933

Open
5 tasks done
ikeq opened this issue May 10, 2024 · 1 comment · May be fixed by #5941
Open
5 tasks done

NPopover doesn't display when triggered using NFloatButton #5933

ikeq opened this issue May 10, 2024 · 1 comment · May be fixed by #5941
Labels
bug Something isn't working

Comments

@ikeq
Copy link

ikeq commented May 10, 2024

Describe the bug

image

Steps to reproduce

https://www.naiveui.com/en-US/light/components/float-button#tooltip.vue

Link to minimal reproduction

https://www.naiveui.com/en-US/light/components/float-button#tooltip.vue

System Info

System:
    OS: macOS 14.4.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 102.00 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.16.0 - ~/.nvm/versions/node/v18.16.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v16.20.0/bin/yarn
    npm: 9.5.1 - ~/.nvm/versions/node/v18.16.0/bin/npm
    pnpm: 7.27.1 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 124.0.6367.156
    Edge: 124.0.2478.80
    Safari: 17.4.1
  npmPackages:
    naive-ui: ^2.38.2 => 2.38.2 
    vue: ^3.4.19 => 3.4.21

Used Package Manager

yarn

Validations
@ikeq ikeq added the untriaged need to sort label May 10, 2024
@OrbisK OrbisK added bug Something isn't working and removed untriaged need to sort labels May 10, 2024
@jizai1125
Copy link
Contributor

jizai1125 commented May 13, 2024 •
edited

There's an issue with the popover component. It is a workaround, wrapping an element of the trigger slot.

<template>
  <n-tooltip trigger="hover" placement="right">
    <template #trigger>
      <span style="display: inline-block">
        <n-float-button position="relative">
          <n-icon>
            <cash-icon />
          </n-icon>
        </n-float-button>
      </span>
    </template>
    Lpsum
  </n-tooltip>
</template>

jizai1125 added a commit to jizai1125/naive-ui that referenced this issue May 13, 2024
@jizai1125
fix(float-button): error when using with popover, closes tusen-ai#5933
5b8bfae
@jizai1125 jizai1125 linked a pull request May 13, 2024 that will close this issue
Open
jizai1125 added a commit to jizai1125/naive-ui that referenced this issue May 17, 2024
@jizai1125
fix(float-button): error when using with popover, closes tusen-ai#5933
36e3e32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(float-button): error when using with popover jizai1125/naive-ui
3 participants
@ikeq @jizai1125 @OrbisK