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

TOK-579: change backer est. reward tooltip text #641

Merged
merged 4 commits into from
Feb 12, 2025

Conversation

jurajpiar
Copy link
Contributor

Updates the backer rewards estimated rewards tooltip text.
Screenshot 2025-02-04 at 12 01 26

Note

The table tooltip is not fully shown. This is outside of the scope of the ticket, unless requested to be in.

Task

@jurajpiar jurajpiar requested a review from a team February 4, 2025 12:05
Copy link

github-actions bot commented Feb 4, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

@jurajpiar jurajpiar force-pushed the TOK-579/estimated_reward_tooltip_text branch from 98c394f to 0fc9ac8 Compare February 4, 2025 12:10
Copy link
Contributor

@franciscotobar franciscotobar left a comment

Choose a reason for hiding this comment

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

I left a comment, everything else seems to be good.

@antomor antomor changed the title chore(cr): change backer est. reward tooltip text TOK-579: change backer est. reward tooltip text Feb 5, 2025
Copy link
Collaborator

@antomor antomor left a comment

Choose a reason for hiding this comment

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

@jurajpiar couldn't we create a single component for the tooltip being it the same?
Also, for the tooltip visibility, I'd change the overflow of the table to set it as visible

@jurajpiar jurajpiar force-pushed the TOK-579/estimated_reward_tooltip_text branch from 0fc9ac8 to 0277640 Compare February 6, 2025 14:14
@jurajpiar jurajpiar force-pushed the TOK-579/estimated_reward_tooltip_text branch from 0277640 to b8acbd9 Compare February 6, 2025 14:19
Copy link
Contributor

@franciscotobar franciscotobar left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

Choose a reason for hiding this comment

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

Apparently the whole text isn't exactly the same, but it's slightly different. Could you please double-check it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@jurajpiar jurajpiar requested a review from antomor February 7, 2025 11:21
@antomor antomor merged commit 83adbab into develop Feb 12, 2025
9 checks passed
@antomor antomor deleted the TOK-579/estimated_reward_tooltip_text branch February 12, 2025 13:39
jurajpiar added a commit that referenced this pull request Feb 14, 2025
* chore(cr): change backer est. reward tooltip text

* fix: make overflow visible

* chore(cr): change backer est. reward tooltip text

* fix: remove unused component

---------

Co-authored-by: Antonio <antonio@iovlabs.org>
jurajpiar added a commit that referenced this pull request Feb 17, 2025
* chore(cr): change backer est. reward tooltip text

* fix: make overflow visible

* chore(cr): change backer est. reward tooltip text

* fix: remove unused component

---------

Co-authored-by: Antonio <antonio@iovlabs.org>
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.

4 participants