Skip to:
Content

BuddyPress.org

Opened 7 years ago

Closed 7 years ago

#7723 closed defect (bug) (fixed)

BP-Nouveau: Cannot Star or Unstar in single message view

Reported by: mercime's profile mercime Owned by: imath's profile imath
Milestone: 3.0 Priority: normal
Severity: normal Version:
Component: Templates Keywords: has-screenshots has-patch commit
Cc:

Description

There's no way to do Star/Unstar using either mouse click, keyboard press, nor touch screen. Tooltip does not show up as well.

Attaching one animated gif showing the issue using mouse click, keyboard press, and touch screen not working on star.

Attachments (4)

star-unstar-single-message.gif (111.8 KB) - added by mercime 7 years ago.
7723.patch (3.9 KB) - added by imath 7 years ago.
7723-tooltips-etc.patch (6.7 KB) - added by mercime 7 years ago.
message-delete-star.png (14.1 KB) - added by mercime 7 years ago.

Download all attachments as: .zip

Change History (11)

#1 @mercime
7 years ago

  • Keywords has-screenshots added

#2 @DJPaul
7 years ago

  • Milestone changed from Awaiting Review to Awaiting Contributions

@imath
7 years ago

#3 @imath
7 years ago

  • Keywords has-patch reporter-feedback added
  • Milestone changed from Awaiting Contributions to 3.0

Hi !

7723.patch is a first attempt to improve feedbacks about starring/unstarring a message.

I also moved above the messages the feedback messages that were added at the bottom.

Is it better this way ?

This ticket was mentioned in Slack in #buddypress by imath. View the logs.


7 years ago

#5 @mercime
7 years ago

@imath. Patch works great. 7723-tooltips-etc.patch realigns the tooltips for the "Delete Conversation" and "Unstar Message" buttons as shown in image above as well as apply changes in original patch to sass file.

I also moved above the messages the feedback messages that were added at the bottom.
Is it better this way ?

+100 from me. Much easier for users to see the feedback on mobile and desktop.

Last edited 7 years ago by mercime (previous) (diff)

#6 @imath
7 years ago

  • Keywords commit added; reporter-feedback removed

@mercime Awesome ! thanks a lot for your feedback and for having updated the patch to include the sass thing 👌. I'll add it to the tonight's commit list ☺️

#7 @imath
7 years ago

  • Owner set to imath
  • Resolution set to fixed
  • Status changed from new to closed

In 12041:

BP Nouveau: add interactivity to starring buttons in single Messages

  • Make sure the BP Tooltip is displayed
  • Toggle the color of the star when starring unstarring
  • Move the feedback messages over the threads

Props mercime

Fixes #7723

Note: See TracTickets for help on using tickets.