-
Notifications
You must be signed in to change notification settings - Fork 2
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
General #10
Comments
Opportunities
Clarifications
|
@Circa75 Gotcha regarding the cart comment, I think I missed that the cart auto slides out. That is perfect! |
@Circa75 I updated some comments above in opportunities that I've been running into while I've been translating your theme into a wp-theme. Even if you don't have time to execute the solution, it'd be great if we could hash it out together a bit and I try and execute. |
Regarding links in paragraph text, the dotted border looks fine... nice work. Regarding sub-menus, I won't be offended at all if we just need to re-build the pop-up menu to better suit WP eCommerce users. Do you have some examples of menu UI/UX you think might be better for this theme? |
Loves
Opportunities
<a>
tag into a post/about page and it is not discernable that it is a link. Probably needs some styling for links.[update 8/15]
Navigation
Clarifications
The text was updated successfully, but these errors were encountered: