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

[DRAFT] feat: tooltip component #561

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

[DRAFT] feat: tooltip component #561

wants to merge 12 commits into from

Conversation

CTAPCKPIM
Copy link
Contributor

@CTAPCKPIM CTAPCKPIM commented Dec 28, 2023

Goal: add tooltip for FDM

Steps:

  1. Detect place in the code
  2. Update the code

Log:

  • Research time: 2 hrs

ETA: 1-4 hrs

@CTAPCKPIM CTAPCKPIM self-assigned this Dec 28, 2023
Copy link

vercel bot commented Dec 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
holdex-venture-studio ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 18, 2024 10:37am

@CTAPCKPIM
Copy link
Contributor Author

@CTAPCKPIM
Copy link
Contributor Author

Problem:

  • Don't update display of the date

@CTAPCKPIM
Copy link
Contributor Author

@georgeciubotaru

  • I searched the place for an update of the component, let's merge, and in the next PR I finish

Copy link
Member

@zolotokrylin zolotokrylin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. wrong PR title
  2. why you even working on this issue? is it is not important!

@zolotokrylin
Copy link
Member

There is no problem with the date. It is a fake and problem, and a solution is a waste of resources.
You need to work on something that brings value. If in doubt, ask leads in Discord about the problems to work on.

@CTAPCKPIM
Copy link
Contributor Author

  • goal: redo the task

@CTAPCKPIM CTAPCKPIM changed the title fix(ui): corrects the display of the date on price chart feat(ui): implements tooltip for the FDM field Dec 29, 2023
@georgeciubotaru georgeciubotaru changed the title feat(ui): implements tooltip for the FDM field [DRAFT] feat(ui): implements tooltip for the FDM field Dec 29, 2023
Copy link

github-actions bot commented Dec 29, 2023

Hey there! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

No release type found in pull request title "[DRAFT] feat: tooltip component". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - fix
 - feat
 - ci
 - chore
 - docs
 - test
 - release

@CTAPCKPIM
Copy link
Contributor Author

CTAPCKPIM commented Dec 29, 2023

@georgeciubotaru

  • Check pls

PR size is 4 hours, if possible, let's merge, and in the next PR I will correct the display tooltip e.t.

@CTAPCKPIM CTAPCKPIM requested a review from zolotokrylin January 2, 2024 09:39
@CTAPCKPIM CTAPCKPIM changed the title [DRAFT] feat(ui): implements tooltip for the FDM field [DRAFT] feat: implements tooltip component Jan 2, 2024
@@ -0,0 +1,65 @@
<script>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need for this file

Comment on lines 47 to 57
mixin information-circle
svg(width="16" height="16" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg")
circle(cx="12" cy="12" r="10" fill="lightblue")
text(
x="50%"
y="50%"
text-anchor="middle"
alignment-baseline="central"
font-size="14"
fill="white"
) i
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no in this scope

@@ -2,7 +2,7 @@
"editor.formatOnSave": true,
"editor.defaultFormatter": "esbenp.prettier-vscode",
"editor.codeActionsOnSave": {
"source.fixAll.eslint": true
"source.fixAll.eslint": "explicit"
Copy link
Contributor

@georgeciubotaru georgeciubotaru Jan 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert this

Comment on lines 1 to 38
.tooltip
padding: calc(#{theme('spacing.6')} - 1px)
@apply block z-10 text-p2 text-gray-5 rounded-base-radius normal-case min-w-[250px] max-w-[346px] w-fit

&-arrow
position: absolute
height: 9px
width: 12px
@apply text-gray-70 stroke-gray-40

svg
position: relative
display: block
transform-origin: 50% 50%

&[data-placement^='top'] > &-arrow
bottom: -9px

svg
transform: rotate(0deg)

&[data-placement^='bottom'] > &-arrow
top: -9px

svg
transform: rotate(180deg)

&[data-placement^='left'] > &-arrow
right: -10.5px

svg
transform: rotate(-90deg)

&[data-placement^='right'] > &-arrow
left: -10.5px

svg
transform: rotate(90deg)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please check so the styles will match the Figma Design

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

@georgeciubotaru georgeciubotaru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see my comments.

@georgeciubotaru georgeciubotaru changed the title [DRAFT] feat: implements tooltip component [DRAFT] feat: tooltip component Jan 3, 2024
@CTAPCKPIM
Copy link
Contributor Author

@georgeciubotaru

  • Can you check and if need to correct my pls

@CTAPCKPIM
Copy link
Contributor Author

@georgeciubotaru

  • Done, check pls

@zolotokrylin
Copy link
Member

@georgeciubotaru

1 similar comment
@zolotokrylin
Copy link
Member

@georgeciubotaru

@zolotokrylin zolotokrylin requested a review from angelicawill May 18, 2024 10:35
Copy link

pr-time-tracker bot commented May 18, 2024

@georgeciubotaru
@CTAPCKPIM
⚠️⚠️⚠️
You must submit the time spent on this PR.
⚠️⚠️⚠️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants