Skip to:
Content

BuddyPress.org

Opened 3 years ago

Closed 3 years ago

#8395 closed defect (bug) (fixed)

BP Nouveau needs some improvements to get along with Twenty Twenty One

Reported by: imath's profile imath Owned by: imath's profile imath
Milestone: 7.0.0 Priority: high
Severity: major Version:
Component: Templates Keywords: has-patch 2nd-opinion
Cc:

Description

Have you tried to activate the next WordPress default theme (twenty twenty one) ? BuddyPress parts are looking pretty ugly 😱. Have a look below:

https://cldup.com/UaoanvHpNW.png

I've started working on improvements from this GH branch and I believe we should start enjoying CSS variables like this theme does!

I think this is important we fix this ticket before 7.0.0 final release. As it's only CSS, I think we can finish this after Release Candidate is released.

Attachments (2)

8395.patch (138.7 KB) - added by imath 3 years ago.
8395.2.patch (139.8 KB) - added by imath 3 years ago.
Adds improvements made by @vapvarun

Download all attachments as: .zip

Change History (6)

@imath
3 years ago

#1 @imath
3 years ago

  • Keywords has-patch 2nd-opinion added; needs-patch removed

It took me longer than I thought to improve the way BuddyPress parts are looking into Twenty Twenty-One. Unlike what I thought there were more than one or two things to improve. The reason is: Twenty Twenty-One comes with a dark mode. And as you can see in this screenshot, when users choose this mode BuddyPress parts are looking even worse.

https://cldup.com/vc9uUCIlLS.png

The interesting thing about Twenty Twenty-One is the use of CSS variables. I believe we should use these into our Template packs to make it easier to customize them. As the number of SCSS lines were grewing, it became important to reintroduce into BP Nouveau the companion stylesheet feature so that people who doesn't use Twenty Twenty-One do not have to load the specific rules to this theme. Here's a screenshot showing the result of the patch:

https://cldup.com/Gkr1LGHc_Y.png

I believe, we should commit it and package a new release candidate (RC2) to let people eventually test it and improve it if needed.

Last edited 3 years ago by imath (previous) (diff)

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


3 years ago

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


3 years ago

@imath
3 years ago

Adds improvements made by @vapvarun

#4 @imath
3 years ago

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

In 12814:

BP Nouveau gets ready for Twenty Twenty-One

Improve the BP Nouveau template pack to make BuddyPress appearance looks great into the next WordPress default theme (Twenty Twenty-One).

Use the companion stylesheet feature to only load CSS rules specific to Twenty Twenty-One if it is the active theme or the parent of the active theme.

Props vapvarun

Fixes #8395

Note: See TracTickets for help on using tickets.