Updating new design.

[agamdua]

Corresponds to #6

2 will be covered in next PR.

[agamdua]

@gpranav88 @kartikdhar -

This is the working pull request for integrating the new design.

Currently home page & product.html have been updated. Do have a look and let me know if they are working properly.

Command:

$ node dummy.js

Urls to check:

• [ ] localhost:3000
• [ ] localhost:3000/product/1

Please tick mark if complete (@kartikdhar - this is mainly for you to check if its rendering according to the new design) or comment if there's a problem.

[agamdua]

Also, what to do about reviews section? Delete as discussed? Will this affect your layout?

[kartikdhar]

delete all reviews - will have a look at the templates

Kartik A. Dhar kartik.dhar@gmail.com

On 19 January 2014 21:00, Agam Dua notifications@github.com wrote:

Also, what to do about reviews section? Delete as discussed? Will this affect your layout?

— Reply to this email directly or view it on GitHubhttps://github.com/agamdua/clothzy/pull/9#issuecomment-32710806 .

[agamdua]

@kartikdhar - Two things

first problem

How do I merge these two parts of the templates? Links to lines in both files:

1. New file: https://github.com/agamdua/clothzy/blob/0f3865aa5169c313201d7328ac446cbee5fb4d5f/frontend/templates/StoreHome-test.html#L682
2. store.html: https://github.com/agamdua/clothzy/blob/0f3865aa5169c313201d7328ac446cbee5fb4d5f/frontend/templates/store.html#L113

Replacing the p & span in 1 with i tag in 2 makes the phone icon disappear.

Having to do this manually, diffs not working for huge file changes.

second problem

Code line: https://github.com/agamdua/clothzy/blob/0f3865aa5169c313201d7328ac446cbee5fb4d5f/frontend/templates/store.html#L113

What's up with the carousel? Straight copy-paste? Lots going on there.

If you do have time it'll be cool if we can sit and merge these two files together. Can be done remote as well (Screenshare/teamviewer) in case we can't meet today/tomorrow.

The products one was relatively easy to merge which you can see in commit: 4c76f2d. Commit is above on this page in this PR in case you want to see.

[agamdua]

Bugs:

• [x] Include font-awesome CDN in base.html
• [x] Remove categories in store.html (kurti, etc)
• [x] Fix icons in product template
• [x] Fix icons in store template
• [x] Remove reviews + closer look (bottom content)
• [x] Store/Product view buttons not rendering correctly on store.html (can check out what's used in product.html)

[agamdua]

Share icons not looking good on product page.

Issue #14 - @kartikdhar handling

Update: Postponing to post-MVP

[agamdua]

Icons fixed. Store specific: Perhaps can improve spacing. Also, closed #13 (share icons)

[agamdua]

More changes:

base.html

• [x] Full update

product.html

• [x] Remove hold item

store.html

• [x] Update collections as per ProductSearch-test.html (from Dropbox) Spacing - Issue #14 ( @kartikdhar handling) - Postponed as mentioned above

Shifting to next version

• [ ] Get hyperlinks (tabs) working

home.html

• [ ] Navbar route to search results