Implemented Display a user's banners on their member card

[Chris D]

I created an add-on TM for this, but feel it could be worth including in XenForo 1.2.

http://xenforo.com/community/resources/user-banners-on-member-cards.2014/

 

[Andy.N]

Yes, always want to do this since 1.1.x versions.

 

[Jeremy]

Depending on how many banners + how much adds to the card, this could be come problematic.

 

[Chris D]

Surely that's true of displaying it in messages too? If you have 10 banners stacked then it'd look ridiculous. On a member card that would look even less-so, I think.

 

[Shelley]

Depending on how many banners + how much adds to the card, this could be come problematic.

I can definitely see potential problems with this the banners i'm using are huge. Full of hugeness.

 

[Jeremy]

Surely that's true of displaying it in messages too? If you have 10 banners stacked then it'd look ridiculous. On a member card that would look even less-so, I think.

Yeah, but the post expands. Member card is a static size, IIRC.

 

[Chris D]

The whole point is though, how many banners will be needed for that to even be a problem?

I did a very hacky cut and paste job in dev tools and 3 lines of banners, about 15 of them and there were no noticeable issues.

Maybe 30 or more could be an issue, but that just seems silly anyway

 

[Mike]

The member card is expandable in 1.2 (no image background).

 

[Jeremy]

And the staff proves me wrong. I'm still in the mindset of v1.1 with the member card. I'll go hide in the corner

 

[Paul B]

It's to do with responsive

 

[Jeremy]

It's to do with responsive

View attachment 49893

I know, I know. I'll take the public shaming. -_- LOL. I have no problem with this suggestion now. lol.

 

[Chris D]

I think it's only broken due to no spacing between the banners. Seems ok to me even with lots of banners on a small screen. Would expect larger banners to be ok too, @Shelley

 

[Shelley]

View attachment 49894

I think it's only broken due to no spacing between the banners. Seems ok to me even with lots of banners on a small screen. Would expect larger banners to be ok too, @Shelley

Yeah. My main concern is it's a design flaw on my part. I'm using images as banners and the curl is part of the banner so it'll look odd. I'll need to to re-visit it and re-code it so the lip is separate and removable in responsive.

KK you won me over, it gets a like from me but means I need to move my backside and redo my banners.

 

[Andy.N]

This has practical use for me as I use conditional to display the custom badge for highest level the member has achieved. Before, I have to go to their posts to see it but now it's possible to see it directly from member card.

 

[Paul B]

In 1.2 banners also appear on profile pages @Andy.N.

http://xenforo.com/community/members/brogan.521/

 

[James]

Nice idea. Instead of displaying every banner, I'm sure limits could apply. Whether that's highest display priority first (limit 5/10/etc) or just a checkbox on creation "display on membercard".

 

[Jeremy]

I'd prefer every banner personally, and since you can already choose highest ranked (I think Brogan made a suggestion that's now marked as [Implemented]), it should be repeated in here.

 

[Shelley]

View attachment 49894

I think it's only broken due to no spacing between the banners. Seems ok to me even with lots of banners on a small screen. Would expect larger banners to be ok too, @Shelley

Larger banners (btw nice add-on chris just installed) this is what i mean by larger. Could still be problematic for people using custom banners, using custom classes.

 

[Amaury]

It's to do with responsive

View attachment 49893

Is this how the member card will look in 1.2?

 

[Jeremy]

That's already how it looks, @Amaury. Resize your browser and it'll appear that way in Brogan's picture.