Jump to content


Photo

Design| blacksun.magistri.net


  • This topic is locked This topic is locked
30 replies to this topic

#1 crashmaster

crashmaster
  • Members
  • PipPipPip
  • Advanced Member
  • 169 posts
  • LocationCzech Republic, Prague

Posted 23 July 2006 - 11:36 PM

Hi there...I would like to hear your opinion of GRAPHICAL design not functional side..
here you are: blacksun.magistri.net

Also you can LOG IN : demo | demo
I'll never be the same...

#2 moberemk

moberemk
  • Members
  • PipPipPip
  • Advanced Member
  • 695 posts

Posted 24 July 2006 - 02:20 AM

Well, if I WAS critiquing the functional side, I would critique the invalid/deprecated code and tables, as well as the use of images for text. But I'm not. Instead, I'll critique the graphical side of things. Graphically, it's good, and except for the divider to the left and the ugly horizontal rules in the main content, this looks pretty good.
Disclaimer: No matter how harsh I may be, I'm always IMO only.

#3 wildteen88

wildteen88
  • Staff Alumni
  • Advanced Member
  • 10,482 posts
  • LocationUK, Bournemouth

Posted 24 July 2006 - 10:50 AM

Not a bad design, nice and simple. However there is too many images. You dont need images to for your headers, instead you could use the htmlentity of the square character you are using, or remove the little square from your headers, but leave it in for your main header, as the square is making it all cramped together. Your naviagation is made of images, which isn't need instead you could produce the same result with CSS.

Your buttons doesnt look very nice either. I would remove the bold text, or pad out the button a bit. Also when you login you should remove the login box and possibly display the users details, or a logout link so the person can logout rather than display the login box all the time. The shopping basket is ok, however I found it hard to get back to my basket once I navigate away from the basket. In the end I found my way to the basket by clicking the hidden basket link. You should make this link stand out alot more, prehaps underline it/make it bold. As at the moment it looks as though its just normal text and not a link, until you hover over it.

I would recommend you to define the font your site should use rather the browsers defualt font (Times New Roman). Prehaps Arial/Vardana style font. You can do this with the following CSS:
body, table {
    font-family: Arial, Verdana;
    font-size: 12px;
    background-color: #333333;
}
This will then effect all the text on your site. I wouldnt use Arial Black, howver it is fine for your main header, but not the text for your site.

#4 steviewdr

steviewdr
  • Moderators
  • Advanced Member
  • 1,364 posts
  • LocationIreland

Posted 24 July 2006 - 12:49 PM

echo above.

You need more mouse over effects for the navigation and the like.

Rgds,
Steve

#5 crashmaster

crashmaster
  • Members
  • PipPipPip
  • Advanced Member
  • 169 posts
  • LocationCzech Republic, Prague

Posted 24 July 2006 - 02:11 PM

I want to sorry :) BUt Username and Password I've posted isnt correct.. I was looking to another database :D
So If you want you can login there : admin | admin
Sorry... :P

Your naviagation is made of images, which isn't need instead you could produce the same result with CSS.

I am not sure that i understand this sentence... IF I can use CSSS insted of Images...Tell me how : ?

To remove square from all images, but not from main [ [square]blacksun] ?? I will try :) But all FUN is that we will sell t-shirts... designs of T-shirts will create I and 2 my friends... And all friends are studying art school, deisgn, interier design..and nobody had never said: Remove thiis squares :D

You dont like my self-done buttons ?? If text will not be bold...It will lose in background...and It doesnt like very well...after a few minutes you will fill pain in your eyes..

Why dont you like TimeNewRomans???
I'll never be the same...

#6 Gast

Gast
  • Members
  • PipPipPip
  • Advanced Member
  • 131 posts
  • LocationSouth East England

Posted 24 July 2006 - 02:15 PM

I want to sorry :) BUt Username and Password I've posted isnt correct.. I was looking to another database :D
So If you want you can login there : admin | admin
Sorry... :P

Your naviagation is made of images, which isn't need instead you could produce the same result with CSS.

I am not sure that i understand this sentence... IF I can use CSSS insted of Images...Tell me how : ?

