| Minor Problem on zen-garden-screenshot1.png | |
| 1 | Image for "border-image" not correct |
| Serious Problem on zen-garden-screenshot1.png | |
| 1 | When I resize my browser window, there is a lot of "whitespace" on the left that makes me scroll horizontally, which is very difficult with a mousewheel… Centering the content can be done through css with margin: 0 auto; |
| Positive Feature on zen-garden-screenshot1.png | |
| 1 | the bottom of the content is not good, shouldn't you put there some bamboo aswell?
(check online) |
| 2 | good styling here, clean, readable, … |
| New Idea on zen-garden-screenshot1.png | |
| 1 | I should style the links otherwise, this is too old-fashioned… |
| 2 | do not underline, at first glance, they looked like links |
Why do they want "to see as much CSS1 as possible" even if the sites must render properly in FF6, Chrome 13 and IE9 which support even most of CCS3???
they actually haven't updated the site in a while (at least 2 years). That's why I'm not submitting it them, just to the prospective employer.
@Becca Hey! I was wanting to review this but I wasn't sure what exactly. Is it check if these 2 render ok in the browsers you mentioned? www.vegasdesigner.net/ZenGarden and www.vegasdesigner.net/ZenGarden/comp2.htm
Or do I have to review visuals as well? Let me know and I'll be more than glad to help.
Hey Eze!! I already checked and they render correctly in all major browsers. I actually already submitted the urls to the prospective employer. It was his criteria about the specific browsers. I was just looking for general feedback.
Hey Rebecca! I only see one issue, I have a 23inch display and the background cut off and the layout is not centered. Please take a look: http://creativeway.com.ar/ss.jpg. Great design though!
Hi Bec, before I review it I just want to know exactly am I going to be reviewing. Is it the page itself, or how well it renders in different browsers? But seriously no CSS3????
sorry, I should have explained better.... I was just looking for a general review. I did incorporate CSS3. The Zen Garden site, because it is a little dated, only requires CSS1. But the good news is (drum role....) I got hired today by the "prospective employer" so I guess my concept can be taken down. But I appreciate all the effort from my buddies on CF!!!!!
| Serious Problem | |
color choiceI think contents color compete with background image. And middle gradients bothers me some reason. Consider choosing more simple background | |
| Positive Feature | |
second concept seems working better than first one.second concept is simpler and more pursuasive
| |
I think you should work little more on color
The css is spot on! Great job, I gave you all 5's cause you're a star, but you really need to expand your design!
Ciao Bella!
thanks Aaran!!
| Critical Problem | |
UnderliningI have to agree with an earlier review. Underlining is not recommended at all. | |
Hey Paul, welcome to the community. We are trying to keep the quality on the reviews so then you can get good ones as well. Would be nice if next time you can add more when reviewing someone else's project. Thanks in advance.
thank you for wording that so nicely Eze!
Great wording Eze, and Paul hope you have fun contributing to the community. :)
yes, I know what margin:0 auto; is.