Topic: If you have a moment…..

[mishca09]

Hi ! for my web design class we had to implementgoogle analytics and my professor wants to see some traffic. So if you have a moment could you just check out my site. Its not quite finished yet http://lildelish.com/. thank you !

[goldshirt*9]

me want a $1 pizza, do they deliver to the uk 

[smokester]

Very nice.

You need to move the "content" div up a few pixels as some of the text overlaps the bottom of the background image when rendering in Firefox and Opera. I can't speak for I.E but of you test it cross browser you should be able to find the happy medium.

[xtopave]

Nice mishca. I like it.

[townie2]

me want a $1 pizza, do they deliver to the uk 

i was going to ask if they delivered to Canada ( 20 minutes or free?  ).
nice work Mishca. are they just checking hits, or IP's? i checked it out last night, and will again, but if their checking IP's, i can pop on with some different Countries IP's with my VPN.

[mishca09]

Thanks again!

Very nice.

You need to move the "content" div up a few pixels as some of the text overlaps the bottom of the background image when rendering in Firefox and Opera. I can't speak for I.E but of you test it cross browser you should be able to find the happy medium.

Thanks ! I'll fix it.

i was going to ask if they delivered to Canada ( 20 minutes or free?  ).
nice work Mishca. are they just checking hits, or IP's? i checked it out last night, and will again, but if their checking IP's, i can pop on with some different Countries IP's with my VPN.

I think he'll probably check IPs.

latest review is a bit thin: past reviews 404.

I know I haven't uploaded the content for past reviews yet and I changed everything last minute.

[townie2]

cool. you're about to get some international visitors 

[dweez]

I'd probably "center align" the webpage too.

[smokester]

I'd probably "center align" the webpage too.

That's not always a simple operation when using CSS and divs to layout your page.  You can cheat and stick everything in a centered table but your tutor wouldn't like that approach very much.

Otherwise you can run the gauntlet with the:
margin-left: auto;
margin-right: auto;
technique but not all browsers understand that.

Or finally, center the body using the:
text-align: center;
rule, but don't forget that you'll have to set each div's CSS style to:
text-align: left;
otherwise all text will center.

[mishca09]

Smokester  or anyone….


how do i make it so there isn't multiple  little boxes. Is it because I've assigned a class to each paragraph? I just want to have it all in one box with a scroll bar. http://www.lildelish.com/past.html

thanks !

[smokester]

You have multiple paragraphs some with a set height of 50px.  You are going to get separate instances of these paragraphs which is what's causing your problems.

My approach would be to style one particular div and then specify how each <p> tag (and header tags etc) displays within that div. So if your div was called .pastDiv, the paragraph style would be: .pastDiv p and so on.

I'll give it some more thought as this was off the top of my head.

[mishca09]

thanks I'll give it a try.

[subvinorosa]

Visited the site.  I'd check out the places you mentioned but...

[smokester]

I tried this just now Mish using a div with overflow set to auto and a predefined height and width of 500px.  I then styled the <p> and <h2> tags inherently to that div (so .pastDiv p  and .pastDiv h2) and then just used whichever tag without specifying a class or ID as everything inside that was already specified by virtue of being within the .pastDiv div.

It operates exactly how you want although I didn't position it in my model as I wasn't worried about that value in this test set-up.

Another thing, are you intentionally using a non machine font?  I ask as although it might be free, I expect it would worry a lot of people if a page asked them to download something they weren't sure about.  The best practice is to always use standard fonts and when something funky is needed, create an image of the text and place in you page as an <img> element.

[mishca09]

Thanks smokester. For quick fix I just added a height to my content div, so it contained it. Ill try your way to.

[smokester]

Thanks smokester. For quick fix I just added a height to my content div, so it contained it.

I was under the impression you wanted everything to show and be viewed by scrolling the container.  I hadn't realised that you also wanted the paragraphs to scroll independently.  I have to be honest and say I really don't like the 3 scrollbar navigation (page-container-paragraphs), as scrollbars should be used minimally where possible. 

[mishca09]

I dont like it either but I just wanted it to look neat. But I plan on fixing,so it is justone scroll bar.

[smokester]

You have quite a lot of syntax errors too, would it be ok if I listed them for you to fix?  I don't want to overstep the mark you see.

[mishca09]

sure I don't mind.

[smokester]

Just in the Past page you have these errors which are probably on the others:

Missing 'f'
.caption {
      margin:2px 30px 10px 0px;
      font-family: Arial Narrow,Arial,Helvetica,sans-serif;

Errant break:
<h2>Georgetown Cupcakes</h2>
<p class="caption"> Georgetown Cupcakes</br>
111 Mercer St   


Orphaned header closing tag:
....totally defeats the purpose of wearing one, when your hairs all in your face.</p>
Mariebelle Fine Chocolates  </h2>
484 Broome St

Missing bracket (which is screwing the following paragraph tag I think):
<h2>Pig Heaven Restaurant</h2>

Superfluous paragraph closing tag:




  </p>

</div>

[mishca09]

Thanks . I will fix it. on a happy note I passed with a B+. 

[smokester]

Well done Mish'.

[townie2]

Congrats Mishca 

[mishca09]

Thankies