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/chart markup #320

Merged
merged 6 commits into from
Aug 7, 2015
Merged

Feature/chart markup #320

merged 6 commits into from
Aug 7, 2015

Conversation

JuusoV
Copy link
Contributor

@JuusoV JuusoV commented Aug 7, 2015

Outlook after styling:
screenshot-localhost 3000 2015-08-07 09-52-34

@JuusoV
Copy link
Contributor Author

JuusoV commented Aug 7, 2015

Still couple things needed to be fixed. WIP tag added

@JuusoV JuusoV removed the WIP label Aug 7, 2015
@JuusoV
Copy link
Contributor Author

JuusoV commented Aug 7, 2015

This can be merged now. @apinf/developers

@@ -1,5 +1,10 @@
<template name="filterData">
<h3>Filter by:</h3>

<div class="box box-primary">
Copy link
Contributor

Choose a reason for hiding this comment

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

Check indentation on this file. There may be a semantic error, indicated by the odd indentation structure.

@brylie
Copy link
Contributor

brylie commented Aug 7, 2015

Everything seems good, except the odd indentation structure in filter.html. Update the file indentation and double check the opening/closing tag pairs as well as nesting.

@brylie brylie added the WIP label Aug 7, 2015
@JuusoV
Copy link
Contributor Author

JuusoV commented Aug 7, 2015

@brylie how does it look now?

brylie added a commit that referenced this pull request Aug 7, 2015
@brylie brylie merged commit 13cc78b into develop Aug 7, 2015
@brylie brylie removed the in progress label Aug 7, 2015
@brylie brylie deleted the feature/chart-markup branch August 7, 2015 12:23
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.

None yet

2 participants