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.

Sign up for GitHub

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

Jump to bottom

NFTs: The disappearing toast reappears when you navigate away and then back #17385

Closed
darkwing opened this issue Jan 24, 2023 · 0 comments · Fixed by #17392
Closed

NFTs: The disappearing toast reappears when you navigate away and then back #17385

darkwing opened this issue Jan 24, 2023 · 0 comments · Fixed by #17392
Assignees
@darkwing
Labels
area-NFTs team-extension-ux DEPRECATED: please use "team-wallet-ux" label instead type-bug

Comments

@darkwing
Copy link
Contributor

Describe the bug

The logic I used to auto-hide the toast doesn't permanently autohide it. That's a problem.

Steps to reproduce

  1. Add an NFT
  2. See the "Added successfully" toast
  3. Wait 5 seconds
  4. Click into the NFT and then back
  5. See the "Added successfully" toast again

Error messages or log output

No response

Version

10.23

Build type

None

Browser

Chrome

Operating system

MacOS

Hardware wallet

No response

Additional context

No response
@darkwing darkwing added type-bug area-NFTs team-extension-client team-extension-ux DEPRECATED: please use "team-wallet-ux" label instead labels Jan 24, 2023
@darkwing darkwing self-assigned this Jan 24, 2023
darkwing added a commit that referenced this issue Jan 24, 2023
@darkwing
Fix #17385 - Provide autohide callback for ActionMessage
76e8306
@darkwing darkwing mentioned this issue Jan 24, 2023
Merged
8 tasks
darkwing added a commit that referenced this issue Jan 25, 2023
@darkwing
Fix #17385 - Provide autohide callback for ActionMessage
5a6fb66
darkwing added a commit that referenced this issue Jan 27, 2023
@darkwing
Fix #17385 - Provide autohide callback for ActionMessage (#17392)
b21e0df 
* Fix #17385 - Provide autohide callback for ActionMessage

* Add autohide to the removeal message
409H added a commit to 409H/metamask-extension that referenced this issue Jan 27, 2023
@409H
Merge remote-tracking branch 'origin/develop' into feature/toggle-eth…
26c7253 
…sign

* origin/develop: (142 commits)
  Use network provider state, instead of CurrencyRateController state, to select 'nativeCurrency' (MetaMask#17450)
  [e2e]Add e2e test for deleting custom network (MetaMask#17254)
  Fix MetaMask#17362 - Ensure NFT icons are correct (MetaMask#17440)
  Fix MetaMask#17385 - Provide autohide callback for ActionMessage (MetaMask#17392)
  button link housekeeping (MetaMask#16885)
  Bump lavapack version to 5 (MetaMask#17431)
  feature: convert shared/modules/contract-utils.test.js to typescript (MetaMask#17435)
  Refactor confirm page container component from class to functional component (MetaMask#16907)
  Test parameter change cache miss (MetaMask#17346)
  Document callback-style background calls as deprecated (MetaMask#17376)
  Improving getMemoizedMetadataContractName selector (MetaMask#17432)
  [GridPlus] Bumps `gridplus-sdk` to v2.4.1 (MetaMask#16847)
  Put hardware wallets behind an HARDWARE_WALLETS_MV3 flag (MetaMask#17354)
  Consolidated all component import paths (MetaMask#17368)
  updated new links for the stories (MetaMask#17369)
  Fix MetaMask#17388 - Remove dismiss button from NFT notification (MetaMask#17389)
  Remove a notification for falling back from STX to regular swaps (MetaMask#17374)
  Feat/15086/add banner severities component (MetaMask#17307)
  fix icon names (MetaMask#17391)
  Fix Playwright install step (MetaMask#17415)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@darkwing darkwing

Labels
area-NFTs team-extension-ux DEPRECATED: please use "team-wallet-ux" label instead type-bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #17385 - Provide autohide callback for ActionMessage MetaMask/metamask-extension
1 participant
@darkwing