Skip to:
Content

BuddyPress.org

Opened 14 months ago

Closed 14 months ago

Last modified 14 months ago

#7768 closed defect (bug) (fixed)

[BP Nouveau] Unstyled load newest link

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

Description

Hi everyone !

BP Nouveau really looks terrific ! Congrats. When there are newest activities to load, a link is added to the activity stream so that we can load them. This link appears unstyled.

https://cldup.com/iWYfu2kcqK.png

It's a JavaScrip issue in src/bp-templates/bp-nouveau/js/buddypress-activity.js where the li element is added to the parent of the ul.activity-list. If the attached patch is applyed, then it's really better :)

https://cldup.com/i68hj-eobe.png

PS: sorry screencaps are in french ;)

Attachments (1)

7768.patch (740 bytes) - added by imath 14 months ago.

Download all attachments as: .zip

Change History (5)

@imath
14 months ago

#1 @DJPaul
14 months ago

  • Keywords commit added

Hi @imath

It does look better! Thanks for testing and writing a patch.
If you want to commit it, please go ahead, otherwise someone else will add it later.

#2 @imath
14 months ago

  • Owner set to imath
  • Status changed from new to assigned

Hi @DJPaul thanks for your feedback, I'll commit it in a few hours (after office hours!).

#3 @imath
14 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 11992:

BP Nouveau: the newest link must be inserted into the activity stream

Fixes #7768

#4 @hnla
14 months ago

@imath just want to comment that we made some changes to avoid loading things like messages in act streams within the act items LI constructs so only concern here is adding the link in a LI element, styling looks fine though, and does need improving from what is current, thanks for patching.

Note: See TracTickets for help on using tickets.