HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.

Positive/Negative class names

SudoCatSudoCat Drowning under a Sea of Clients New

Hi,

For the theme I'm having to make for a client at the moment, he wants the counts colour coded so that positive reactions have a green count, and negative red. I've added a class name to do this myself, however I have to re-apply these changes after each update, which is a little annoying. I could see this functionality being useful to others, so I was wondering if there was any possibility of this being included in a future update?

Here is the code I am currently using for it
// /yaga/library/functions.render.php - Starting from line 22 foreach($Reactions as $Action) { if(CheckPermission($Action->Permission)) { $CountString = ($ShowCount && $Action->Count) ? $Action->Count : ''; $Alignment = ( $Action->AwardValue > 0 ? ' ReactPositive' : ' ReactNegative' ); // Code Added $ActionsString .= Anchor( Wrap(' ', 'span', array('class' => 'ReactSprite React-' . $Action->ActionID . ' ' . $Action->CssClass)) . WrapIf($CountString, 'span', array('class' => 'Count')) . Wrap($Action->Name, 'span', array('class' => 'ReactLabel')), 'react/' . $Type . '/' . $ID . '/' . $Action->ActionID, array( 'class' => 'Hijack ReactButton' . $Alignment, // Code Added 'title' => $Action->Tooltip) ); } }

Thanks :)

Comments

  • hgtonighthgtonight ∞ · New Moderator

    Thanks for using my addon!

    You can currently modify the CSS class for actions in the dashboard. When editing an action, click on the advanced options label and append your desired class after the current class.

    Search first

    Check out the Documentation! We are always looking for new content and pull requests.

    Click on insightful, awesome, and funny reactions to thank community volunteers for their valuable posts.

  • R_JR_J Ex-Fanboy Munich Admin

    I guess you hear it quite often, but I think I haven't said it by now: that application is a versatile masterpiece!

  • SudoCatSudoCat Drowning under a Sea of Clients New
    edited August 2014

    Yeah true, it was just much easier to hardcode it as the client is yet to set up their desired reactions.

    @R_J said:
    I guess you hear it quite often, but I think I haven't said it by now: that application is a versatile masterpiece!

    I have to agree with you there, probably the best piece of code I've seen for Vanilla yet.

    EDIT: I'm starting to think the only users on this forum are you two and peregrine!

  • SudoCatSudoCat Drowning under a Sea of Clients New

    @hgtonight‌ Also I'm not sure if this is just for me, but there appears to be a closing div missing somewhere within YAGA. When I enable the plugin, my #Panel moves inside of my #Content. For the time being I've added an extra to the dashboard/view/profile/index.php

  • hgtonighthgtonight ∞ · New Moderator
    edited August 2014

    @SudoCat said:
    hgtonight‌ Also I'm not sure if this is just for me, but there appears to be a closing div missing somewhere within YAGA. When I enable the plugin, my #Panel moves inside of my #Content. For the time being I've added an extra

    to the dashboard/view/profile/index.php

    I am very interested in finding bugs like this.

    • What theme are you using?
    • What page do you notice it on?
    • What modules are in the panel?

    Also, thanks for the kind words :)

    Search first

    Check out the Documentation! We are always looking for new content and pull requests.

    Click on insightful, awesome, and funny reactions to thank community volunteers for their valuable posts.

  • SudoCatSudoCat Drowning under a Sea of Clients New
    edited August 2014

    @hgtonight‌

    Theme

    Custom Theme; However it can be replicated within the default theme: If you change
    <div class="Column PanelColumn" id="Panel"> {module name="MeModule"} {asset name="Panel"} </div> <div class="Column ContentColumn" id="Content">{asset name="Content"}</div>
    to
    <div class="Column ContentColumn" id="Content">{asset name="Content"}</div> <div class="Column PanelColumn" id="Panel"> {module name="MeModule"} {asset name="Panel"} </div>
    (i.e. move the panel to after the content)

    Page

    Viewing profile (e.g. http://vanillaforums.org/profile/54556/SudoCat )

    Panel Modules

    Error persists with all other addons disabled. Only modules are the Photo and ProfileFilters.

  • hgtonighthgtonight ∞ · New Moderator

    You found a bug in the Reaction Count display. The fix is on GitHub in this commit. It will be included in the next release.

    Thanks for helping me make my addon better!

    Search first

    Check out the Documentation! We are always looking for new content and pull requests.

    Click on insightful, awesome, and funny reactions to thank community volunteers for their valuable posts.

  • SudoCatSudoCat Drowning under a Sea of Clients New

    @hgtonight said:
    Thanks for helping me make my addon better!

    Not a problem! Glad to try and contribute to the community :)

Sign In or Register to comment.