New website crit

Russinating · 17 Sep 2015 at 10:38

Just working on a new site for us, appreciate all feedback/crit/bug spots. Going for a more modern metro style design that more prominently features content.

At the moment the only links on the homepage and clients that work are The Royal Mint (example of multiple projects) and Magners (example of a single project) to give you an idea of their page's layouts.

Address: http://new.jonesmillbank.com

The existing site is at www.jonesmillbank.com if you want to compare and contrast.

Thanks in advance!

xbenjiiman · 17 Sep 2015 at 11:56

Looks nice.

Though after clicking a few links, I managed to get to a URL such as
http://new.jonesmillbank.com/our-cl...lients/our-clients/our-clients/harbour-hotels

You might want to make links relative to the root, such as:

Code:

<a href="/our-clients/wcs"></a>

Russinating · 17 Sep 2015 at 12:09

Yeah a load of the links are broken

.

roccles · 17 Sep 2015 at 13:37

Looks good, Have you thought about moving the side navigation to the top on smaller screens such a mobile? The right hand side seems very responsive but the left hand navigation doesnt and seems to take up a lot of screen space on smaller screens.

Craig R · 17 Sep 2015 at 14:11

As above, you need to fix the navigation for mobile devices.

dacads · 17 Sep 2015 at 14:20

Maybe make the navigation text a bit bigger?

No I'm not blind

Russinating · 17 Sep 2015 at 14:41

Yeah I'll be sorting out the mobile bits once it's all in a final state; font size obviously increased across the board and the nav probably changing to a menu icon.

ak22 · 17 Sep 2015 at 14:46

Nice and clean i think the left menu should be bigger or at least make the text bigger.

Russinating · 17 Sep 2015 at 14:56

Thanks everyone.

Also trying to work out the best way of making the content blocks float left, but also up. For example below, the two small blocks on their own would ideally slot up above, like an upside down tetris. In fact the bottom right square should fall in-line and be underneath the text block, rather than being 'blocked' by the longer horizontal one.

Any pointers?

lsg1r · 17 Sep 2015 at 16:38

there's scripts out there that already do the 'sorting' for you, there is a term for it but I can't remember what it is

This is one as an example although I'm sure there are others.
http://www.final-tiles-gallery.com/index.html

roccles · 17 Sep 2015 at 18:35

Masonry?

Russinating · 17 Sep 2015 at 18:39

Yeah found Masonry. Presume there's no CSS alternative?

uniQ · 17 Sep 2015 at 19:20

Google?

https://www.google.co.uk/search?cli...&oe=UTF-8&gfe_rd=cr&ei=LQT7VZbKEcOA8Qe-w4HwBw

lsg1r · 17 Sep 2015 at 19:20

Russinating said:
Yeah found Masonry. Presume there's no CSS alternative?

not that I know of with different height panels, if they were all the same height you could probably do it though.

edit: actually thinking about it you could kind of do it in css but it wouldn't be as flexible in layout like the js option

roccles · 17 Sep 2015 at 20:09

Haven't read through the whole article but this claims to be css only masonry effect

Link

Edit: sorry just seen google link above which shows that as 2nd link

roccles · 17 Sep 2015 at 20:17

I notice that you are not using any css frameworks is this because you want to keep keep file size down/learning css? As something very light weight like purecss could give you a lot of the stuff you need without reinventing the wheel.

Trucamo · 17 Sep 2015 at 21:31

Looks nice! Much better than anything I could currently do.

Russinating · 17 Sep 2015 at 22:21

roccles said:
I notice that you are not using any css frameworks is this because you want to keep keep file size down/learning css? As something very light weight like purecss could give you a lot of the stuff you need without reinventing the wheel.

Yeah I like coding from scratch

. Means there's nothing there that's not needed (hopefully!) and I know what's what when something doesn't display right.

Finished off this page today which I'm quite happy with! http://new.jonesmillbank.com/our-clients/skiworld/. Pretty simple JS with the help of jQuery but works quite well I think (playing the individual videos).

xbenjiiman · 21 Sep 2015 at 09:24

Russinating said:
Yeah found Masonry. Presume there's no CSS alternative?

You might be able to do it with flexbox, but it's not supported in IE10 or below.

Russinating · 19 Oct 2015 at 21:06

Site went live today, changed a bit

www.jonesmillbank.com