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.

Sign up for GitHub

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

Jump to bottom

[Calling][Android] Feature RTT #1125

Draft
wants to merge 67 commits into
base: develop
Choose a base branch
from
Draft

[Calling][Android] Feature RTT #1125

wants to merge 67 commits into from

Conversation

pavelprystinka
Copy link
Member

Purpose

  • Feature RTT

Does this introduce a breaking change?

  • Yes
  • No

Pull Request Type

What kind of change does this Pull Request introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Documentation content changes
  • Other... Please describe:

Pull Request Checklist

  • Public API changes
  • Verified for cross-platform
  • Synced with iOS, Web team
  • Internal review completed
  • UI Changes
    • Screen captures included
    • Tested for Light/Dark mode
    • Tested for screen rotation
    • Tested on Tablet and small screen device (5")
    • Include localization changes
  • Tests
    • Memory leak analysis performed
    • Tested on API 21, 26 and latest
    • Tested for background mode
    • Unit Tests Included
    • UI Automated Tests Included

How to Test

  • Open...
  • Click on...

What to Check

Verify that the following are valid

  • ...

Other Information

@pavelprystinka
Add expandable captions layout
7c4cbb4
@pavelprystinka
Make draggable.
146dab4
@pavelprystinka
Rename variables and clipChildren false
3bbe9cd
@pavelprystinka
Variable renamings
b040c5d
@pavelprystinka
Limit drag beyond min and mis height
64268be
@pavelprystinka
Resize captions bottom spacer based on captions visibility
d611f85
@pavelprystinka
RTT re-alignment for landscape view
1187985
@pavelprystinka
RTT captions to the Table
fa8957d
@pavelprystinka
Add grab handle
c42217f
@pavelprystinka
adjust edit text colors
f5d0f50
@pavelprystinka
Speed up maximize/minimize animation
4ca0359
@pavelprystinka
Update RTT edit text behaviour
b84b00f
@pavelprystinka
hide keyboard
b788d98
@pavelprystinka
Handle some keyboards triggering back press
5fdd6b0
@pavelprystinka
Add maximized captions background
4c74088
@pavelprystinka
Rollback color change
3582335
@pavelprystinka
disable CALL_START_TIME
331b1ee
@pavelprystinka
Revert
48c29be
@pavelprystinka
Adjust colors for dark mode
a22c2d2
@pavelprystinka
ktlint
443b9c0
@pavelprystinka
Enable RTT menu
375c388
@pavelprystinka
Add menu label
17ea66c
@pavelprystinka
Refactor captions view - move manager to the viewModel
0a8d4b0
@pavelprystinka
Hide control panel when keyboard is open
ceff7de
@pavelprystinka
re use isKeyboardOpen
c9be760
@pavelprystinka
Rendering messages. WIP
c91d18a
@pavelprystinka
Display RTT with captions
7c59a60
@pavelprystinka
Do not show Rtt text edit if RTT is not active
fa994e0
@pavelprystinka
Send isFinalized with rtt text
36dc8fa
@pavelprystinka
Use RealTimeTextResultType
8394ceb
@pavelprystinka
do not set rttInputText.isVisible based on device
7f7d7e9
@pavelprystinka
Updates for tablet
9c2def5
@pavelprystinka
Display correct icon on the captions header
da75361
@pavelprystinka
Display is speaking indicator when participant is typing rtt
4701fd9
@pavelprystinka
Do not turn screen off on proximity sensor when keyboard input is open
793d165
@pavelprystinka
For captions switch use setOnClickListener instead of setOnCheckedCha…
5c223f8 
…ngeListener
@pavelprystinka
RTT Accessibility
1c89cfa
@pavelprystinka
Captions on/off accessibility notification
4d6d8a0
@pavelprystinka
Captions and RTT to the right of the Grid when in landscape
533e9d5
@pavelprystinka
Fix layout
7d42cce
@pavelprystinka
align avatar vertically
4c4f9f5
@pavelprystinka
Consume rtt sender name
1226f04
@pavelprystinka
Use strings
e53cd6c
@pavelprystinka
accessibility announcement
9760e9f
@pavelprystinka
For echo RTT messages for not read sender info
cc4d959
@pavelprystinka
Align Starting Captions
8b4bbc1
@pavelprystinka
Apply text rtl to a whole call composite
013ecfe
@pavelprystinka
Delete RTT record when text is blank
fdb9642
@pavelprystinka
Send empty text message
c0425fe
@pavelprystinka
Keep empty RTT message
df2cf94
@pavelprystinka
move opening and closing menus to navigation state
e46d473
@pavelprystinka
Prevent fantom addTextChangedListener triggering when rotating device
25fa975
@pavelprystinka
address some unit tests
08dc620
@pavelprystinka
Add learn more link
831ed23
@pavelprystinka
Code format
52ded16
@pavelprystinka
Fix unit tests
b3795e4
@pavelprystinka
Update tests
c8d6690
@pavelprystinka
CaptionsDataManagerUnitTests WIP
a29afd1
@pavelprystinka
Merge branch 'develop' into feature/rtt
96bf97c
@pavelprystinka
Code format
31d2661
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pavelprystinka