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

Show tooltips on hovering over icons with no labels #3711

Merged
merged 5 commits into from
Dec 20, 2023
Merged

Conversation

xpaczka
Copy link
Contributor

@xpaczka xpaczka commented Dec 18, 2023

Resolves #3172

What has been done:

Added tooltips to the following icons:

  • Send
  • Swap
  • Filters
  • Connect dapp
  • Disconnect dapp

Also updates _locales files

Zrzut ekranu 2023-12-18 o 17 13 17
Zrzut ekranu 2023-12-18 o 17 13 02
Zrzut ekranu 2023-12-18 o 17 12 55

Latest build: extension-builds-3711 (as of Wed, 20 Dec 2023 12:10:19 GMT).

@xpaczka xpaczka self-assigned this Dec 18, 2023
@xpaczka xpaczka marked this pull request as ready for review December 18, 2023 16:13
jagodarybacka
jagodarybacka previously approved these changes Dec 19, 2023
Copy link
Contributor

@jagodarybacka jagodarybacka left a comment

Choose a reason for hiding this comment

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

Looks good, just one thing - up to you and @VladUXUI what we want to do here

@@ -804,6 +806,7 @@
},
"topMenu": {
"showCurrentDappConnection": "Show current website connection",
"currentDappConnection": "Current website connection",
Copy link
Contributor

Choose a reason for hiding this comment

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

I would change it to have it in one line - Dapp connection or something like that? cc @VladUXUI
image

@jagodarybacka
Copy link
Contributor

Ok let's merge and fix later if needed

@jagodarybacka jagodarybacka merged commit 3ef6e81 into main Dec 20, 2023
6 checks passed
@jagodarybacka jagodarybacka deleted the hover-tooltip branch December 20, 2023 13:56
@xpaczka xpaczka mentioned this pull request Dec 27, 2023
xpaczka added a commit that referenced this pull request Dec 29, 2023
## What's Changed
* Enable swaps on Sepolia testnet by @michalinacienciala in
#3710
* Prevent read-only address submission when validating input by @xpaczka
in #3705
* Support video avatars by @xpaczka in
#3700
* Replace `customStyle` property with `style` by @xpaczka in
#3715
* Show tooltips on hovering over icons with no labels by @xpaczka in
#3711
* Minor UI issues by @xpaczka in
#3716
* Fix hiding unverified assets by @jagodarybacka in
#3708
* v0.54.0 by @xpaczka in
#3706
* chore(ui): typo fix by @IssouChancla in
#3717
* Switch e2e tests run on Goerli to Sepolia by @michalinacienciala in
#3719

## New Contributors
* @xpaczka made their first contribution in
#3705
* @IssouChancla made their first contribution in
#3717

**Full Changelog**:
v0.54.0...v0.55.0

Latest build:
[extension-builds-3720](https://github.com/tahowallet/extension/suites/19343332470/artifacts/1136165985)
(as of Wed, 27 Dec 2023 09:28:52 GMT).
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.

Show tooltips on hovering over icons with no labels
2 participants