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

Already on GitHub? Sign in to your account

Include user flair class in body tag #498

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
4 participants
Contributor

rasher commented Aug 29, 2012

Would open some interesting customizing possibilities if the body tag included the user's flair classes, just like it includes the "loggedin" etc. classes.

This might cause surprises with some stylesheets (if they've been using the flair classes without being more specific about it), so maybe the class should be named differently (user-flair-xxx maybe).

You should post this in: http://www.reddit.com/r/ideasfortheadmins/. However I think while this opens up for a lot of things, it might not be that good and just add to confusion and maybe abuse.

Contributor

andre-d commented Aug 21, 2012

This bug tracker is for confirmed bugs in reddit, please post to /r/ideasfortheadmins.

@chromakode chromakode closed this Nov 20, 2012

Contributor

chromakode commented Nov 20, 2012

After considering the potential use (and abuse cases) for this, we've decided not to add it at this time. Thanks very much for taking the time to file this pull request.

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