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

Feature/tab content selector #38

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

LOK-Soft
Copy link

Adding the ability to set selector for tab content

Lars-Olof Krause and others added 3 commits October 12, 2017 08:43
@elboletaire
Copy link
Owner

Thanks for your PR @LOK-Soft, apparently LGTM.

Could you add a test for this new feature? If not, don't worry, I'll do it when I've time, just tell it to me.

@LOK-Soft LOK-Soft force-pushed the feature/tabContentSelector branch from 13f10f2 to fa362e1 Compare October 16, 2017 14:21
@LOK-Soft
Copy link
Author

added basic test, is this test sufficent?

@elboletaire
Copy link
Owner

Sorry didn't see it before! I think it's enough 😄

@elboletaire
Copy link
Owner

Ignore the travis build failing.. I think it's related to something I did some days ago. I'll check it when possible and accept the PR.

Again, thanks for your contribution @LOK-Soft 😃

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.

2 participants