Skip to:
Content

BuddyPress.org

Opened 9 years ago

Closed 9 years ago

#7715 closed defect (bug) (fixed)

BP-Nouveau: Give visual clue for pencil icon for Messages subnav link

Reported by: mercime Owned by: hnla
Priority: normal Milestone: 3.0
Component: Templates Version:
Severity: normal Keywords: has-patch
Cc:

Description

Screen reader users will hear that the link is for composing messages (within <span class="bp-screen-reader-text">), but some users won't know what the pencil icon represents.

Attachments (1)

7715.patch​ (6.1 KB ) - added by mercime 9 years ago.

Download all attachments as: .zip

Change History (8)

#1 @hnla
9 years ago

It's one of those icons I would have wanted our bp tooltip implementation on, so think we should look to doing that or better revert to a link text :)

#2 @mercime
9 years ago

I was thinking of reverting the compose link from icon to text at this time. Attaching patch.

@mercime
9 years ago

#3 @DJPaul
9 years ago

  • Milestone Awaiting Review → 3.0
  • Owner set to hnla
  • Status new → assigned

#4 @mercime
9 years ago

  • Keywords has-patch added

#5 @mercime
9 years ago

@hnla when you get the time, please check attached patch which removes the pencil icon and makes the link all text.

#6 @DJPaul
9 years ago

Please may we aim to have this in trunk by next Wednesday 4th April please for beta 1?

#7 @hnla
9 years ago

  • Resolution → fixed
  • Status assigned → closed

In 11916:

Nouveau: Re-factor Compose pencil icon

Commit changes the compose link to a text one removing the pencil icon.

As an icon it might not have been clear to the purpose of the link, as text it's clearer and is also easier to handle styling/layout within the li element structure.

Props mercime

Fixes #7715

Note: See TracTickets for help on using tickets.