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

introduce details tab in sidebar #88

Merged
merged 1 commit into from
Sep 25, 2016
Merged

Conversation

georgehrke
Copy link
Member

@mention-bot
Copy link

@georgehrke, thanks for your PR! By analyzing the annotation information on this pull request, we identified @raghunayyar, @tcitworld and @MathieuSchopfer to be potential reviewers

@georgehrke georgehrke added the 3. to review Waiting for reviews label Sep 24, 2016
@coveralls
Copy link

coveralls commented Sep 24, 2016

Coverage Status

Coverage decreased (-0.1%) to 28.577% when pulling e19f1e0 on sidebar_details_pane into 85b2a64 on master.

@georgehrke
Copy link
Member Author

EditorController is not yet tested, therefore the decreased coverage is fine for now.

@tcitworld
Copy link
Member

Love it !

@eppfel
Copy link
Member

eppfel commented Sep 25, 2016

👍 Really nice improvement

@georgehrke georgehrke merged commit 6499ac9 into master Sep 25, 2016
@georgehrke georgehrke deleted the sidebar_details_pane branch September 25, 2016 17:52
@jancborchardt
Copy link
Member

Good stuff, as discussed :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants