fix: Auto teleport tooltip #8821
Conversation
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
AI-Generated Summary: This pull request introduces a change in the tooltip implementation for the 'autoTeleport' feature in the application. The The patch includes the creation of the new Also, the Overall, this should yield a smoother and enhanced user experience in the Auto Teleport feature segment. |
|
Code Climate has analyzed commit d890ba5 and detected 0 issues on this pull request. View more on Code Climate. |
|
hassnian
requested review from
preschian and
Jarsen136
and removed request for
a team
|
@exezbcz please review |
first time using v-tippy , so idk , seems to be better in some aspects but we are only using it in 3 places The only reason I ended up using v-tippy it's because I couldn't get the same behaviour using the oruga component. using only one would good idea though |
|
yup looks good thanks! one issue i came across is this now. Not sure if related to the deploy, but this does not seem right
seems like its also a canary issue the autoteleport should not be active my address: |
@exezbcz can you add the visual ok tag thanks ;D |
| @include ktheme() { | ||
| background: theme('background-color'); | ||
| } |
There was a problem hiding this comment.
Please check theme-background-color at global.scss
prury
added
the
S-works-for-me-✅
|
Thanks! |
|
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
PR Type
Context
use tippy instate of NeoTooltip
fixed vertical margins
renamed file from AutotelportTooltip to AutotelportPopever
Closes Auto teleport tooltip #8820
Closes Auto-teleport in confirm modal simplified follow-up #8451
Did your issue had any of the "$" label on it?
Community participation
Screenshot 📸