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

Master #2

Open
wants to merge 536 commits into
base: oauth-patch
Choose a base branch
from
Open

Master #2

wants to merge 536 commits into from

Conversation

b-3-n
Copy link
Owner

@b-3-n b-3-n commented May 17, 2019

No description provided.

jainkuniya and others added 30 commits January 31, 2017 10:48
Show it when user narrowed to stream/topic/pm from drawer.
Fills the fields in the chatbox. This saves the time in
composing message.

Fixes #330.
Included a case when narrowed to self, then
list size is 1, and set title as user name.
While narrowing, we should check if we have already narrowed to the
required stream.
Account for message headers generated when messages are loaded above a
list of messages.

This is important because message headers are also counted as items in the
recyclerview and hence when retaining the position of visible message at
the top of list, we need to account for all the message headers generated
for the set of LoadPosition.ABOVE messages.

Fixes #353.
Don't toggle toolbar and floating button
if messages doesn't occupy full screen.
New layout for in app notification
which slides from top of the screen.

part 8th of #309
When a picture is clicked and sent directly (without a crop or edit), it
gets rotated by 90 as most phone cameras are landscape. To solve this,
it'd be better to use the displayed image and save it in the current
photo path before sending.
Currently, search is performed in home view. This is especially
undesirable when the query gives a lot of matching results. It'd be
helpful to be able to search in current view of the user.
@deepcodeapplocaldev
Copy link

DeepCode analyzed this pull request.
There are 2 new important issues, 3 new warnings and 53 new info reports. 4 info reports were fixed.

Click to see more details.

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.