vibins — Mon, 08 Aug 2005 15:43:52 +0000

It reminds me of grand theft auto san andreas for some reson i think it has the same font

mscreashuns — Mon, 08 Aug 2005 14:44:54 +0000

It looks pretty good. The layout is very well done and the colors are great. I don't like how the links are set up. The link color is too dark for the background it is on, and then when you hover, a box appears around it and the color changes. That's not how I would've done it. You have a lot of links up in the top navigation bar. Maybe try combining some similar pages. Overall, nice work.

totalnewbie — Sat, 06 Aug 2005 05:41:13 +0000

I really like it!!! Very original!! Maybe I would change the word "merch" to "swag", but I did know what was meant right away! Maybe it's a more common term than "swag" anyway..

IanD — Thu, 21 Jul 2005 01:42:33 +0000

Ok, I rewrote the News and Journals pages so that they're split up into 25 entries/page. If anyone's interested here's the script for it:

<!-- #include virtual="\header.asp" -->
<!-- #include virtual="\database\dblink.asp" -->

<%
	Dim newsQuery
	Dim page
	Dim pages
	
	page = Request.QueryString("page")
	strLowerBound = Request.QueryString("page")
	
	
	' Determine number of pages based on how many records in DB
	objRS.Open "SELECT TOP 1 news_id FROM tbl_news ORDER BY news_id DESC", objDBConn
	pages = int(objRS("news_id") / 25)
	
	
	' Setup query
	If page = "all" Then
		newsQuery = "SELECT * FROM tbl_news ORDER BY news_date DESC"		
	Else
		If page = "" Then
			Response.Redirect "/news.asp?page=" & pages
		Else
			strLowerBound = strLowerBound * 25
		End If	

		strUpperBound = strLowerBound + 25
	
		newsQuery = "SELECT * FROM tbl_news WHERE news_id > " & strLowerBound & " AND news_id < " & strUpperBound & " ORDER BY news_date ASC"
	End If
	
	
	' Write navigation links

	Response.Write "<p style='text-align: center;'>"
	
	If page = "all" Then
	
		Response.Write "<a href='/news.asp?page=0'>Oldest</a> | <a href='/news.asp?page=" & pages & "'>Newest</a>"
	
	Else
		
		
		
		If page > 0 Then
				Response.Write "<a href='/news.asp?page=0'>Oldest</a> | <a href='/news.asp?page=" & page - 1 & "'>&laquo;</a>"
			Else
				Response.Write "Oldest | &laquo;"
			End If
	
	
		Response.Write " | <a href='/news.asp?page=all'>All</a> | "
	
	
		If cint(page) < pages Then
				Response.Write "<a href='/news.asp?page=" & page + 1 & "'>&raquo;</a> | <a href='/news.asp?page=" & pages & "'>Newest</a>"
			Else
				Response.Write "&raquo; | Newest"
			End If

	End If
	
	Response.Write "</p>"
	
	' Get records
	getNews newsQuery, "large"


	Set objRS = Nothing
	objDBConn.Close
	Set objDBConn = Nothing
%>

<!-- #include virtual="\footer.asp" -->

IanD — Wed, 20 Jul 2005 21:33:28 +0000

I had our graphics guy come up w/ a small image to separate entries on the front and also news/journal pages, I think that's helping some.

I also added some photos to the tour page. I'll probably choose different ones (ones that are actually from shows) and play around w/ them in Photoshop to blend them into the page and each other, but it at least gives an idea of what it'll look like.

Does anyone know how I can set the Journal div on the front page to only have a vertical scrollbar? It's not an inline frame, just a CSS element set w/ "overflow: auto". There isn't a horizontal-only/vertical-only property for overflow. It shouldn't actually have a horizontal scroll since the content doesn't overflow horizontally, and originally it didn't, but when I changed the font size it did. I guess the browser thinks it is overflowing. I tried adding extra padding but that didn't change anything.

IanD — Wed, 20 Jul 2005 19:45:25 +0000

Cool, thanks for the concern, I just don't have the same view towards it as you do. In most cases we'd probably agree about the need for conforming to standards, but this time I think it adds value without hurting anything, so I don't have a problem with it.

chris_bcn — Wed, 20 Jul 2005 19:17:20 +0000

I shall respectfully disagree and withdraw from the thread!

IanD — Wed, 20 Jul 2005 18:24:09 +0000

That's a really good idea Megan, I hadn't thought about that. Thanks

"Merch" is a pretty standard term in the subculture, I think most people will get it right away. And I don't think everything always has to be obvious. I realize people don't care about exploring a site most of the time and just want information, but in this case I don't think it's a big deal. Plus, I don't have the font the graphic designer used, so I can't change it anyway

IanD — Wed, 20 Jul 2005 18:18:34 +0000

The cursor changes onmouseover, that's pretty standard feedback for an image link.

Actually, I used CSS because it makes it easier to design and maintain the site, not for slimmer markup. It's 6 extra lines, I wouldn't really consider that bloated.

You're right about the news/journal pages being too long. Editing the query to pull older entries in sections of 25 is at the bottom of my TODO list, but I should probably do it now.

Megan — Wed, 20 Jul 2005 18:11:15 +0000

IanD wrote: "merch" stands for "merchendise."

Yes, I see that, but the point is that its not clear right away. Users shouldn't have to think to figure out what a navigation item means, it should be obvious. Store is obvious.

Quote: I agree about the news, tour and journal pages needing some details. I'm not sure about splitting them into multiple columns though because it could be confusing as to the chronology of the entries. I'm definitely open to suggestions about how to separate them visually though.

What about putting some photos along the side? Some action shots would be a good way to liven things up there. That would be a good connection for the event listings.