Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

#5697 closed enhancement (fixed)

New class to style visibility levels

Reported by: SGr33n Owned by: boonebgorges
Priority: normal Milestone: 2.1
Component: Settings Version:
Severity: normal Keywords: has-patch
Cc:

Description

Hi,

As anticipated on ​https://buddypress.trac.wordpress.org/ticket/5693, I would suggest to add a class for visibility settings level radio buttons, in order to make it possible to style them with icons.

So there is a patch to make it.

Attachments (1)

patch-li-class-visibility-level.patch​ (1.1 KB ) - added by SGr33n 12 years ago.

Download all attachments as: .zip

Change History (4)

@SGr33n
12 years ago

#1 @DJPaul
12 years ago

  • Component Core → Settings
  • Milestone Awaiting Review → 2.1

Thanks for the patch. The request seems reasonable.

We'll need to adjust the patch, slightly: printf( $r['before_radio'], $level['id'] ) should be printf( $r['before_radio'], esc_attr( $level['id'] ) ) in case someone figures out a way of inserting harmful XSS-type markup into the class= property. It also helps from a code review perspective that we know the output's being escaped safely in case a future change to BuddyPress accidentally introduces a problem.

#2 @SGr33n
12 years ago

Good :)

Thanks r-a-y!

#3 @boonebgorges
12 years ago

  • Owner set to boonebgorges
  • Resolution → fixed
  • Status new → closed

In 8606:

Add a CSS class to xprofile visibility radio buttons describing the visibility value

Fixes #5697

Props SGr33n

Note: See TracTickets for help on using tickets.