Commit Graph

1055 Commits (c5744321ba707bf36a1112ce9c282de18dc3d298)

Author SHA1 Message Date
Matthew Chen c5744321ba Backup misc collections. 6 years ago
Matthew Chen 4556025566 Update migrations. 6 years ago
Michael Kirk 61758dcf05 Only show caption for multiple images 6 years ago
Michael Kirk 0ac8f13c0b remove redunant method, consolidate naming, adding array getter 6 years ago
Michael Kirk 6fdd5d1001 dont initializer pagerScrollView as sideEffect 6 years ago
Michael Kirk 9bcc6a6c57 show navbar for photo/album picker, not approval 6 years ago
Matthew Chen dcaaff7eac Add isRegisteredAndReady to TSAccountManager. 6 years ago
Matthew Chen 8110e0c765 Clean up usage of TSAccountManager. 6 years ago
Michael Kirk 26ca47b511 Avoid CaptionTextView animation glitch while dismissing MessageTextView 6 years ago
Michael Kirk 78c74d87bc Merge tag '2.32.0.16' 6 years ago
Michael Kirk 9317ee9c99 design comment 6 years ago
Michael Kirk e3120a5b87 cleanup keyboard animation code 6 years ago
Michael Kirk 0562619ca9 smaller margins between rail images, avoid choppy change as the margin updates
are not being animated smoothly.
6 years ago
Michael Kirk 55807f9a4d iPhoneX compatible keyboard animations 6 years ago
Michael Kirk 279694e704 keyboard animation cleanup 6 years ago
Michael Kirk 0808458392 fix caption dismiss animation/placeholder for multiline message body 6 years ago
Michael Kirk 4f1f09f231 Use snapshot view to avoid momentary missing bottomToolbar while switching
firstResponder from CaptionView to AttachmentApprovalViewController.
6 years ago
Michael Kirk 3bfda7ea81 Smooth kbd dismiss: avoid bouncing CaptionView due to quick transition of firstResponder 6 years ago
Michael Kirk b108f284bd WIP: hide caption keyboard
It's tricky because we're hopping from one first responder to another.

Specifically, from the CaptionView.textView, which shows the keyboard, to
making the AttachmentApprovalViewController first responder, which shows the
BottomToolbar message text field, so in short order, we're getting multiple
notifications.

User hit's "Done" with caption

- Point A - CaptionView is positioned at the top of the keyboard
- Hide keyboard (frame change details must be calculated by y offset, since willChanage notification doesn't "shrink" the keyboard frame, it just offsets it to be non-visible.
- Point B - caption view is positioned at the bottom of the screen, input accessory view not visible
- Show Keyboard (not actually showing the *keyboard* here, but rather the VC's input accessory view)
- Point C - caption view is positioned atop the input accessory view

We want to animated smoothly from A->C, skipping B. But how do we do that robustly? We could track something like "last known input accessory view height" and never present the captionView below that. But I'm worried it won't be very robust since the input accessory view can change height, e.g. text view grows with text content or dynamic text changes.
6 years ago
Michael Kirk 838012d1ec Caption length limit and label 6 years ago
Michael Kirk e0f7513df0 white tint for attachment approval textview cursors 6 years ago
Michael Kirk a946ec0056 new icon assets per design 6 years ago
Michael Kirk 8776dd1909 New "add caption" and "done" assets 6 years ago
Michael Kirk feb5a0c444 fix initial CaptionView layout glitch 6 years ago
Michael Kirk e65eeff0fd Keyboard should cover _Caption_ TextView when _Message_ TextView becomes first responder. 6 years ago
Michael Kirk 33750baf67 finally got dismiss-before-swipe 6 years ago
Michael Kirk dd82803a10 second abandoned attempt to require dismiss before page 6 years ago
Michael Kirk b98b3d1fdd WIP: require dismiss before swipe 6 years ago
Michael Kirk 706dd3d0c1 initial layout of keyboard is correct across pages 6 years ago
Michael Kirk 280664c763 WIP: keyboard 6 years ago
Michael Kirk eed2558050 Avoid glitch in keyboard dismiss.
iOS adjusts the inputAccessoryView's host input views layout margins when
popping/dismissing the keyboard, which causes a noticeable glitch.
6 years ago
Michael Kirk 8b5d1d9e69 Only add delete button once 6 years ago
Michael Kirk 28f8fc5911 per cr, avoid unnecessary 6 years ago
Michael Kirk fcc4b516a5 fix typo in logging 6 years ago
Michael Kirk 4f0092615a Support captions *and* independent message body 6 years ago
Michael Kirk cd88ef2bec CaptionView text field per page 6 years ago
Michael Kirk 79995cc52c rename captioning -> messageText
TODO
-[] caption field per photo, separate from album message
-[] caption field sticks to keyboard on becoming first responder
-[] swipe updates caption field (not album message)
-[] limit caption length to 240 chars
-[] add more asset

TODO picker
-[] Done becomes "next"
-[] shared navbar, album picker doesn't cover entire screen
-[] new selected checkmark asset
6 years ago
Michael Kirk 47affb81c0 Move gallery rail into input accessory view
TODO

-[] caption field per photo, separate from album message
-[] caption field sticks to keyboard on becoming first responder
-[] swipe updates caption field (not album message)
-[] limit caption length to 240 chars
-[] add more asset

TODO picker
-[] Done becomes "next"
-[] shared navbar, album picker doesn't cover entire screen
-[] new selected checkmark asset
6 years ago
Matthew Chen 544bdbd7f5 Fix edge cases around registration. 6 years ago
Michael Kirk 31782af2f1 dark theme section headers in tile gallery 6 years ago
Michael Kirk 6cdef57e2a Merge tag '2.32.0.15' 6 years ago
Michael Kirk 18766280f6 fix crash when non-registered user upgrades 6 years ago
Michael Kirk dbe8e5706d avoid crash on iOS9/10 6 years ago
Matthew Chen 8eb2550e0c Respond to CR. 7 years ago
Matthew Chen 8b24fba095 Add "add more" button to image picker. Provide caption editing continuity. 7 years ago
Michael Kirk 87bfdbb72c Sender Rail 7 years ago
Matthew Chen ea080eda72 Sketch out the photo collection picker. 7 years ago
Michael Kirk 92135af8be Merge branch 'release/2.32.0' 7 years ago
Michael Kirk ff63c31dae CR: rename colors 7 years ago
Michael Kirk 47a7114317 Gallery pager style changes 7 years ago