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.

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

Fix inconsistent bars state when scrolling #2733

Merged
merged 1 commit into from
Apr 12, 2024
Merged

Conversation

dus7
Copy link
Contributor

@dus7 dus7 commented Apr 12, 2024

Task/Issue URL: https://app.asana.com/0/414235014887631/1207045192875790/f
Tech Design URL:
CC:

Description:

When scrolling content which almost fits in the screen height, top and bottom bar could end up in an inconsistent state with alpha channel set on the controls. This makes sure layout pass is always done after all the properties are set. More details can be found in a subtask.

Steps to test this PR:

  1. Go to https://kitese.duckdns.org:8080/
  2. Rotate device to landscape
  3. Scroll quite rapidly to hide bars
  4. Bars should become visible again by the scroll finishes with alpha on controls set to 1.0 (see video in the task for details)

Internal references:

Software Engineering Expectations
Technical Design Template

@dus7 dus7 marked this pull request as ready for review April 12, 2024 13:58
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Was able to reproduce easily on the old version but couldn't at all with this.

@dus7 dus7 merged commit 398bed3 into main Apr 12, 2024
19 checks passed
@dus7 dus7 deleted the mariusz/bars-animation-glitch branch April 12, 2024 15:12
samsymons added a commit that referenced this pull request Apr 15, 2024
# By Daniel Bernal (33) and others
# Via Mariusz Śpiewak (5) and others
* main: (245 commits)
  Fix inconsistent bars state when scrolling (#2733)
  fix tests (#2732)
  Release 7.115.0-4 (#2729)
  Fix VPN denial prompt loop (#2728)
  Small UI Fixes for subscriptions (#2690)
  SPM updated: SwiftSoup, Lottie, ZIPFoundation (#2724)
  Release 7.115.0-3 (#2727)
  VPN: Specific TunnelController start failure reporting (#2714)
  update bsk dependency (#2725)
  Subscriptions: Fix thread issue on Subscription Restore (#2719)
  Manage ‘Stale’ PRs (#2723)
  maestro: hide dax dialogs if visible and cancel keyboard after fireproof (#2695)
  Remove timezone offset from the VPN server object (#2701)
  Reverting accidental push to main (#2718)
  Add SubscriptionContainerViewModel and
  Manually hide loader + Pixel (#2687)
  Release 7.115.0-2 (#2712)
  soft revert history suggestions (#2711)
  Bring back accessibility identifiers for onboarding buttons (#2709)
  BSK release 133.1.0 (#2708)
  ...

# Conflicts:
#	Core/Pixel.swift
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants