-
Notifications
You must be signed in to change notification settings - Fork 857
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
Fixed account switch transition #13968
Conversation
38c48a5
to
9ff1e5c
Compare
android/java/org/chromium/chrome/browser/crypto_wallet/adapters/WalletCoinAdapter.java
Outdated
Show resolved
Hide resolved
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.
++
notifyItemRangeChanged( | ||
previousSelectedPos, selectedAccountPosition - previousSelectedPos); | ||
} else { | ||
notifyItemRangeChanged( | ||
selectedAccountPosition, previousSelectedPos - selectedAccountPosition + 1); |
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.
[Adding the comment here to be marked as resolved]
Unfortunately, this still happens (video at 15 second and at the end), I guess an ideal solution could be to use custom checkbox instead of ImageView. Probably discuss with @srirambv or @jamesmudgett to make sure it's good enough to go or requires more polishing.
selector-issue.mp4
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.
due to account switch transition is a minor p4 issue so would not merge that issue, Will only merge brave/brave-browser#23558 after review.
00a1fdc
to
671f767
Compare
671f767
to
a274cc6
Compare
Resolves brave/brave-browser#23558
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
,npm run lint
,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Screenrecorder-2022-06-28-21-50-46-984.mp4