Support » Your WordPress » Review my Site/Theme

Viewing 15 replies - 1 through 15 (of 19 total)
  • Moderator Andrew Nevins

    (@anevins)

    Validate your website.

    Nobody gives a crap if your site validates or not. Especially considering it is almost impossible to validate when using css3 properties as you are. Maybe it’s important where Andrew works that his code validates, but in the real-world it’s usually not of any concern.

    • Looks good. I like the simple design.
    • Loads slowly. I assume this is on shared hosting which will explain the lag when loading. Sometimes times out without loading entire page.
    • “Comments RSS” link in footer goes to broken/missing page
    • I like the use of shortcodes
    • Good job on making it device responsive. All sites built nowadays should be doing this.

    Overall, I think it’s a nice clean responsive design. Good work.

    Moderator Andrew Nevins

    (@anevins)

    Please don’t take this as an excuse to argue; but in the real world websites need to be browser-compatible to reach realistic audiences and conforming to W3C’s standards helps achieve this.

    Validation is important.

    I thought the post was to critique the website in question, not argue about code validation.

    Fact of the matter is 100% of users and 99% of web developers don’t care if the code if valid. Only thing that actually matters and people care about is that the pages display correctly. Only anal-retentive programmers that think they are superior believe that CSS and HTML needs to validate. It just simply is not important in most cases and most anybody that knows what they are talking about will agree it’s a waste of time.

    Seriously, when was the last time you left a site because the code didn’t validate? You kidding me?

    Nevins – If it’s so important, then why do none of your sites validate? Do you not practice what you preach?

    [Links moderated. There was no need for that]

    Moderator Andrew Nevins

    (@anevins)

    Please don’t take this as an excuse to argue

    Moderator Jan Dembowski

    (@jdembowski)

    Brute Squad and Volunteer Moderator

    *Blows whistle and makes the time out sign*

    @codebotics Nice site and layout. Constructively you may want to address some of those validation issues. Or not, your call. 😉

    Thanks for your feedback everyone.

    I will validate where possible, although it’s not always possible. Using my site as an example, webkitallowfullscreen and all the other allowfullscreen‘s are part of the embed code copied directly from Vimeo. This stuff isn’t hard coded into the theme, and if it’s a problem I’m sure Vimeo wouldn’t still be adding it to it’s own embed code.

    As for the rest, I should have used classes instead of ID’s and I will fix that now.

    @chomachomachoma thank you for the specific feedback, yes I am currently using a shared host unfortunately. This might be sorted in the future. The comments feed wasn’t working as there were no comments, I’ve added a comment now to fix that! I might add an if statement to hide the comments link if there are no comments as a better fix though.

    Thanks again everyone, and feel free to check the validator in a few minutes time!

    Also, regarding the duplicate ID’s surrounding the WP calendar widget – for now I’ll just remove one of the widgets from my home page, as it’s WP generated code.

    Moderator Andrew Nevins

    (@anevins)

    Yes but do check whether browser issues are now resolved (if you haven’t already).
    http://www.browserstack.com/

    I don’t have browser stack, but I’ve tested it on the major browsers. Haven’t tested below IE8.

    Michael

    (@alchymyth)

    Forum Moderator

    imho, in this case, pointing out validation errors was correct and important, as the review was about a (new?) theme, not just a site.

    re older browsers – the menu drops behind the slide and is unusable in IE7;

    in FF16, when narrowing the browser, there is some 10-20% extra horizontal scrollbar space.

    is the theme licensed under GPL?

    I agree, it is important. And yep, new theme.

    Do you think it’s worth trying to fix the IE7 issue? Surely a line has to be draw somewhere with IE, but where?

    Thanks for pointing that out, there’s a little scrolling in Chrome as well, but nowhere near as much as in FF. I’ll look into fixing that shortly.

    The PHP files are licensed under GPL, as required. Everything else will be licensed under a single use license.

    Thanks for your comments.

Viewing 15 replies - 1 through 15 (of 19 total)
  • The topic ‘Review my Site/Theme’ is closed to new replies.