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

[milestone/12.0.1] Milestone 12.0.1 #1220

Merged
merged 12 commits into from
Jun 20, 2023
Merged

[milestone/12.0.1] Milestone 12.0.1 #1220

merged 12 commits into from
Jun 20, 2023

Conversation

felix-schwarz
Copy link
Contributor

@felix-schwarz felix-schwarz commented Jun 13, 2023

Description

Version 12.0.1 fixes all findings left over from version 12.0:

Regarding (12):

The string naming that action (Share) is stored in the File Provider UI's Info.plist and therefore can't be changed dynamically at runtime. To keep it branding-friendly, I chose "Share by cloud" instead of "Share via ownCloud", "Share (ownCloud)", which would have required to customize both the Info.plist and the translations for branding.

Related Issue

#1222

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • Added an issue with details about all relevant changes in the iOS documentation repository.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • Added changelog files for the fixed issues in folder changelog/unreleased

@CLAassistant
Copy link

CLAassistant commented Jun 13, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ hosy
❌ felix-schwarz
You have signed the CLA already but the status is still pending? Let us recheck it.

@jesmrec
Copy link
Contributor

jesmrec commented Jun 14, 2023

(1), (7), (9) and (12) are fixed

About (5), i still reproduce it. I notice that a new Loading dialog is visible in streaming mode (cool!), but in download mode:

RPReplay_Final1686743044.MP4

File is downloading in the background bur black screen is there.

CC @felix-schwarz

@delete-merged-branch delete-merged-branch bot deleted the branch master June 14, 2023 13:23
@felix-schwarz felix-schwarz changed the base branch from milestone/12.0 to master June 14, 2023 19:03
@felix-schwarz felix-schwarz changed the base branch from master to milestone/12.0 June 14, 2023 19:05
	- add ability to show/hide navigation bar (with or without animation)
	- add convenience accessor to UIViewController to access the enclosing browserNavigationViewController
- Branding+App: fix SwiftLint warnings
- PDFViewerViewController: hide navigation bar via browserNavigationController rather than navigationController (fix finding (7))
… for laying out the view in a keyboard-responsive manner

- CollectionViewController: add .compressForKeyboard property that layouts the UICollectionView in a keyboard-responsive manner
- ShareViewController: adopt compressForKeyboard
- ClientItemViewController: adopt compressForKeyboard
- fixes finding (1) in #1095
- MediaDisplayViewController: when streaming, show a loading indicator (fixing finding (5) in #1095)
@felix-schwarz felix-schwarz changed the base branch from milestone/12.0 to master June 14, 2023 19:22
…g video files (fixing part 2 of finding (5) in #1095)
@felix-schwarz
Copy link
Contributor Author

felix-schwarz commented Jun 14, 2023

@jesmrec Thanks for the video. The remaining issue from finding (5) should be fixed now, too.

@jesmrec
Copy link
Contributor

jesmrec commented Jun 15, 2023

(5) fixed

@jesmrec
Copy link
Contributor

jesmrec commented Jun 15, 2023

(13) [FIXED]

Feature: Copy/Move
Severity: High

  1. Select any item and Copy or Move
  2. Choose destination directory...
  3. Files

Current: Copy Here / Move here buttons are misplaced and not clickable

Screenshot 2023-06-15 at 09 28 51 Screenshot 2023-06-15 at 09 29 04

Expected: correctly alligned and clickable

iPhone XR v16.5
ebd4c1ba85

EDIT: same behaviour in sharing:

Screenshot 2023-06-15 at 09 38 04 Screenshot 2023-06-15 at 09 40 05

…tton.centerYAnchor

- ClientLocationPicker: prevent usage of .keyboardLayoutGuide to fix finding (13) in #1220 (misplaced select button after navigating to Files view)
- CollectionViewController: use the .safeAreaWithKeyboardAnchorSet for the whole stack view rather than just the collection view
@felix-schwarz
Copy link
Contributor Author

@jesmrec (13) is fixed now, too.

hosy and others added 2 commits June 15, 2023 11:09
- added in-app release notes
- added calens release notes
@jesmrec
Copy link
Contributor

jesmrec commented Jun 15, 2023

Everything fixed

@hosy hosy merged commit b242f83 into master Jun 20, 2023
2 checks passed
@delete-merged-branch delete-merged-branch bot deleted the milestone/12.0.1 branch June 20, 2023 07:13
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.

None yet

4 participants