Comments

dhtml

Fri 23 Nov, 2018 08:26 am

This is an excellent work, I will give it a very thorough review later. From the little I have seen so far, it is professionally done.

dhtml

Fri 23 Nov, 2018 11:18 am

Here are my first set of reviews:
1. Your url redirect pattern, try remove www. from it (be like facebook and africoders)
2. Your search, when i search and the result shows, and I click on the page. The search result should go away. Just like on this site, if you click to expand the notification from the bell icon, if you click on the page, the notifications go away.
3. Within the search result, I think you need a hover effect to highlight the current result your mouse is hovering on - again the notification of facebook and africoders do this.
4. Are you using google analytics?
5. Checking the console, I can spot -- smartchat-2.3.7.min.js:2 Refused to connect to 'wss://s31.smartsupp.com/chat/224/yfin03it/websocket' because it violates the following Content Security Policy directive: "default-src http: data: 'unsafe-inline' 'unsafe-eval'". Note that 'connect-src' was not explicitly set, so 'default-src' is used as a fallback +++ some other javaScript errors.
6. I think your social network links should open to a blank page, twitter, facebook since they are offsite links.
To be continued.....

dhtml

Fri 23 Nov, 2018 02:37 pm

I can see them, weldone. I will do more reviews later.

That your newsletter, when i dont put anything and just try to submit, it writes invalid email.

mreffk

Fri 23 Nov, 2018 07:27 pm

Yes that was they way I did it but I'll change it cause even to me it is somehow.

dhtml

Fri 23 Nov, 2018 08:53 pm

when there is no email, then the submit should not work when clicked in the first place.

dhtml

Fri 23 Nov, 2018 08:58 pm

Keep up the good work, you did an excellent job there. I will still do more reviews, i will even fire some random CSRF attacks on it later.