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

Portfolio setting dropdown text doesn't have a smooth transition when changing #7946

Closed
srirambv opened this issue Aug 24, 2023 · 1 comment · Fixed by #7950
Closed

Portfolio setting dropdown text doesn't have a smooth transition when changing #7946

srirambv opened this issue Aug 24, 2023 · 1 comment · Fixed by #7950
Assignees
@StephenHeaps
Labels
bug Epic: Brave Wallet QA Pass - iPhone X QA/Yes release-notes/exclude
Milestone
1.57

Comments

@srirambv
Copy link
Contributor

Description:

Portfolio setting dropdown text doesn't have a smooth transition when changing

Steps to Reproduce

  1. Open Portfolio setting
  2. Change the setting from dropdown
  3. Text updates transition is janky
  4. Update Sort assets to A to Z
  5. Click on reset
  6. Drop down text only shows Hig... instead of High to Low

Actual result:

Dropdown.text.MP4

Expected result:

Text update should be smooth and not janky

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easy

Brave Version:

  • Can you reproduce this issue with the most recent build from TestFlight? Yes
  • Can you reproduce this issue with the previous version of the current build from TestFlight? NA
  • Can you reproduce this issue with the current build from AppStore? NA

Device details:

All

Website problems only:

  • did you check with Brave Shields down? NA
  • did you check in Safari/Firefox (WkWebView-based browsers)? NA

Additional Information

cc: @nuo-xu @StephenHeaps
@srirambv srirambv added this to Web3 Aug 24, 2023
This was referenced Aug 24, 2023
Closed
Closed
@StephenHeaps StephenHeaps mentioned this issue Aug 24, 2023
Merged
11 tasks
@StephenHeaps StephenHeaps added this to the 1.57 milestone Aug 24, 2023
@StephenHeaps StephenHeaps moved this to In Review in Web3 Aug 24, 2023
StephenHeaps added a commit that referenced this issue Aug 24, 2023
@StephenHeaps
Fix #7947, #7944, #7948, #7946: Portfolio Fixes (#7950)
98c7464 
* Hide/Remove Filecoin accounts from Portfolio & NFT filters until they are supported.

* Fix `Network Logo` and `Hide Unowned` NFT Filters & Display Settings toggles not resetting to defaults when `Reset` tapped

* Disable `Reset` button on Filters & Display Settings when filters are showing defaults.

* Fix Filters & Display Settings Picker text truncated, janky animation when switching selection.
@github-project-automation github-project-automation bot moved this from In Review to Done in Web3 Aug 24, 2023
StephenHeaps added a commit that referenced this issue Aug 24, 2023
@StephenHeaps
Fix #7947, #7944, #7948, #7946: Portfolio Fixes (#7950)
c5a8cdb 
* Hide/Remove Filecoin accounts from Portfolio & NFT filters until they are supported.

* Fix `Network Logo` and `Hide Unowned` NFT Filters & Display Settings toggles not resetting to defaults when `Reset` tapped

* Disable `Reset` button on Filters & Display Settings when filters are showing defaults.

* Fix Filters & Display Settings Picker text truncated, janky animation when switching selection.
@srirambv
Copy link
Contributor Author

Verification passed on iPhone 13 with iOS 16.6 running 1.57 (23.8.28.22)

  • Verified steps from issue description
  • Verified portfolio settings dropdown text change has a smooth transition
7946.MP4

arthuredelstein pushed a commit to brave/brave-core that referenced this issue Feb 13, 2024
@StephenHeaps
Fix brave/brave-ios#7947, brave/brave-ios#7944, brave/brave-ios#7948, b…
40d28b4 
…rave/brave-ios#7946: Portfolio Fixes (brave/brave-ios#7950)

* Hide/Remove Filecoin accounts from Portfolio & NFT filters until they are supported.

* Fix `Network Logo` and `Hide Unowned` NFT Filters & Display Settings toggles not resetting to defaults when `Reset` tapped

* Disable `Reset` button on Filters & Display Settings when filters are showing defaults.

* Fix Filters & Display Settings Picker text truncated, janky animation when switching selection.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@StephenHeaps StephenHeaps

Labels
bug Epic: Brave Wallet QA Pass - iPhone X QA/Yes release-notes/exclude
Projects
Web3
Archived in project
Milestone
1.57
Development

Successfully merging a pull request may close this issue.

Fix #7947, #7944, #7948, #7946: Portfolio Fixes brave/brave-ios
2 participants
@StephenHeaps @srirambv