Skip to content
This repository has been archived by the owner on Apr 24, 2018. It is now read-only.

Allow to add the showcase view to a custom parent #304

Conversation

martinbonnin
Copy link
Contributor

I needed this because at the startup of my app, the LoginFragment competes with the ShowcaseView to be on top of everything else in the Activity

@@ -387,6 +387,9 @@ public void showButton() {
final ShowcaseView showcaseView;
private final Activity activity;

private ViewGroup mParent;
Copy link
Owner

Choose a reason for hiding this comment

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

Could you remove the m prefix from these fields please? :)

@amlcurran
Copy link
Owner

Like the idea of this, just a couple of issues, and a conflict, but happy to merge once these have been addressed :)

@martinbonnin
Copy link
Contributor Author

thanks for the comments. Should be better now.

@amlcurran
Copy link
Owner

Looks great! There are still conflicts; just make sure you merge the master branch into your branch, and then that should work 👍

…_parent

Conflicts:
	library/src/main/java/com/github/amlcurran/showcaseview/ShowcaseView.java
@martinbonnin
Copy link
Contributor Author

Should be good now, tell me if anything else is required.

@amlcurran
Copy link
Owner

Awesome, thank you very much! I will make a release containing shortly :)

amlcurran added a commit that referenced this pull request Oct 28, 2015
…iew_to_a_custom_parent

Allow to add the showcase view to a custom parent
@amlcurran amlcurran merged commit 37e2b8a into amlcurran:master Oct 28, 2015
@martinbonnin martinbonnin deleted the allow_to_add_the_showcase_view_to_a_custom_parent branch October 28, 2015 20:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants