[fix/collapsible-progress-bar-position-flex] Collapse Progress Bar in Detail View #911
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… in the safeArea - ProgressSummarizer: make .setNeedsUpdate() public to allow triggering updates to observers - ClientRootViewController: - add .allowProgressBarAutoCollapse property that allows auto collapse even with "persistent" messages - animate CollapsibleProgressBar alongside UINavigationController transitions - rename progressBarHeightConstraint to progressBarBottomConstraint for clarity
|
|
hosy
approved these changes
Feb 28, 2021
(1)Problem is fixed in portrait mode. If the device is in landscape, the |
…e progress bar at the wrong offset - adapt the bugfix to also work on iOS 12
|
@jesmrec Thanks! (1) should now be fixed. I've tested it with iOS 12 and iOS 14.4 in the Simulator, rotating before and while viewing content. |
…psible-progress-bar-position
…github.com/owncloud/ios-app into fix/collapsible-progress-bar-position-flex
|
Fixed. Tested also with iOS14.4 and iOS13.4. Approved |
jesmrec
approved these changes
Mar 1, 2021
delete-merged-branch
bot
deleted the
fix/collapsible-progress-bar-position-flex
branch
hosy
added a commit
that referenced
this pull request
Mar 3, 2021
* Preparing new version: changed version and build number to 11.5.2 and 188 * Add Themeable support to the Acknowledgements (#900) * [feature/flat-mdm-settings] Support for MDM setting hierarchies with flat keys (#904) * - automatic termination + relaunch by notification on MDM changes push - fix indentation * - react to setting changes pushed via MDM differently, in a two-step process - first notify the user about the changes - if the user taps the notification, schedule a relaunch notification and then quit - if the user taps the relaunch notification, the app is launched - request authorization to post notifications if needed * [fix/pdf-thumbnailview-position] PDF thumbnail view position on the iPad (#905) * Fixing the thumbnail view position on the iPad in portrait view - added orientation changed notification to recreate the constrains, when the device was rotated * added changelog issue * Calens changelog updated * fixed QA finding (jumpy thumbnail view on iPhone) * isPortrait mode is not working and removed it with size class Co-authored-by: hosy <[email protected]> * [fix/collapsible-progress-bar-position-flex] Collapse Progress Bar in Detail View (#911) * Accessing hyper links in PDF view (#913) * Updated release notes for version 11.5.2 * Calens changelog updated * [feature/owncloud-online-screentshots] Create Screenshots for Branded Builds with Fastlane (#912) * updated In-App release notes with marketing fix * moved calens changelog files into release folder * Calens changelog updated * updated SDK to latest master commit Co-authored-by: Felix Schwarz <[email protected]> Co-authored-by: hosy <[email protected]> Co-authored-by: Michael Neuwert <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR fixes the misplaced Collapsible Progress Bar in Detail View (originally #907 by @hosy).
Related Issue
#906 #907
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:
QA