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

Filtering fixes #6184

Merged
merged 4 commits into from
May 19, 2022
Merged

Filtering fixes #6184

merged 4 commits into from
May 19, 2022

Conversation

stefanceriu
Copy link
Member

Fixes

  • home screen shrinking too much on opening the keyboard
  • the search bar cancel button not clearing the search
  • the homescreen filtering not being cleared properly when switching tabs

@stefanceriu stefanceriu requested review from a team, pixlwave and phloux and removed request for a team May 19, 2022 13:01
@stefanceriu stefanceriu marked this pull request as ready for review May 19, 2022 13:01
@stefanceriu stefanceriu requested a review from Anderas May 19, 2022 13:18
@@ -114,7 +114,7 @@ - (void)viewWillAppear:(BOOL)animated
[recentsDataSource setDelegate:self andRecentsDataSourceMode:RecentsDataSourceModeHome];

// Reset filtering on the shared data source when switching tabs
[self searchBarCancelButtonClicked:self.recentsSearchBar];
[recentsDataSource searchWithPatterns:nil];
Copy link
Member Author

Choose a reason for hiding this comment

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

This basically makes it the same as all the other tabs in which it worked properly.

stackView.axis = .vertical
stackView.distribution = .fill
stackView.alignment = .fill

view.vc_addSubViewMatchingParentSafeArea(stackView)
view.addSubview(stackView)
NSLayoutConstraint.activate([stackView.topAnchor.constraint(equalTo: self.view.safeAreaLayoutGuide.topAnchor),
Copy link
Member Author

Choose a reason for hiding this comment

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

Only caring about the top safe area layout guide as that's the place we show the version banner. The bottom one will be handled by the real homeVC.

@@ -0,0 +1 @@
Fixed home screen shrinking too much on opening the keyboard.
Copy link
Contributor

Choose a reason for hiding this comment

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

Will this differently named changelog be picked up?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep, it's part of our normal naming scheme - https://github.com/matrix-org/matrix-ios-sdk/blob/develop/CONTRIBUTING.md#changelog -> pr-[PRNumber].type for a PR with no related issue

@github-actions
Copy link

github-actions bot commented May 19, 2022

📱 Scan the QR code below to install the build for this PR.
🔒 This build is for internal testing purpose. Only devices listed in the ad-hoc provisioning profile can install Element Alpha.

QR code

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/wyqwLG

@stefanceriu stefanceriu changed the base branch from develop to hotfix/1.8.15 May 19, 2022 13:45
@stefanceriu stefanceriu changed the base branch from hotfix/1.8.15 to develop May 19, 2022 13:46
@stefanceriu stefanceriu changed the base branch from develop to hotfix/1.8.15 May 19, 2022 13:46
@stefanceriu stefanceriu merged commit 63ee678 into hotfix/1.8.15 May 19, 2022
@stefanceriu stefanceriu deleted the stefan/filtering-fixes branch May 19, 2022 13:49
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