Skip to content

demo page style update #208

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

Merged
merged 1 commit into from
May 21, 2016
Merged

demo page style update #208

merged 1 commit into from
May 21, 2016

Conversation

Nazanin1369
Copy link
Collaborator

@Nazanin1369 Nazanin1369 commented May 21, 2016

Improved demo page styling and design.

@Nazanin1369 Nazanin1369 changed the title demo style update demo page style update May 21, 2016
@coveralls
Copy link

coveralls commented May 21, 2016

Coverage Status

Coverage remained the same at 93.197% when pulling b2f7812 on Nazanin1369:master into 68e3ef9 on huei90:master.

@Nazanin1369
Copy link
Collaborator Author

@huei90 Could u plz review this change.

@lvarayut
Copy link
Collaborator

It looks good to me. Just one small thing, do you prefer to align your code using 2 or 4 spaces? In the JavaScript code, everything seems to be aligned using 2 spaces.

@Nazanin1369
Copy link
Collaborator Author

Sure.

On Saturday, May 21, 2016, Varayut Lerdkanlayanawat <
notifications@github.com> wrote:

It looks good to me. Just one small thing, do you prefer to align your
code using 2 or 4 spaces? In the JavaScript code, everything seems to be
aligned using 2 spaces.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#208 (comment)

Best Regards,

@Nazanin1369 Nazanin1369 merged commit 920e964 into hueitan:master May 21, 2016
@hueitan
Copy link
Owner

hueitan commented May 21, 2016

Hello, feel free to merge or change, you have the authority to do it =)

On Sunday, 22 May 2016, Nazanin Delam notifications@github.com wrote:

Merged #208 #208.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#208 (comment)

@Nazanin1369
Copy link
Collaborator Author

Thank you @huei90

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.

4 participants