To remove square from all images, but not from main [ [square]blacksun] ?? I will try :) But all FUN is that we will sell t-shirts... designs of T-shirts will create I and 2 my friends... And all friends are studying art school, deisgn, interier design..and nobody had never said: Remove thiis squares :D

You dont like my self-done buttons ?? If text will not be bold...It will lose in background...and It doesnt like very well...after a few minutes you will fill pain in your eyes..

Why dont you like TimeNewRomans???


It is nice, but like wildteen88 said, you should lose the Times New Roman font. Why? Because it is horrible. You should only really stick to Arial, Verdana, Tahoma or maybe Trebuchet MS. You should also make the left navigation nicer.
OS and Internet Specs:
Windows XP (with SP2), 1680x1050, PHP 4, MySQL 3, Internet Explorer 6 &7 /FireFox 2.0/Opera 9.2

Tutorials:
Simple Introduction to AJAX and XMLHttpRequest
Creating Live Data with AJAX Making a Google Suggest like application

#7 crashmaster

crashmaster
  • Members
  • PipPipPip
  • Advanced Member
  • 169 posts
  • LocationCzech Republic, Prague

Posted 24 July 2006 - 02:17 PM

and 1 more question: why dont you like to use tabs ???or...why does many peoples say that using tabs is so noobish ?? I dont see any BAD side of using TABS..

Maybe coz I've never used <DIV>s :P

And If somebody knows , why using DIV is better than TABLES, tell me it...
-Konstantin
I'll never be the same...

#8 moberemk

moberemk
  • Members
  • PipPipPip
  • Advanced Member
  • 695 posts

Posted 24 July 2006 - 02:59 PM

I'd link you to some articles on the subject, but they seem to be experiencing technical difficulties.
All I can say is that the code is easier, the design is more flexible, it makes semantic sense, and it makes your site more accessible to the disabled.
Disclaimer: No matter how harsh I may be, I'm always IMO only.

#9 crashmaster

crashmaster
  • Members
  • PipPipPip
  • Advanced Member
  • 169 posts
  • LocationCzech Republic, Prague

Posted 24 July 2006 - 04:16 PM

ok...any ideas why DIV is Better than TABLE ??
I'll never be the same...

#10 wildteen88

wildteen88
  • Staff Alumni
  • Advanced Member
  • 10,482 posts
  • LocationUK, Bournemouth

Posted 24 July 2006 - 04:26 PM

What moberemk said pretty much sums it up.

All I can say is that the code is easier, the design is more flexible, it makes semantic sense, and it makes your site more accessible to the disabled.

When you use tableless layouts you can reduce the amount of HTML code by probably more than 50%! Which can mean less file sizes and faster page loades. However your stylesheet size will go up.

I am trying to find a decent article I found when I was trying to understanding tables vs tableless layouts, I cant find it at the mo.

#11 crashmaster

crashmaster
  • Members
  • PipPipPip
  • Advanced Member
  • 169 posts
  • LocationCzech Republic, Prague

Posted 24 July 2006 - 04:40 PM

ok...I'd be happy, if you find it and post it here..
I'll never be the same...

#12 moberemk

moberemk
  • Members
  • PipPipPip
  • Advanced Member
  • 695 posts

Posted 24 July 2006 - 05:20 PM

I've got one!
http://www.sitepoint...e/tables-vs-css
Disclaimer: No matter how harsh I may be, I'm always IMO only.

#13 steviewdr

steviewdr
  • Moderators
  • Advanced Member
  • 1,364 posts
  • LocationIreland

Posted 24 July 2006 - 06:12 PM

CSS versus tables in my workds.

1. It gives mouse hover effects WITHOUT Javascript.
2. It renders HTML and Background Graphics Quicker than table background=""
3. All your CSS code is kept in an EXTERNAL Style Sheet. This means that clients when going through your site have the CSS and Styling Code CACHED on their computer -> so all they have to download is JUST the few <div></div> and html tags. 
4. If you want to change the colour of then awful standard blue <a href's etc. you can change it in 1 file and the whole site updates 
5. You can tell a background image to repeat in the X or Y direction, or both to form a nice gradient full browser width background.
and it goes on
and on and on....

Also - you have mysql errors on your page and I cant login.

Rgds,
Steve

#14 makeshift_theory

makeshift_theory
  • Members
  • PipPipPip
  • Advanced Member
  • 226 posts

