- This topic is empty.
December 15, 2012 at 12:58 pm #41362
I have made [this static design](http://stevenwatson.co.uk/examples/spartans/ “Go to Spartans”) for my brother-in-law’s basketball team, and I’m just after a bit of critique.
I will converting it into a WP website once I have finalised the design and made a few other pages.
**Ps**. The page has only just been coded up tonight, so I may be changing a few things with it depending on how OCD’ish I get.
Thanks =)December 15, 2012 at 1:38 pm #117537
Looks pretty good.December 15, 2012 at 1:58 pm #117542ConorHainingParticipant
I don’t think there’s anything I can say there, it’s spot on!December 15, 2012 at 3:11 pm #117549
@jurotek – I have increased the font size in the navigation, I agree it was quite small, I liked uppercase too, but I prefer the increased font size more :)December 15, 2012 at 3:44 pm #117559
One thing I just want to point out is that the logo doesn’t really go with the color palette.December 15, 2012 at 4:28 pm #117563
@Paulie_D – the logo was made by the client, I will look into making it more compatible with the colour scheme or I may even change the colour of the ball to a more realistic colour rather than the dark brown.
Cheers :)December 15, 2012 at 5:10 pm #117578
Ha. I’m Chris, not Paulie.December 15, 2012 at 5:39 pm #117581
Wow! I can’t believe I failed as much as that…apologies man.
Looks like Paulie is going to get an unnecessary email :/December 15, 2012 at 8:13 pm #117587
Haha. It’s not a problem.December 15, 2012 at 11:25 pm #117593TheInfectionParticipant
Try adding a wrapper around everything but the header and then apply a box-shadow on it. Using
box-shadow: 0 0 5px #000;
box-shadow: 0 0 15px #000;
look the best.
Also, try using jQuery for the animations as CSS3 isn’t supported in older browsers, unless you don’t really mind that.
Other than that, you’ve got a great design there!December 16, 2012 at 5:18 am #117601
@TheInfection – thanks, I think I will add a subtle box shadow at some stage of development, I will also be using jQuery for the transitions on the links for the extended browser support, although if a user who has a non-supported browser then it’s not like they’re going to experience something totally different, it’s just going to go to the solid colour on hover a bit quicker.
@jurotek – thanks for the input but i think I’ll leave my nav as it is now :)
@joshuanhibbert – thanks for the detailed feedback. I was using the inset box-shadow on the navigation hover also on the previous design so the picture would flow nicely but now it doesn’t I can see what you mean :) it will be a jQuery slider using a plugin for wp anyway, I just left it there for some structure.
I know about the page title that will be getting changed, haha, +1 for noticing though ;)
Regarding the gradient, it’s the first time I’ve used it in this way, I kind of like it but maybe it’s just a preference thing :)
Yeah I should add some more padding here and there to balance out the page. I’ll get to that definitly.
No the footer doesn’t need to be as ‘in your face’ like the navigation. I will look into seeing what colour goes good :) and thanks!December 16, 2012 at 7:08 am #117606
I will be adding current page styling to the tabs, but I’ll just be using the `.current-page-item` class to do it in WordPress :)December 16, 2012 at 5:19 pm #117716stevencraderMember
I know this isn’t design related but your charset tag is incorrect. You have `
` when it should be ``December 16, 2012 at 5:30 pm #117719stevencraderMember
@jurotek It isn’t the closing that I was pointing out. It is the dash. There is no `meta-charset` tag. Guess I should have explained a bit more.December 17, 2012 at 12:23 am #117744hendrix940Participant
- The forum ‘Other’ is closed to new topics and replies.