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

Timeline composer layout loop fix #6743

Merged
merged 4 commits into from
Sep 20, 2022
Merged

Conversation

ismailgulek
Copy link
Contributor

@ismailgulek ismailgulek commented Sep 19, 2022

Fixes #6734
Fixes #6735

@github-actions
Copy link

github-actions bot commented Sep 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/wvKBbd

@codecov
Copy link

codecov bot commented Sep 19, 2022

Codecov Report

Base: 9.96% // Head: 9.95% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (975ef78) compared to base (a5804a3).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #6743      +/-   ##
===========================================
- Coverage     9.96%    9.95%   -0.01%     
===========================================
  Files         1484     1484              
  Lines       150507   150561      +54     
  Branches     60751    60769      +18     
===========================================
+ Hits         14992    14994       +2     
- Misses      134961   135013      +52     
  Partials       554      554              
Impacted Files Coverage Δ
Riot/Modules/Room/RoomViewController.m 0.00% <0.00%> (ø)
...estion/Coordinator/UserSuggestionCoordinator.swift 0.00% <0.00%> (ø)
.../Coordinator/UserSuggestionCoordinatorBridge.swift 0.00% <0.00%> (ø)
...Modules/Home/AllChats/AllChatsViewController.swift 28.39% <0.00%> (-0.22%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sonarcloud
Copy link

sonarcloud bot commented Sep 19, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Member

@stefanceriu stefanceriu left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@ismailgulek ismailgulek merged commit f201ea5 into develop Sep 20, 2022
@ismailgulek ismailgulek deleted the ismail/6734_composer_fix branch September 20, 2022 08:40
ismailgulek added a commit that referenced this pull request Sep 20, 2022
* Device manager: add user sessions overview current session strings.

* Create UserSessionCardView to display session informations.

* UserSessionsOverview: Handle current session section view.

* Prepare for new sprint

* Dismiss the launching screen when clearing cache.

* Fix crash when previewing a room.

* Fix typo in empty state

* Code review fixes

* Update swift-ogg and add changelog

* Fix crash on logout from too much wrong pin codes

* Update changelog to original issue

* Disable unnecessary network breadcrumbs in Sentry

* Changelog

* Fix login crash on Xcode 14 builds

Fixes #6722

* Put the session creator on the main actor instead.

This covers all login and registration options.

* Only disable the composer if the room is yet to be created.

* Add missing source views for iPad.

* New App Layout: fixed rooms list doesn't update after log out and log into another account

* New layout: Fixed low priority rooms titles obscured by bottom bar when side scrolling

* Fix replied event content parsing for nested replies

* Fixed failed to join room (was not legal room)

* Timeline composer layout loop fix (#6743)

* Publish suitable view height for user suggestions

* Update container view height

* Add changelog

* Fix user suggestions layout for pre iOS 16

* version++

Co-authored-by: SBiOSoftWhare <[email protected]>
Co-authored-by: Doug <[email protected]>
Co-authored-by: Germain Souquet <[email protected]>
Co-authored-by: Stefan Ceriu <[email protected]>
Co-authored-by: Aleksandrs Proskurins <[email protected]>
Co-authored-by: David Langley <[email protected]>
Co-authored-by: aringenbach <[email protected]>
Co-authored-by: aringenbach <[email protected]>
Co-authored-by: Andy Uhnak <[email protected]>
Co-authored-by: Johannes Marbach <[email protected]>
Co-authored-by: Gil Eluard <[email protected]>
Co-authored-by: Gil Eluard <[email protected]>
Co-authored-by: gulekismail <[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.

App locks up attempting to show the emoji keyboard App locks up when tapping the message composer
2 participants