Posted 25 July 2006 - 07:11 PM

+ css div's load quicker then tables and a lot of companies are transitioning to straight css designs, which you have to use divs for.  I work as a web designer and these standards are...forced upon me....makes me cry at night.... :P
<b>Emacs</b> it's great for the body.
<br><br>
<b>Trust the TechnoLust</b>

<b><u>Have a question check here:</u></b>
PHP Manual: http://www.php.net

#15 pixy

pixy
  • Members
  • PipPipPip
  • Advanced Member
  • 295 posts

Posted 25 July 2006 - 07:27 PM

It's simple...but maybe a little too simple? And I get these errors on the main page:

Array ( )
--------------------------------------------------------------------------------
Array ( )
--------------------------------------------------------------------------------

STATUS is 0
REF is
LEVEL is
Warning: array_keys(): The first argument should be an array in /srv2/www3/magistri.net/subdomains/blacksun/scripts/home.php on line 24

Warning: Wrong parameter count for max() in /srv2/www3/magistri.net/subdomains/blacksun/scripts/home.php on line 24

NUM_ITEM +1 is 1
--------------------------------------------------------------------------------
CLEAR $_S[cart]

This is a .44 Caliber Loveletter straight through my heart.

Tabulas + Threadless + Hire Me!


#16 akitchin

akitchin
  • Staff Alumni
  • Advanced Member
  • 2,516 posts
  • LocationCalgary, AB, Canada

Posted 25 July 2006 - 07:29 PM

when trying to login with demo/demo, i get referred back to this thread.  i'd suggest not sending someone back to their referrer if it's not from your site.

#17 makeshift_theory

makeshift_theory
  • Members
  • PipPipPip
  • Advanced Member
  • 226 posts

Posted 25 July 2006 - 07:56 PM

He said at the beginning "ignore the functionality."  ;D
<b>Emacs</b> it's great for the body.
<br><br>
<b>Trust the TechnoLust</b>

<b><u>Have a question check here:</u></b>
PHP Manual: http://www.php.net

#18 crashmaster

crashmaster
  • Members
  • PipPipPip
  • Advanced Member
  • 169 posts
  • LocationCzech Republic, Prague

Posted 25 July 2006 - 08:15 PM

I know..But it isnot Error :P
"Home" page is created for showing me, if variable has been setted...
For example
1st line: Array is showing me my cookies
2nd line: Array is showing me my $_SESSION variable
3rd line: STATUS is showing me if I am looged or not
4th line: is showing me $_SERVER[referer] variable
5th line: Level Is Showing me level of user, logged in... If you will log as main admin your LEVEL will be "2', if you will log as normal user your level is "0"... if you log as moderator your level is 1
6th line: NUM_ITEM -> look my topic at PHP Questions and Answers '$num++'
and 7th line: CLEAR $_S[cart] is link [action] which unsets $_SESSION[cart] variable ...

Its so simple :P
I'll never be the same...

#19 crashmaster

crashmaster
  • Members
  • PipPipPip
  • Advanced Member
  • 169 posts
  • LocationCzech Republic, Prague

Posted 25 July 2006 - 08:35 PM

And logining in..I dont know how do it ...
My problem is :
whn somebody is shopping..And wants to login, he just lOGIN , but if Logining is OK, it makes refesh to that place where he was before logining....before this script it was that after logining he was refreshed to deafut page for all users...

here is my script of logining

at the header of index.php
session_start();
require_once("scripts/auth.inc.php");
require_once("mysql.php");

	
	$action = $_GET[action];



if ($action == "login") 
{
check_database($_POST[username], $_POST[password]);
//this function will return :$db_status = 0 [if user and pass are bad] 
or db_status = 1 + $db_array [if user and pass are OK]

	if ($db_status == 1) {
				set_cookie($db_array,session_id());	
//and function set_cookie will setcookie (SESSION, session_id());
//also will set $auth_status = 1  			
					     }

}




if ($action == "logout") 
{

  $session = $_COOKIE[SESSION];
  $q = mysql_query(" DELETE FROM sessions WHERE session = '".$session."' ")
    or die(mysql_error());
	unset($_SESSION[user_id],$_SESSION[username],$_SESSION[user_level]);
  setcookie('SESSION', '', 0);
  header("location: index.php");
}	
	
	
check_auth($_COOKIE);

