[buddypress-trac] [BuddyPress Trac] #9111: BuddyPress 12.3.0 can not click into the URLs settings screen.
buddypress-trac
noreply at wordpress.org
Mon Apr 1 12:55:27 UTC 2024
#9111: BuddyPress 12.3.0 can not click into the URLs settings screen.
-------------------------------------------------+-------------------------
Reporter: mervyntsao | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 12.5.0
Component: Core | Version: 8.0.0
Severity: normal | Resolution:
Keywords: needs-patch reporter-feedback dev- |
feedback |
-------------------------------------------------+-------------------------
Changes (by emaralive):
* cc: emaralive (added)
* keywords: => needs-patch reporter-feedback dev-feedback
* version: => 8.0.0
* milestone: Awaiting Review => 12.5.0
Comment:
Hi @mervyntsao, perhaps you could help the **!BuddyPress** Team narrow the
possible causes for your reported issue by providing additional
information. As it stands, the screenshot you have provided appears to be
a **composite**, e.g., composed of a number of images, and does not
provide enough information to identify a, possible, "**User Role**" when
this issue occurred. Any additional information that you could provide
would be greatly appreciated.
In the meantime, @imath & @vapvarun, since this ticket involves "**Hello
!BuddyPress**", of which @vapvarun had mentioned during the
[https://wordpress.slack.com/archives/C02RQBYUG/p1711566798146759 March
27, 2024 docs-chat] regarding a CSS rendering issue or the lack thereof,
making it hard to read some text within the "**Get Involved**" tab, there
is opportunity to fix the CSS issue within this ticket, as well.
So, first things first, it seems, at least to me, that "**Hello
!BuddyPress**" is intended for "**Site Administrators**" due to the type
of information provided and links to various "**!BuddyPress Settings**"
pages/screens, e.g., **bp-rewrites (URLs)** & **bp-credits (Credits)**
that require either the "**[https://wordpress.org/documentation/article
/roles-and-
capabilities/#administrator:~:text=manage_network_themes-,manage_network_options,-upgrade_network
manage_network_options]**" (**Multisite - Super Admin**) or
"**[https://wordpress.org/documentation/article/roles-and-
capabilities/#administrator:~:text=manage_links-,manage_options,-moderate_comments
manage_options]**" (**Single site - Administrator**) capabilities. The
code that provides the requisite capabilities is located on
**[https://github.com/buddypress/buddypress/blob/92ae36820deafc8c67fa6d615f4c8eb8affa6fbb/src
/bp-core/classes/class-bp-admin.php#L136 line 136 of src/bp-core/classes
/class-bp-admin.php]**. Thus, the following "**User Roles**" will receive
the "**Sorry, you are not allowed to access this page**" when trying to
access any page/screen that is associated with "**!BuddyPress Settings**"
(see **screenshot** screenshot-win10-me-2024.03.29-06_01_17.png):
- Subscriber
- Contributor
- Author
- Editor
Furthermore, Subscriber & Contributor view the "**Hello !BuddyPress**"
modal window differently than Author & Editor (see **screenshots**
screenshot-win10-me-2024-03-27-19_06_08.png & screenshot-
win10-me-2024.03.27-19_27_25.png, respectively) and the reason the 4
(four) indicated "**User Roles**" are able to view the "**Hello
!BuddyPress**" modal window is because the menu item is available to them
(see **screenshot** screenshot-win10-me-2024.03.27-13_42_38.png).
Given that there are a number of ways to provide a remedy, my thoughts are
that the simplest would be to have a conditional based on the
aforementioned capabilities which would dictate when the "**Hello
!BuddyPress**" menu item is displayed, i.e., display only for "**Super
Admin/Administrator**". An example of such would be to enclose
**[https://github.com/buddypress/buddypress/blob/92ae36820deafc8c67fa6d615f4c8eb8affa6fbb/src
/bp-core/classes/class-bp-admin.php#L188 line 188 of src/bp-core/classes
/class-bp-admin.php]** within a conditional such as:
{{{
if ( bp_current_user_can( 'bp_moderate' ) ) {
add_action( 'admin_bar_menu', array( $this, 'admin_bar_about_link' ),
100 );
}
}}}
Not knowing how others may feel about a remedy, I'll just offer my thought
as a proposal (see screenshot screenshot-win10-me-2024.03.29-07_09_51.png)
and wait for feedback.
Next up is the CSS issue that @vapvarun reported during the aforementioned
docs-chat (see screenshot screenshot-win10-me-2024.02.25-05_17_57.png). A
remedy would be to insert the following CSS rules after
**[https://github.com/buddypress/buddypress/blob/master/src/bp-
core/admin/css/hello.css#L295 line 295 of src/bp-
core/admin/css/hello.css]** &
**[https://github.com/buddypress/buddypress/blob/master/src/bp-
core/admin/css/hello-rtl.css#L295 line 295 of src/bp-core/admin/css/hello-
rtl.css]** (see **screenshot** screenshot-
win10-me-2024.02.25-05_35_32.png).
{{{
.bp-hello-content p.has-white-color.has-text-color.has-background
.has-medium-font-size {
color: white;
padding: 1.25em 2.375em !important;
}
.bp-hello-content p a mark.has-inline-color.has-white-color {
color: white;
}
.bp-hello-content p a {
text-decoration: none;
}
.bp-hello-content p a:hover {
color: #dd823b;
text-decoration: underline;
}
.bp-hello-content .wp-container-core-buttons-is-layout-1.wp-
container-core-buttons-is-layout-1 {
display: flex;
justify-content: center;
}
.bp-hello-content .has-vivid-red-background-color {
background-color: #be3631
}
.bp-hello-content .wp-block-button.is-style-fill a:hover {
text-decoration: underline;
}
.bp-hello-content .wp-block-button__link {
color: #fff;
box-shadow: none;
text-decoration: none;
padding: calc(.667em + 2px) calc(1.333em + 2px);
font-size: 1.125em;
}
}}}
Again, not knowing how others may want to provide a remedy for the CSS
issue, the indicated CSS rules are just a proposal and I'll wait for
feedback.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/9111#comment:2>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list