![](https://forums.phpfreaks.com/uploads/set_resources_1/84c1e40ea0e759e3f1505eb1788ddf3c_pattern.png)
moberemk
-
Posts
695 -
Joined
-
Last visited
Posts posted by moberemk
-
-
Aside from the white-on-red here and there (I'd tone that down a little bit were I you), this looks good.
-
[QUOTE]IDs have a higher specificity rating, making them overrulling over classes. So when you give something a class and an ID, the ID's rules come first.[/QUOTE]
Also, Javascript can only call ID's, unless you use the Prototype Library's dollar function. -
What's a "Navajo Language Pack"? As in a font?
-
Ah. I was wondering about that.
-
Isn't that first email one to a completely different site, and not PHP Freaks as he asks? Ah, cross-promotion.
-
Seems kinda redundant. HTML isn't a hideously complex topic, and I doubt that you could create an entire site around HTML tutorials that aren't elsewhere on the web.
-
Just a quick thing before I sleep; the site takes a long time to load fully with all the images, and you should really look into using more CSS. And get rid of all those <font> tags; replace them with classed <span>s, as well as getting rid of all the inline styles and replacing them with an external stylesheet.
-
Yes, except that is a deprecated attribute. Your HTML is filled with them, using style attributes where you probably should start switching to CSS. It will work without any changes, but it's obsolete, and generates a lot of extra markup. All those font tags you use could be replaced with a <span> tag with a class, for example. Rather then specifying on a by-tag basis what you want it to look like, use more general CSS classes. It makes life a lot easier.
-
If you want it to stretch, all you would need to do is make it a CSS background image. However, you don't use CSS, you use HTML. And if you want to have less deprecated code, you should probably get rid of the <center> and <font> tags and instead use CSS for that stuff.
-
NO! The entire filename [i]is[/i] .htaccess. The htaccess is the file extension, but there is no file name. No .txt, just the .htaccess. Nothing else. That's an Apache server config file.
-
IDs have a higher specificity rating, making them overrulling over classes. So when you give something a class and an ID, the ID's rules come first.
-
What version of Dreamweaver are you using? It should be as simple as setting it to only pull records from the database with the same user ID as the logged-in user.
-
Well, for one thing, add some spacing. Everything is really crammed together. In addition to that, your side boxes are just floating, with no border or definition, and also probably to the border of the page itself. As it stands, the site is really disjointed and hard-to-read, and those two weird grey bars you have everywhere really don't add anything. I'll bet that your site is based on your forum skin, right (speaking of which, shouldn't there be some kind of notification outside of the code that said that the theme isn't your design)? Not to mention that your site doesn't resize AT ALL. You can see it fine at 1024*768, but nothing lower (even though there are still plenty of people who prefer the size of 800*600). Your topic images are also fairly ugly. And what's with this?
[img]http://www.pcritics.com/images/Dave.jpg[/img]
On the code side, you have over [url=http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.pcritics.com%2F]135 validation errors[/url], half your code is deprecated elements since HTML4.0, and you've got a ton of javascript commands and you've taken a 2px image and stretched it to 936px.
In summary, your site is an IPB skin clone that fails to capture the essence of the original, with some fairly inept HTML code, deprecated elements for everything, and a few reviews that aren't organized outside of alphabetically. And once you get more reviews, won't that page get a little long? -
Well, you should use a parameter in the recordset dialog linked to the user's session ID.
-
Aah. Okay, that explains that. Thanks for clearing that up for me.
Now the only thing I don't get is why every time I post, it brings me back to the parent forum. Couldn't that maybe become some sort of option for those of us whom it merely annoys? It's a useful feature for some, but because I always browse the forum in tabs, I just don't need it around, and I'm used to being able to look over my post and make sure that I said what I meant. -
Sure, it's probably better from a server-side perspective, and definitely mroe secure. But am I the only one who doesn't like the default theme's colours, the use of <font> tags, or the fact that the forum includes a link to the validator and claims valid XHTML but actually fails the test?
-
Actually, if you're trying to make something of an email system, you should rather store it in text files or a database, instead of sessions. Unless I misunderstood what you are looking for.
-
This is what is termed as "Programmer Art", or something created by someone who can outcode anyone but who probably shouldn't touch the graphics. It's big, it's red, it generates [url=http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.knightsdivine.net%2F]bad HTML[/url] and [url=http://jigsaw.w3.org/css-validator/validator?profile=css2&warning=2&uri=http%3A%2F%2Fwww.knightsdivine.net%2F]CSS[/url], and then there's this piece here:
[code]<link href=css.css rel=stylesheet type=text/css>
<link rel="icon" href="http://kd.madur.net/little.jpg">
<link rel="shortcut icon" href="http://kd.madur.net/little.jpg">
<META NAME="keywords" CONTENT="Knights Divine, kd, bearruler, MMORPG, online game, game, free, army, medieval, fun, recruit, master, apprentice, rpg, php, mysql, nerd, geek">
<META NAME="description" CONTENT="Knights Divine is an MMORPG that lets players build their own army, attack/spy on/steal from other players, buy weapons, upgrade, and train followers.">
<META NAME="author" CONTENT="bearruler, Knights Divine">
<html>
<head>
<title>Knights Divine</title>
</head><body><center>[/code]
Aren't LINK and META tags supposed to go inside the <head> element? You don't have a DOCTYPE either, you have a beveled circle as your logo (that technique went out when it came in) and you've got more tables then I even want to think about. In summary, get someone to help you redesign the entire HTML output from the ground up, and delete everything you have relating to style.
Okay, a little harsh, but that is all IMO. -
PHP tags at last! Now if only it weren't for the fact that it used the <font> tag, things would be even better!
-
It looks cool, but it may be the most pointless use of tables I've ever seen. You should really take those out. It would also probably help if you had a DOCTYPE at the top too.
-
To actually answer the topic question...
That means that your PHP script isn't running as a PHP script, it's running as a file. I'd make sure that your PHP is still running. -
Give it a document outline. For example, use an image replacement technique in your header to create that image, and use a CSS list for your navigation. Also, take out any extra images, like the big solid colour block to the left of your logo.
-
That's true. For a CSS-based layout, this still shows much of it's table-based origins. You really should get rid of all the extra images you have floating everywhere where you could just create it in CSS. Like in you navigation-that could just be som CSS blocks.
In addition, your document doesn't have a very good semantic value to it. Try looking at your page without the linked CSS enabled, and see what you think of it then. -
[b]D[/b]ocument [b]T[/b]ype [b]D[/b]efinition.
Teen community
in Website Critique
Posted
Oh yeah, try using less images (it takes a while for them all to load for my computer) and change the font color at the top, it's way too hard to read.