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: [iOS] On Course "Home" tab the row height #24

Merged
merged 3 commits into from
Jul 17, 2024

Conversation

forgotvas
Copy link
Collaborator

@forgotvas forgotvas commented Jul 17, 2024

[iOS] On Course "Home" tab the row height changes after a while openedx#483

@forgotvas forgotvas requested review from rnr and saeedbashir July 17, 2024 13:48
@@ -36,8 +45,7 @@ public extension UIApplication {
}

var windowInsets: UIEdgeInsets {
guard let windowScene = UIApplication.shared.connectedScenes.first as? UIWindowScene,
let window = windowScene.windows.first else {
guard let window = windows?.first else {
Copy link
Collaborator

@rnr rnr Jul 17, 2024

Choose a reason for hiding this comment

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

could we use here guard let window = window - property added on line 18?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

Copy link

@saeedbashir saeedbashir left a comment

Choose a reason for hiding this comment

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

It would be better if you can mention opededX PR or Issue it addresses in the PR description

@forgotvas
Copy link
Collaborator Author

It would be better if you can mention opededX PR or Issue it addresses in the PR description

done

@rnr rnr merged commit 0b41009 into 2U/develop Jul 17, 2024
3 checks passed
@rnr rnr deleted the 2U/fix/download-states branch July 17, 2024 20:34
rnr added a commit that referenced this pull request Oct 1, 2024
fix: [iOS] On Course "Home" tab the row height
rnr added a commit that referenced this pull request Oct 8, 2024
* Merge pull request #23 from edx/small-fix-for-downloading-cancelling

fix: fixes for downloading

* chore: fix for Xcode 16 and after merge

* Merge pull request #24 from edx/2U/fix/download-states

fix: [iOS] On Course "Home" tab the row height

* fix: after merge, deleted IAP part

fix: [iOS] On Course "Home" tab the row height

* Merge pull request #25 from edx/2U/feat/primary-horizontal

feat: Landscape mode Improvement

* fix: removed IAP part

* chore: remove snack bar error for course dates info API on course home (#27)

* Merge pull request #28 from shafqat-muneer/Shafqat/LEARNER-10020-ErrorHandling

feat: Course Level Error Handling for Empty States

* chore: remove IAP part after merging

---------

Co-authored-by: Anton Yarmolenko <[email protected]>
Co-authored-by: Saeed Bashir <[email protected]>
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.

3 participants