Open JereWong opened 9 years ago
Comment about 3rd comment
Don't make animation now. Just make sure it positions in correct place. It shouldn't "jump" when top bar appears. Instead all elements (price, button and wishlist) should stay in the same position, just with border. Take a look at AirBnB for reference.
it’s very easy fix. Remove left padding from sticky price block, make it appear few pixels earlier and fix button. Should be fine.
All other comments is good!
1. Make each bookmark-point abit higher. So just below the previous box
http://cocorium.com/courses/kml6o#description-page-curriculum-wrap
It's nice to have some space betweeen the bar and the content. Now it stops right above the box on the line, which is too close.
2. Better name for the bookmarks.
We should try making plain like #top #description #curriculum #reviews
Also, rename the menu items to: top description curriculum reviews
Because some don't have video, so it's better to say "top" - but if we can do an IF statement to show "video" if it has video it'd be awesome. :)
Also, for the author's name in the bar, I think it's great if we make it so that when they click, the modal comes down! The author's profile. Instead of going up.
3. Enroll Button Should be Same Size as the original?
Gotta ask Pat!
4. Reviews Section
If no reviews, we'll just use our box (that we use for all sections) but just a simple box with text inside
"This course has no reviews yet."
Maybe in greyish font? Find one of our styles. :D
This way, the "Reviews" menu item goes to something.