Skip to content
View in the app

A better way to browse. Learn more.

Web Designer Forum

A full-screen app on your home screen with push notifications, badges and more.

To install this app on iOS and iPadOS
  1. Tap the Share icon in Safari
  2. Scroll the menu and tap Add to Home Screen.
  3. Tap Add in the top-right corner.
To install this app on Android
  1. Tap the 3-dot menu (⋮) in the top-right corner of the browser.
  2. Tap Add to Home screen or Install app.
  3. Confirm by tapping Install.

Website made for college assignment

Featured Replies

HI Matt, not too bad of a site certain elements are good but there is a lot that can be improved in my opinion. Starting from the top, I've no idea why your navigation is a table? Were you taught this? You shouldn't be using tables for any layout purposes, only an actual table. The navigation should be written similar to this -

<nav>
<ul>
<li><a>Home</a></li>
<li><a>Games</a></li>
<li><a>System</a></li>
<li><a>Videos</a></li>
</ul>
</nav>

 

and then the <li> elements styled to be inline-block and the <a> tags as display:block; so when hovering over the whole element the link is activated. Currently the only clickable part is the actual text rather than the button.

 

Also the header bar at the top could do with some padding top and bottom as the logo is pushed up right against the top and bottom of the black bar.

 

In regards to the layout it's not bad, not sure of the point of the 'hover to display' top 10 games list. Why not show it? The main improvement I would make is the typography on the site. Do some research into @font-face, your best bet would be to use Google Fonts.

 

The video box on the left is closer to the left hand margin than the right video box is to the right hand margin, if this was even it would look a lot better also. Also the 3 text boxes above have different outside space.

 

Again the 'read more' buttons aren't actually buttons as only the text is clickable. The whole button should be clickable.

 

That's all for now, thought I'd give you some feedback as you've not got any yet and there are improvements to be made.

  • Author

Yeah I know I didn't need any teaching for this lesson since I knew how to do it myself, I made the site 4 months ago and from then till now I know how to program jquery my html and css has gone miles better and im walking into php at work now. That was when I was just using html/css, if it was my choice id re do the whole site and start a fresh but lots of coursework has been done for it so I cant unfortunately but thanks for the review this would be extremely useful for me.

 

Thanks alot

 

Matt :)

You don't need to redo the whole site to improve your navigation. Personally, I try to make each site as professional as possible with what I know at the time meaning that if something is wrong I correct it. I don't see the point in asking for a review and then saying you can't do anything about it.

 

Don't mean to be hard on you but a future employer isn't going to care about the whys and wherefors and neither should your instructors. Jpwd had some good points and suggestions. It can't hurt to try them out.

Create an account or sign in to comment

Account

Navigation

Search

Search

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.