-
Notifications
You must be signed in to change notification settings - Fork 33
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
Update browser extension screenshots #997
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for oasisprotocol-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
||
If everything goes well, you will see a successful ParaTime transaction in your | ||
account history. | ||
|
||
![Account history](../../images/wallet/ext/account_history.png) | ||
<div className="wallet-extension-img"> | ||
![Account history](../../images/wallet/ext/account_history.png) |
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.
This image doesn't show account history - would need to scroll down a bit
import Tabs from '@theme/Tabs'; | ||
import TabItem from '@theme/TabItem'; | ||
import Tabs from "@theme/Tabs"; | ||
import TabItem from "@theme/TabItem"; | ||
|
||
# ROSE Wallet - Browser Extension |
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.
@matevz Kaja added screenshots, but text and flows would need to be updated too. Do you plan to deduplicate most of the text and flows with Web Wallet? The main difference is layout size and ledger flow has an extra step: ledger2.png
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.
Yes, we should deduplicate as much as possible. I would still keep separate chapters for now though (web, extension, mobile).
1. Click the avatar on the top right corner 2. Pick an account you want to | ||
transfer tokens out of 3. Click "Send" 4. Type in the token amount under | ||
"Amount" 5. Type in the Receiver’s wallet address 6. Click "Next" 7. Check | ||
the "Send Details"; if everything looks good to you, click "Confirm" 8. Wait | ||
for a couple of seconds, and you’ll get a status update of your transaction | ||
9. Go back to the Account page, and you’ll see the transfer has gone through |
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.
Something in your tools broke this again
## Create a New Account | ||
|
||
Enter the **new wallet password**. You will need to enter this password each | ||
time you open the wallet and after some time when the wallet automatically | ||
locks. Click "Next". | ||
|
||
![Create new wallet: Enter password](../../images/wallet/ext/create_wallet.png) | ||
<div className="wallet-extension-img"> |
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.
<div>
elements clutter the otherwise clean markdown content. Is this really needed?
/* Style Chrome extension screenshot images */ | ||
.wallet-extension-img { | ||
max-width: 350px; | ||
} |
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.
Is this really needed? I think all images should be managed by Docusaurus default theme CSS. Also, if we decide to use some more fancy light-box like images plugin in the future.
Fixes: #912