![](https://forums.phpfreaks.com/uploads/set_resources_1/84c1e40ea0e759e3f1505eb1788ddf3c_pattern.png)
john010117
-
Posts
492 -
Joined
-
Last visited
Never
Posts posted by john010117
-
-
Very nice design, first of all. I have a couple of suggestions.
For that nav to work effectively as tabs (as I'm assuming you are), I think it'll be best if you specified where the user is in terms of the tabs. For example, if I'm at the homepage, the tab "Home" should be highlighted. That's my only complaint so far. Again, nice job!
-
The only thing I would complain about is the "bar" that goes across the entire page at the top. It's kinda distracting for me. Other than that, nice job!
-
This is just a simple form for trying to XSS the input/output. If you find any vulnerabilities, please do tell me. I used my custom function to prevent XSS from happening. Any characters are allowed.
In other words, try to "hack" this simple form.
-
Try putting "Navigation" / "Other Links" / "Login" in the top bar.
The login form doesn't work (login.php doesn't exist)
Center the main content.
-
No you didn't (at least on the second page if it's a new movie).
"><script>alert("hi")</script>
-
Your site is XSS vulnerable. Try registering w/ the username of:
"><script>alert('hi')</script>
-
The background hurts my eyes.
-
I'm using Firefox 3 Beta 5 with a 1280 x 1024 monitor resolution. An annoying horizontal scrollbar appears on my browser window while visiting your site.
-
Use AJAX, for one. Nobody wants to reload the entire page just to make a shout.
-
What sstangle73 said is true.
<?php $dir = 'upload/02_00'; echo (count(glob($dir . '/*')) === 0) ? '<a href="upload/0200.php"><img src="img/0200.png" border="0"></a>' : '<img src="img/booked.png" border="0">'; ?>
-
<?php error_reporting(E_ALL); ?>
at the top of every page.
If you're using a custom error handler, use set_error_handler(function_name)
-
Letting users use HTML is a huge security risk. That's what I believe BBCodes are for.
-
Um... do you have this on a website somewhere?
-
Just found the include directory. Huge security risk.
-
Ok, thank you very much for giving me a detailed explanation. I have modified my site to most (if not all) of the things you've mentioned. Again, more suggestions are great.
-
Looks nice and clean, but quite basic.
I would lose the forum too as I cant understand who would use it?
I just put the forums there just as a test. I'll get rid of it soon.
Just a suggestion , but the first thing that jumped out at me was how wide your pages are.
I know your pages are liquid and thats the point, but from a reader's perspective it is easier to read if it has generous left-right padding. Another thing you can do is float a big image right and have the text on the left side.
Everyone likes to see pictures, and it make the text length shorter, more concise and therefore, much easier to read.
It's scientifically more strenuous to have your eyes move that far from left to right. Think of how newspapers are layed out.
HOpe this helps
Oh yeah, I have a 22" widescreen, so obviously, this exemplifies the problem.
Ok, I'll put more padding in. As for the pictures, how big do you think the screenshots of the websites in my portfolio should be? If it's too big (as it is right now), I'll have to put a lot of text on the right side to make the page look good.
Thanks everyone for their opinions. If you have any more opinions, please feel free to say them. I'm looking to improving my site (and others) constantly.
-
Wrong section, I believe. Try this forum.
-
Ok, thank you for the suggestions. I have fixed the typo in the portfolio page, and have implemented screenshots of my projects. However, I'll leave the copyright as "John the Freelancer". Again, thanks.
-
http://www.johnthefreelancer.com/
I aimed for simplicity while giving the user maximized info about me and my freelance services. Please let me know if something does not display correctly for you (although I have tested my website in 5 different browsers in Vista - Firefox, IE 7, Opera, Safari, Netscape). If you would like to suggest more content (especially for the "about" section - I definately need more in there, but I have no idea what else to put), please feel free to do so. It'll be great also if you could suggest what the colors for the text in the forum section should be.
-
You have way too much scripts going on per page. I suggest you size it down.
-
may i ask how you got those errors?
He shows you how he did it, by removing your existing HTTP vars and editing it to adding single quotes, this is SQL injection...
use the function mysql_real_escape_string($_GET['id'])
He did tell you how to fix those errors.
-
Ok, I think I got it. Thanks.
-
I need help with this query:
SELECT polls. * , poll_options. * FROM polls, poll_options LEFT JOIN poll_votes ON polls.poll_id = poll_votes.poll_id WHERE polls.poll_id =1 AND poll_options.poll_id = polls.poll_d
Here's my database structure for the three tables:
-- -- Table structure for table `polls` -- CREATE TABLE `polls` ( `poll_id` mediumint(5) NOT NULL auto_increment, `poll_question` varchar(200) NOT NULL, `poll_start` int(11) NOT NULL, `poll_end` int(11) NOT NULL, `poll_type` tinyint(1) NOT NULL default '1', PRIMARY KEY (`poll_id`), KEY `poll_start` (`poll_start`,`poll_end`) ) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=2 ; -- -------------------------------------------------------- -- -- Table structure for table `poll_options` -- CREATE TABLE `poll_options` ( `option_id` mediumint(5) NOT NULL, `poll_id` mediumint(5) NOT NULL, `option_name` varchar(200) NOT NULL, KEY `option_id` (`option_id`,`poll_id`) ) ENGINE=MyISAM DEFAULT CHARSET=latin1; -- -------------------------------------------------------- -- -- Table structure for table `poll_votes` -- CREATE TABLE `poll_votes` ( `vote_id` int(11) NOT NULL auto_increment, `poll_id` mediumint(5) NOT NULL, `option_id` mediumint(5) NOT NULL, `vote_user` int(11) NOT NULL, KEY `vote_id` (`vote_id`,`poll_id`,`option_id`,`vote_user`) ) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=2 ;
MySQL returns with an error:
#1054 - Unknown column 'polls.poll_id' in 'on clause'Any help will be appreciated.
-
The top nav could be vertically-centered. Also, you could use a different font. Oh, and please fix the spelling mistakes.
Just got my site up - Need help/suggestions
in Website Critique
Posted
Make sure to validate your code. While looking at the source code, you have two DOCTYPEs, two <head> and two <body>.