First of all, nice of you to now do design in CSS, welcome aboard ;)! The website look fine, basic but hey, it’s working and it’s your first CSS design :D!
Here’s a problem I found … :
Why ? Oh god why ? I don’t see why there is so many empty p on the index, and why you give them a class ? They’re empty anyway :/
Thanks for your reply!
The " " had to be put in as when viewing the site in a smaller browser window, the entire menu was not visible. Just thinking, maybe I could make this section have a minimum height, would that work? Also, I didn’t assign a class to these elements on purpose, the text within the "innerwrap" is style 2 and it looks that the  ’s have just adopted that as there in the same section (wasn’t intentional!).
Looks good, I like the color scheme and the layout and all…
The header is too big I think. You should also get some images or something in there with your content, all that plain text is putting me to sleep.
And yes, there is always a way to fix display problems in the stylesheet rather than your HTML markup. So if youre adding empty spaces and breaks in the HTML, youre doing something wrong! Go back to the CSS and tweak it until it’s right without the empty spaces congesting your HTML markup.
I like it! Although I think you might be missing a letter in "Obsetrics," I think you did a great job!
It may be time for you to look into CSS sprites for your rollover effects. It’s kind of a big undertaking for a beginner in CSS, but I believe the force is strong in this one. :)
Is the way I’ve done my rollovers wrong, or is this new way you mention the "correct’ way of doing it?
Appreciate your comments!
Thanks koewnet on the assist – my apologies for not providing any resources for you… but I think Chris did a nice article eons ago about sprites, and that was kind of my "a ha!" moment. http://www.csssprites.com is also a nice generator that you can use to cheat once you’ve learned about it. :)
If you need any help getting it down pat, feel free to contact me. :)
Hi Guys (again!)..
What’s the difference with Edwin’s method and Erika’s method for the menu?
I’ve tried to follow Edwin’s way, but the images are not changing.
Files are as follows:
As far as the JS goes, I have only used the one image rollover with the code Edwin gave me (Home). How would I go about adding more menu rollovers into this? Is it just a case of adding in the following for each part of the menu?
If I were you, I’d check out the Sprite article here on Css-tricks.. (http://css-tricks.com/css-sprites-what-they-are-why-theyre-cool-and-how-to-use-them/)
The method you’re using now uses jQuery, but if you’re gonna do it that way, you should replace the whole "MM_swapImage…." stuff with a jQuery method for image replacement. I’m no jQuery pro, but you can do some stuff with the css, or change that img src attribute… anyway, you’ll need to go watch some jQuery videos, and in my opinion, that’s a bigger learning curve than sprites. (you should at some point watch some jQuery videos cause it’s awesome… check out http://www.jqueryfordesigners.com)
Using sprites, you’ll want to simply give each of those <p> elements an id, like "home" for the home page link, give it a background image of the normal home button image, and add in a :hover attribute for all of those <p> elements that switches the background image. It actually switches the position, but you’ll see that soon!
Check out that post of Chris’ and I think you’ll see that it’s going to be much easier.
(On a side note, you might want to change that menu div and those <p>’s to a <ul> with some <li>’s… it’s the way to go yo. Ask the others… they actually know what they’re talking about.)
You must be logged in to reply to this topic.