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 review

Featured Replies

Hello

Reviews of a website that I have done. I know there is more I need to do.

Is there something that you think I could add/improve?

 

My link

 

Thank you

The menu is really hard to read.

The navigation menu is till hard to read. May be you should make it a bit bolder. Will there be any content on the home page? Why don't you link the above logo to the home page from each page of the site?

  • Author

Have made the navigation slightly larger.

 

Client doesn't really know what they want on it. Am trying to extract from their old site but again there is very little on that as well. - My link

 

On the home page I was going to put 3 boxes underneath the main picture to About Us, Portfolio and newsletter sign up but don't know what I can elaborate on. Struggling with this and trying to make them look bigger than they really are.

Have made the navigation slightly larger.

 

Client doesn't really know what they want on it. Am trying to extract from their old site but again there is very little on that as well. - My link

 

On the home page I was going to put 3 boxes underneath the main picture to About Us, Portfolio and newsletter sign up but don't know what I can elaborate on. Struggling with this and trying to make them look bigger than they really are.

To be honest, the second link you posted is much better than the one in your original post. You need to design it before coding it. make a PSD of it and use dummy text for where you have no content (which the client should provide anyway)

  • Author

To be honest, the second link you posted is much better than the one in your original post. You need to design it before coding it. make a PSD of it and use dummy text for where you have no content (which the client should provide anyway)

 

The client seem happy with the way the new site is progressing and has given it the thumbs up. What did you like about the second link?

 

The original site was only 3 pages and there was an array of links all over it that didn't go anywhere. I wanted them to show off their work more and to link their brochure, e-newsletter and website so it looked more like a "family" with the use of the top band image (they were also in their brochure too)

The client seem happy with the way the new site is progressing and has given it the thumbs up. What did you like about the second link?

The menus were nice, the colour scheme pleasing (except for that 'orrible button thingy half way down the page) and I liked the layout.

  • 3 weeks later...
  • Author

Have made some little improvements but here - My link - the caption isn't where I wanted it to be. Have tried looking at the javascript but the default is still the same. Any thoughts?

 

<script type="text/javascript">
$(document).ready(function(){
$('.boxgrid.captionfull').hover(function(){
$(".cover", this).stop().animate({top:'125px'},{queue:false,duration:250});
}, function() {
$(".cover", this).stop().animate({top:'260px'},{queue:false,duration:250});
});
});
</script>

This applies to the 2nd link.

 

 

On hover the font size increases which causes the text to jump and in one case go from one line to two - I personally think hover works better when the font size remains the same.

Have made some little improvements but here - My link - the caption isn't where I wanted it to be. Have tried looking at the javascript but the default is still the same. Any thoughts?

 

<script type="text/javascript">
$(document).ready(function(){
$('.boxgrid.captionfull').hover(function(){
$(".cover", this).stop().animate({top:'125px'},{queue:false,duration:250});
}, function() {
$(".cover", this).stop().animate({top:'260px'},{queue:false,duration:250});
});
});
</script>

 

 

In your CSS, you've set the top position of the caption so that when the user initially visits the page, they cover the images, but when they hover they first vanish, then come back but lower placed.

 

This, should be your caption CSS:

 

.captionfull .boxcaption {
left: 0px;
top: 260px;
right: 0px;
bottom: 0px;
width: 100%;
}

 

Don't change the positioning in the CSS, because the script will over-ride it. Change it in the script, at the top of your page. Only change the top position in the CSS (the 260) to match the off-hover position in the script (also 260).

Edited by brightonmike

  • Author

Do you know where I can find a jquery slider with a transparent cation box that comes in from the side? Don't really know where to look...

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.