Search

bbPress Showcase » Plugins by _ck_

Unread Posts - class added to SPAN & not TR

RSS

Tags:


  1. kevinjohngallagher
    User has not uploaded an avatar

    new member
    Joined: Nov '08
    Posts: 2

    offline

    My first time posting here, so I apologise if this seems trivial, but I think it's a major oversight.

    Basically, the Unread Posts plugin works really well, and does exactly what it says on the tin, but... the placement of the css code does cause some major issues.

    Its my opinion, that by outputting code directly to 1 cell in a table can give large usability and accessibility issues.

    First off, for those people attempting to emulate the look/feel/usability of say phpBB so that their users dont complain to much about the move, having the code in the TR so that the css styles can be inherited across all the TD or spans as needed makes perfect sense.

    Secondly, when making sites with accessibility in mind, or in my current case a forum aimed at people using screen readers - the ability to output code for the whole TR makes life 1000 times easier for those who are site impaired.

    Having seen the revisions for this plugin, i can see where you're coming from in getting a span with the appropriate class in it, but as with many things in BBpress - i think it's time to start moving specific inline css to a higher more modular level (which in fairness is what Wordpress does).

    Let me know if i can be of any assistance.

    Kev

    Posted 4 years ago #
  2. _ck_
    _ck_

    senior admin
    Joined: Jul '06
    Posts: 910

    offline

    http://bbpress.org/plugins/topic/unread-posts/#post-2211

    Kevin, I've now added that ability for you in 0.9.2

    change the optional setting near the top

    $unread_posts['add_row_class']=false;

    to

    $unread_posts['add_row_class']=true;

    and it will insert one of two different TR classes

    .unread_posts_row
    or
    .unread_login_row

    (unread_login_row means it's new since their last login)

    Posted 4 years ago #
  3. kevinjohngallagher
    User has not uploaded an avatar

    new member
    Joined: Nov '08
    Posts: 2

    offline

    Just to let everyone know, that there has been a new update to 0.9.3 of this plug that works really well.

    Again, thanks to _CK_ .

    Posted 4 years ago #

RSS feed for this topic

Post a reply to “Unread Posts - class added to SPAN & not TR”

You must log in to post.

keep _ck_ coding >> donate $5 <<     Theme Switcher:
26 users online from in the past 30 minutes. 3 bots 23 guests
2,375 views today 8,003 yesterday 89,978 peak. Most at once 37 today 75 yesterday 539 peak. Visited today: devinmarchan, hollylinney
2,430 posts in 416 topics over 79 months by 446 of 2,523 members. Latest: Caryflor80, nafiz2333, devinmarchan