Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

Improve discreet mode #716

Merged
merged 5 commits into from
May 23, 2023
Merged

Improve discreet mode #716

merged 5 commits into from
May 23, 2023

Conversation

mvaivre
Copy link
Member

@mvaivre mvaivre commented May 18, 2023

Hopefully fixes #677

@mvaivre mvaivre requested a review from nop33 May 18, 2023 13:00
Copy link
Member

@nop33 nop33 left a comment

Choose a reason for hiding this comment

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

I think it's much clearer now, good job 👍

  1. Shall we make this PR against next?
  2. I found a missed "3-dot" element:
    image
  3. In the Address Details modal, the percentage seems to be stack in loading mode. Not sure if it's related to these changes
    image

locales/en-US/translation.json Show resolved Hide resolved
@mvaivre mvaivre changed the base branch from master to next May 23, 2023 09:46
@mvaivre mvaivre force-pushed the improve-discreet-mode branch from c85aa9a to a021f3d Compare May 23, 2023 13:49
@mvaivre mvaivre requested a review from nop33 May 23, 2023 14:03
@mvaivre mvaivre force-pushed the improve-discreet-mode branch from b4182c3 to fa0bee0 Compare May 23, 2023 14:18
@@ -421,5 +422,5 @@
"Your addresses": "Ihre Adressen",
"Your CSV file has been generated successfully.": "Your CSV file has been generated successfully.",
"Your CSV file is being compiled in the background.": "Your CSV file is being compiled in the background.",
"Your Wallet": "Ihre Wallet"
"Your Wallet": "Your Wallet"
Copy link
Member

Choose a reason for hiding this comment

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

This translation was actually correct 😅

@@ -421,5 +422,5 @@
"Your addresses": "Your addresses",
"Your CSV file has been generated successfully.": "Your CSV file has been generated successfully.",
"Your CSV file is being compiled in the background.": "Your CSV file is being compiled in the background.",
"Your Wallet": "Dompet Anda"
"Your Wallet": "Your Wallet"
Copy link
Member

Choose a reason for hiding this comment

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

Same here

@@ -421,5 +422,5 @@
"Your addresses": "Your addresses",
"Your CSV file has been generated successfully.": "Your CSV file has been generated successfully.",
"Your CSV file is being compiled in the background.": "Your CSV file is being compiled in the background.",
"Your Wallet": "Su billetera"
"Your Wallet": "Your Wallet"
Copy link
Member

Choose a reason for hiding this comment

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

I guess all that was needed was to capitalize the word billetera...

@@ -421,5 +422,5 @@
"Your addresses": "Your addresses",
"Your CSV file has been generated successfully.": "Your CSV file has been generated successfully.",
"Your CSV file is being compiled in the background.": "Your CSV file is being compiled in the background.",
"Your Wallet": "A sua carteira"
"Your Wallet": "Your Wallet"
Copy link
Member

Choose a reason for hiding this comment

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

Not sure how to capitalize this one tho, so I'd leave it like this.

@mvaivre mvaivre force-pushed the improve-discreet-mode branch from fa0bee0 to 78062ba Compare May 23, 2023 14:55
@mvaivre mvaivre merged commit 95fa7c1 into next May 23, 2023
@mvaivre mvaivre deleted the improve-discreet-mode branch May 23, 2023 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Discreet mode is not clear
2 participants