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
[DDW-992] Fix position of popup on syncing screen #2921
Conversation
@marcin-mazurek Good catch! This definitely should be fixed but IMO it should be done as part of other ticket. This is issue of the |
@alexander-rukin I think there is a separate ticket for the spinner bug. In this PR I didn't change anything regarding it. |
aaahh... i was checking completely different thing sorry (( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@@ -1187,7 +1187,7 @@ | |||
"wallet.summary.transactionsList.syncingTransactionsMessage": "ウォレットのトランザクション履歴は現在ブロックチェーンと同期中です。", | |||
"wallet.summary.transactionsList.todayLabel": "今日", | |||
"wallet.summary.transactionsList.yesterdayLabel": "昨日", | |||
"wallet.token.picker.addButtonLabel": "キャンセル", | |||
"wallet.token.picker.addButtonLabel": "追加する", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agreed with Szymon to add this tiny fix to address the issue from user report: #2931
Slack thread: https://input-output-rnd.slack.com/archives/GGKFXSKC6/p1648039378845489, https://input-output-rnd.slack.com/archives/CBECZ2K47/p1648025188531639
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Great work @szymonmaslowski. Tested on 21363
# Conflicts: # CHANGELOG.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @szymonmaslowski !
This PR fixes the size of space between question mark icon and the tooltip.
There is no need for QAs to test those changes.
This PR also fixes the incorrect "Add" Japanese translation in token picker (DDW-1035).
Screenshots
Incorrect translation fixed:
Testing Checklist
Review Checklist
Basics
input-output-hk/daedalus-dev
andinput-output-hk/daedalus-qa
assigned as PR reviewersrun Chromatic
label to PR to trigger the run)release-vNext
,feature
/bug
/chore
,WIP
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review