Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the user flair of the top ten viewchangers stand out a bit. #18

Closed
Snorrrlax opened this issue Oct 3, 2013 · 4 comments
Closed

Comments

@Snorrrlax
Copy link
Collaborator

This is partly a CSS job, as we'd have to add some code to the subreddit stylesheet, but it should be fairly simple:

Currently, all user flairs have the css class "points". For the top ten viewchangers, this would need to be changed to something else. Perhaps the number one viewchanger could be different altogether. For example:

Normal flair css class: "points".
Leaderboard numbers 10-2 css class: "points1".
Leaderboard number 1 css class: "points2".

Any random three words would do.

This should be for the top ten viewchangers of that month, not all time. Someone's legacy may live on in the all time leaderboard but it would be a better idea to pass on the "golden flair" to the most active viewchanger, in my opinion.

Also, feel free to suggest which colours to use. I was thinking numbers 10-2 could have a golden border, and number 1 would be fully golden (but not too bright).

@PixelOrange
Copy link
Contributor

Enacting this would be easy. Removing it once someone drops off the list might be tricky.

@Snorrrlax
Copy link
Collaborator Author

I've talked about this further here. Might as well copy/paste it so anyone who's unsure can understand the idea/point out the flaws in my logic:

Thanks for taking a look!

At least for the leaderboard flairs, twould be easier for an outsider like me if you went ahead and created some CSS for me to plug in.

I was under the impression that all you needed for the code was the CSS class (which could be any word), and the rest of the CSS would just be in the stylesheet, not the deltabot code? I say this because we originally added the delta symbol after the number on the user flairs using CSS. Here's the current piece of code from the config file:

"flair": {
    "point_text": "%s∆",
    "css_class": "points"
 },

But it would have originally been:

"flair": {
    "point_text": "%s",
    "css_class": "points"
},

In conjunction with this in the subreddit stylesheet:

 /* Post-text for the user flair added by deltabot */
    span.flair.flair-points:after {
    content: "∆";
 }

In fact, I'm not sure why "css_class": "points" hasn't been removed from that part of the config, seeing as it's no longer needed (I think).

So I thought that someone would write the deltabot code that followed the logic of:

  • Number 1 viewchanger of the month has CSS class "x"
  • The remaining 9 viewchangers of the month have CSS class "y"

And then in the subreddit stylesheet I'd include some code that went like:

  • CSS class x makes user flair full gold (for example)
  • CSS class y makes user flair have a gold border.

Apparently the hard part is writing deltabot code that changes the users this applies to when the month is up. I imagine it would work with the monthly leaderboard somehow, and remove the CSS class from the previous month's viewchangers.

@amorde
Copy link
Contributor

amorde commented Apr 15, 2014

I hope what I contributed can be of use to you. It's difficult to know if it works since I can't actually run it as DeltaBot, so please do give me some feedback on any problems you run into.

@PixelOrange
Copy link
Contributor

Closed due to issue #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants