Pinned contracts UX updated #5207
Conversation
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Aniket-Engg
force-pushed
the
pinnedUI
branch
from
1df9bd3
to
38f4c35
Compare
|
| "udapp.deployedContracts": "Deployed/Unpinned Contracts", | ||
| "udapp.deployAndRunClearInstances": "Clear instances list and reset recorder", | ||
| "udapp.deployedContracts": "Deployed Contracts", | ||
| "udapp.deployAndRunClearInstances": "Delete all deployed contracts (including pinned) and reset recorder", |
There was a problem hiding this comment.
I think the word Delete is not correct - I think it should be:
Clear all deployed contract...
There was a problem hiding this comment.
Delete sounds like it will delete them from a public chain - if connected. Of course if the RemixVM is connected the word delete would be correct.
| @@ -108,9 +108,9 @@ | |||
| "udapp.tooltipText13": "Deployed {date}", | |||
|
|
|||
| "udapp._comment_universalDappUI.tsx": "libs/remix-ui/run-tab/src/lib/components/universalDappUI.tsx", | |||
| "udapp.tooltipTextRemove": "Remove from the list", | |||
| "udapp.tooltipTextRemove": "Remove permanently from the list", | |||
There was a problem hiding this comment.
Permanently remove from this list
| "udapp.tooltipTextEdit": "Create a DApp using this contract in the main panel", | ||
| "udapp.tooltipTextPin": "Pin contract", | ||
| "udapp.tooltipTextPin": "Pin contract for select workspace and network to persist after reload", |
There was a problem hiding this comment.
A pinned contract will be persisted after a browser refresh as long as the same Workspace and network are selected.
There was a problem hiding this comment.
this doesn't define the action of button and is more like a sentence. May be we can rephrase this
Aniket-Engg
force-pushed
the
pinnedUI
branch
from
38035da
to
cf5abb7
Compare
Aniket-Engg
force-pushed
the
pinnedUI
branch
from
cf5abb7
to
d57985c
Compare
fixes #5105