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
[VDG] Privacy Ring - Remove Carousel #12541
[VDG] Privacy Ring - Remove Carousel #12541
Conversation
ichthus1604
commented
Feb 22, 2024
- Removes Carousel UI element from Privacy Ring view
- Fade in animation has been temporarily disabled to prevent the glitches. Will restore it in a future PR.
- Fixes [UI] Privacy Progress: Light flickering when hovering out of a coin #12537
- Attempt to fix [UI] Glitch above privacy percentage in privacy progress dialog #12495. I could not reproduce this issue so I'm not sure 100% that it's fixed in this PR. @yahiheb can you confirm?
PreviewItems.Add(PrivacyTile); | ||
// Show PrivacyTile info when SelectedItem is null | ||
Observable.Return(Unit.Default) | ||
.Delay(TimeSpan.FromMilliseconds(750)) // Wait for Ring animation to render |
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.
Why 750 ?
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.
Yeah, this is not correct. The Delay should take into account the number of segments that need to be rendered.
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.
Left a TODO comment and disabled it for now
It fixes #12537 |
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 breaks labels layout:
screencast.2024-02-22.20-22-18.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.
This indeed fixes both issues.
Fixed. |