WordPress.org

Ready to get started?Download WordPress

Forums

WP Dropdown Comments won't validate (13 posts)

  1. jafer
    Member
    Posted 9 years ago #

    I've been trying everything to change this plugin, and, although I've changed the div codes to span (which is how I got it to validate more than it was doing), I still can't figure out how to get it to validate because there's one code left that won't.

    Any ideas on how I can change the code, which doesn't validate because of this:

    'Line 153, column 23: document type does not allow element "ol" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag

    <ol class="commentlist">

    The mentioned element is not allowed to appear in the context in which you've placed it; the other mentioned elements are the only ones that are both allowed there and can contain the element mentioned. This might mean that you need a containing element, or possibly that you've forgotten to close a previous element.

    One possible cause for this message is that you have attempted to put a block-level element (such as "" or "<table>") inside an inline element (such as "<a>", "<span>", or "<font>"). '

    I can't change that to a p or div or span because then I get even more error messages.

    Help from anyone, please?

    You can see the actual invalidation here:
    link

  2. Joshua Sigar
    Member
    Posted 9 years ago #

    what kind of validation error you get when changing to "div"?

  3. jafer
    Member
    Posted 9 years ago #

    the same kind of validation errors, only more of them - not allowed to have this here without object, etc. etc. as above.

  4. Jinsan
    Member
    Posted 9 years ago #

    I had this problem with Kubrick. This theme is based on Kubrick (you really should leave Michael some credit, instead of stripping it all out). What happens when you change li id to li class? The comments file I have is pretty similar but it does validate. You can try it if you want, and see if it makes any difference

  5. jafer
    Member
    Posted 9 years ago #

    Jinsan, the question was about validating the plugin and using some other element which would validate since it's a block-level element within an inline element (see code above). I think the problem lays within the wp-dropdown-comments.php file.

    I've tried what you suggested and class vs. id doesn't make one bit of difference but I changed it to class anyway, in case I have more than one instance that could affect the list.

    I don't really appreciate being told who I should be crediting on my website when I've come to ask for help. It seems to me that I've got all kinds of WordPress links on there for anyone to check out, use, and choose a theme of their own. What next? A list of credits for all the plugins? People who helped me in the forum? Those who didn't? The books I've read?

    I'm here seeking help from anyone who may know the answer to the above, much like I help those who ask me for help.

  6. Jinsan
    Member
    Posted 9 years ago #

    I don't really appreciate being told who I should be crediting on my website

    I said you should, that's the same as I suggest, or I would if I were you. The reasons, simple courtesy that's all.

    A list of credits for all the plugins?

    This isn't uncommon even though try to play it down. In this instance, and in fact in every instance, it's entirely down to you. However, as I understand Kubrick was released under a certain guideline that SOME credit (even if it's the css) be left inside. Otherwise it's almost as though you're claiming the design and code all your own, which we both know isn't the case.

    It's a unique colour scheme, but it's still Kubrick and I don't think asking credit for the author for theme is too much of a discourtesy - if it bothers you so much to put a link on the front page, why not leave something in the CSS? Or is this is all about self-praise and self-credit?

    Any way it seems you have fixed your problem - the only thing not validating is your gravatar just place an alt="" attribute at the end like so:

    <img src="<?php gravatar("R", 40, "http://sekhu.net/wp/wp-content/gravatar/nogravatar.gif", "FF0000"); ?>" alt="ickle icons" />

    Or something to that effect.

  7. jafer
    Member
    Posted 9 years ago #

    Jinsan, if you look at the link to the invalidation which I posted in the first post, you will see that it still does not validate, and it is still the same problem which is as I mentioned in my 3rd post. Adding a missing alt tag is easily done and has nothing to do with the problem.

    And no, the theme is not definitely Kubrick, it is a mix of Kubrick and Manji and all the plugin css in it as well as my own. The look is totally created by me, all the graphics, not only the colour scheme, but not once on my site do I have any "self-praise" or "self-credit".

    It seems that you have a lot of nerve in trying to insult someone because they didn't include what you suggest, especially when this is a help forum where people are here asking for help, not for your personal ideas on what courtesies should and shouldn't be adopted.

    If it's a requirement to include the Kubrick credit inside the css, then I will simply include it, much like I give credit all over my site for different links, products, comments. Anything I've taken out of the css would have been to make the file smaller, but if it's required, then I'll put it in.

    I don't have a problem with giving credit, what I have a problem with is the fact that you're not providing help, you're providing your opinions on what I should include on my site. That's not what I'm here for.

    And if you're unable to provide help in a help forum, then I suggest you say nothing and save your insults and your opinionated ramblings for another venue.

  8. Jinsan
    Member
    Posted 9 years ago #

    I have an opinion so sue me, I won't mention credit again it's obviously a touchy subject :)

    I took a look at a random entry with the comments and it validated bar that alt tag with the gravatar

    The problem isn't you're using p within ol as I have the same and it validates.

    http://validator.w3.org/check?verbose=1&uri=http%3A//www.idiotonastick.com/talk/%3Fp%3D164%23comments

    That comment for example validates.

    I'm guessing it has something to do with this bit:

    <small>Comment by: <b><a href='http://www.all-things-me.net' rel='external nofollow'>Minna</a></b> — June 5, 2005 @ <a href="#comment-200">2:05 pm</a> </small>

    197: </li>

    I'm not sure if you can wrap a small tag within a p tag - additionally you're using a mix of xhtml and html marup - using hspace & b for example which are both deprecated.

    Try removing the p around the small tags. You can also try removing those b tags to see if it makes a difference

    EDIT2: try changing: <span id="comText168" style="display: none"> to a div and placing the style in your css - that should allow for the use of the p tag

  9. techwench
    Member
    Posted 9 years ago #

    the small tag works just fine within p tags.

    Looks like you're using your ol inside the span tag. Span is an inline element, whereas ol/li are blocklevel elements. You cannot put blocklevel elements inside inline. Only inline inside blocklevel.

    Try changing your spans to divs.

    edit: I see jinsan edited to include this, too. ;)

  10. jafer
    Member
    Posted 9 years ago #

    Ok, thanks for the input. I changed all the span codes back to div codes and what happened was that I got twice the error messages, i.e., 'Line 180, column 20: document type does not allow element "div" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag

    <div id="comLink171">

    The mentioned element is not allowed to appear in the context in which you've placed it; the other mentioned elements are the only ones that are both allowed there and can contain the element mentioned. This might mean that you need a containing element, or possibly that you've forgotten to close a previous element.

    One possible cause for this message is that you have attempted to put a block-level element (such as "" or "<table>") inside an inline element (such as "", "<span>", or "<font>").

    ✉

    Line 183, column 42: document type does not allow element "div" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag

    <div id="comText171" style="display: none">''

    So I changed the first one back to span - since I wanted that to display on the same line as the other comment features anyway, and tried it with just the second one as a div.

    So now instead of getting the OL not allowed here, I get the DIV not allowed here.

    I have tried changing these codes all over the place and can't get just that one thing.

    I remember in a previous version of WP that I had edited it and finally got it validated but I can't remember how I did that, or where the wrong code was in the first place, but it was with this very same plugin, only at that time it wasn't a plugin, just coding she'd modified from MT.

    If you have any ideas on code that's somewhere else that is impacting on this code, please feel free to let me know. I'm at a loss. Thanks.

  11. techwench
    Member
    Posted 9 years ago #

    Aah, I missed this earlier...You have divs within the p tag.

    <p class="postmetadata">

    That has to come out or move or change to a div or something.

  12. jafer
    Member
    Posted 9 years ago #

    That's it, techwench!!! You did it!

    Thank you so much, it's so hard to find where the errors are coming from when you're piecing together 3 and 4 files at a time.

    I'm forever indebted :)

  13. qamuri
    Member
    Posted 8 years ago #

    I had the same problem and I wanna thank techwench too!!! Thank you so much!!! I've very grateful!!

Topic Closed

This topic has been closed to new replies.

About this Topic

Tags

No tags yet.