AnthonyJ Posted July 27, 2011 Report Share Posted July 27, 2011 What is wrong with this code; <div id="header"> <h1><a href="index1.html">Plantra Tips and Tricks</a></h1> <img src="images/plantra_logo.png width="300" height="100"/> </div><!--ends header div--> Quote Link to comment Share on other sites More sharing options...
Andrea Posted July 27, 2011 Report Share Posted July 27, 2011 It's missing the closing quote after your image file. <img src="images/plantra_logo.png" width="300" height="100"/> Quote Link to comment Share on other sites More sharing options...
Andrea Posted July 27, 2011 Report Share Posted July 27, 2011 And last time I looked at your coding, you were not using XHTML doctype - unless that's changed, the closing slash in the img tag is also wrong. Quote Link to comment Share on other sites More sharing options...
AnthonyJ Posted July 29, 2011 Author Report Share Posted July 29, 2011 I need to take it out? Quote Link to comment Share on other sites More sharing options...
AnthonyJ Posted July 29, 2011 Author Report Share Posted July 29, 2011 Never mind my last...Can you point me to what I need to watch for adding video? Quote Link to comment Share on other sites More sharing options...
Archadian28 Posted July 30, 2011 Report Share Posted July 30, 2011 You need to put a space before /> ex. height="100" /> Quote Link to comment Share on other sites More sharing options...
AnthonyJ Posted July 30, 2011 Author Report Share Posted July 30, 2011 Has anyone heard of Homestead web building service and what is your impression? Quote Link to comment Share on other sites More sharing options...
AnthonyJ Posted September 8, 2011 Author Report Share Posted September 8, 2011 Hi guys, I have been out of pocket for awhile but I'm back...I need to revisit images....What is wrong with this code to add a full background image? I will add my css file as an attch. style.css Quote Link to comment Share on other sites More sharing options...
Andrea Posted September 9, 2011 Report Share Posted September 9, 2011 It is really better if you start a new thread for a new problem/question/issue - and it's easiest for us to help if you have a link. That way, we can see html, CSS AND the images. It's also helpful if you describe the problem, unless it's obvious when looking at the linked page. Just 'wrong' and the CSS you uploaded don't tell me anything - I can only guess. Is your image not showing up? Is it showing up, but not where/how you want it? (So what is it doing and what should it be doing?) Looking at your CSS I only see for sure - and that has nothing to do with a background image - that this is wrong: * { margin: 0px auto; padding: 0px auto; color: #000; } It should look like this: * { margin: 0; padding: 0; } The * is the universal place holder - by using it, you apply whatever you write there to every single tag in your entire page/site. My version is used to remove any default paddings or margins from anything that comes with one. For example, paragraphs have different margins/paddings, depending on what browser. By making them all zero at first, and then adding them back in later as you want, makes it easier to have your pages look more the same across browsers. I guess adding the color isn't exactly wrong, but unusual - mostly, global colors are added to the body or html tag. Adding the right-left margin of auto is done to center things. You don't want everything centered (hopefully). That part is mostly added to a wrapper division which centers your content division on the monitor. And - and that's one of my many pet peeves: When you say '0', you mean zero anything, so there's no need to say 0px - just 0 will make everything zero: px, em, %, cm, whatever. Adding the 'px' to the '0' isn't wrong, just redundant. Quote Link to comment Share on other sites More sharing options...
AnthonyJ Posted September 9, 2011 Author Report Share Posted September 9, 2011 OK...Let's not peeve Andrea....I will open a new topic with a link Quote Link to comment Share on other sites More sharing options...
Andrea Posted September 9, 2011 Report Share Posted September 9, 2011 OK...Let's not peeve Andrea....I will open a new topic with a link Too late Just kidding - but I did move your new thread into the proper forum Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.