in the pace where  login form is :
		if ($auth_status == '1') 
		       {
	         logged_menu();
			   } else { 
						login_page(); 
					  }



and at last auth.inc.php:
function login_page()
{ 

	echo ' <table width="160" align="center" border="0" cellpadding="0">
      <form method="post" action="'.$_SERVER['HTTP_REFERER'].'&action=login">
      <tr>
        <td colspan="2"><img src="imgs/menu_login.gif"></td>
      </tr>
	  <tr>
        <td width="70">&nbsp;Username: </td>
        <td width="90"><input type="text" name="username" size="10" class="login_box"></td>
      </tr>
      <tr>
        <td>&nbsp;Password: </td>
        <td><input type="password" name="password" size="10" class="login_box"></td>
      </tr>  
      <tr>
        <td colspan="2">
		
		
<table width="100%" border="0" cellspacing="0" cellpadding="0">
  <tr>
    <td><a href="?page=register">&nbsp;<img src="imgs/button_register.png" border="0"></a></td>
    <td align="right"><input type="image" src="imgs/button_login.png" name="login" value="login">&nbsp;</td>
  </tr>
</table>


       </td>
      </tr>
	     </form>
	</table>';

}
 



function check_database($username, $password)
{

global $db_status;
global $db_array;

	
  $query = mysql_query("SELECT user_id, user_level, username FROM users WHERE username = '".$username."' AND password = '".$password."' LIMIT 1");
  $db_array =  mysql_fetch_array($query);
  $count = mysql_num_rows($query);
 
  if($count == 1) 
  {
  $db_status = 1; 
  }
  else {
  $db_status = 0; 
        }
  
  
}




function check_auth($cookie)
{ 
   global $auth_status;
   $session = $cookie[SESSION];

  $q = mysql_query(" SELECT user_id, username FROM sessions WHERE session = '".$session."' ")
    or die(mysql_error());
 
  if   (mysql_num_rows($q) == 1) {

     $r = mysql_fetch_array($q);
      $_SESSION['user_id']  = $r['user_id'];
      $_SESSION['username'] = $r['username'];
	  $q2 = mysql_query("SELECT user_level FROM users WHERE user_id = '$r[user_id]'");
	  $r2 =  mysql_fetch_array($q2);
	  $_SESSION['user_level']  = $r2['user_level'];
	 
	 
	  $auth_status = 1;
                                 } else { 
								 $auth_status = 0;
								 		}

 
  }




function set_cookie($db_array,$session)
{


 setcookie('SESSION', $session, time() + 60*60*24*21);
 
  
  
  $C = mysql_query(" SELECT * FROM sessions WHERE user_id = '".$db_array[user_id]."' AND username = '".$db_array[username]."' ");
  
  
  if(mysql_num_rows($C) == 0) {
    $Q = mysql_query(" INSERT INTO sessions (session, user_id, username) VALUES ('".$session."','".$db_array[user_id]."','".$db_array[username]."') ");
  							  } else {
    $Q = mysql_query(" UPDATE sessions SET session = '$session' WHERE user_id = '".$db_array[user_id]."' AND username = '".$db_array[username]."' ");
                                     }
								

refresh(0,$_SERVER['HTTP_REFERER']);

}




function logged_menu()
{
echo 'Now you are Logged IN';
//I've just cutted this part off, coz here is code that shows your user_menu when you are logged in
}




function fm($String)
{
  return addslashes(strip_tags(trim($String)));
}


I'll never be the same...

#20 crashmaster

crashmaster
  • Members
  • PipPipPip
  • Advanced Member
  • 169 posts
  • LocationCzech Republic, Prague

Posted 27 July 2006 - 09:03 PM

Hi..I 've just updated my web, now you can try also functional side of the web :P
Username: admin
password: admin
And...I have a question to designer..Look at the logo..
I think, that flower doesnt match to that text, blackSun but there is a flower...
I have another logo, but I dont like, whcih is the better one ...
look this one : Posted Image

or this one: Posted Image

or do you have any other ideas ??
[that blacksun is a  real logo of the clothes]
I'll never be the same...




0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users