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

Move activities dressed as dialogs into actual dialogs #3158

Closed

Conversation

mcginty
Copy link
Contributor

@mcginty mcginty commented May 13, 2015

No description provided.

@agrajaghh
Copy link
Contributor

pls have a look at #2821. I combined ReceiveKeyActivity and ConfirmIdentityDialog after your comment 😉

descriptionText.setMovementMethod(LinkMovementMethod.getInstance());
}

private static IncomingPreKeyBundleMessage getMessage(MessageRecord messageRecord)
Copy link
Contributor

Choose a reason for hiding this comment

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

drop the static

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it's necessary since it's used in the constructor while passing a result into super. (this is the case for all of those statics, so applies to the other comments)

@mcginty mcginty force-pushed the feature/faux-dialog-activities branch 2 times, most recently from 0ec3789 to 414cb90 Compare May 15, 2015 00:44
@mcginty mcginty force-pushed the feature/faux-dialog-activities branch from 414cb90 to c83f5e5 Compare May 15, 2015 23:46
@mcginty
Copy link
Contributor Author

mcginty commented May 15, 2015

updated

@mcginty mcginty closed this in fa39c23 May 20, 2015
pR0Ps pushed a commit to SilenceIM/Silence that referenced this pull request May 25, 2015
pR0Ps pushed a commit to SilenceIM/Silence that referenced this pull request May 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants