Olivero: my account focus color not matching with other items

Applied patch #2 and it works My account and logout links.

Before patch:

After patch:

#7 Patch failed

#11 please review the patch

Patch works as intended.

Tested in a Ipad screen (768x1024)

Home focus:

My account and Log Out Focus before:

My accout and Log Out focus after patch:

RTBC +1

Verified and tested patch #11.
Patch applied successfully and looks good to me.

Testing Steps:
# Goto any Responsiveness devices (Mobile, iPad)
# Goto: Appearance -> Install Olivero theme.
# Go to Homepage -> Click on hamburger menu
# Press Tab key and check the focus color which is not matching for My account and a logout button.

Expected Results:
# Focus color for My account and a logout button should match with other items.

Actual Results:
# My account and logout focus color not matching with other items

Looks good to me.
Can be a move to RTBC.

Reran the test for #11. The previous test failing doesn't seem to be related to the patch. Reviewed the patch and found that it works correctly. RTBC + 1

+++ b/core/themes/olivero/css/components/navigation/nav-secondary.pcss.css
@@ -66,6 +66,23 @@
+        height: var(--sp3);

Setting this to a fixed height results in text potentially overflowing the container, as seen in this screenshot. Unfortunately, there's a reasonable enough chance that this might occur to justify finding a solution that accommodates links with longer text content.

I was not able to thoroughly check every change, but noticed this during a brief review and wanted to be sure it was reported.

The long text bug is a good catch but this issue is specific to the color of the focus and that issue seems to be addressed by the previous patch. Hence I would recommend to open a separate issue for it, to avoid increasing the scope of this issue.

Applied patch #11 working fine and giving expected result
Thanks for the patch
for ref sharing screenshot....

Verified and tested patch #11 in drupal-9.5.x-dev.
Patch applied successfully and looks good to me.
Can be a move to RTBC.

Verified and tested patch #11 on Drupal 9.5.x-dev. Patch applied successfully and looks good to me.

Testing steps:
1. Go to admin/Appearance
2. Set Olivero theme in the Administration theme (Scroll down and see)
3. Save configuration
4. Check on the responsive layout (Mobile)
5. Observe the My account and Log out focus color not matching with other items
5. Apply the patch
6. Reload the page. Now check, that the My account and Log out focus color match with other items

Testing Result:
1. After applying the patch the My account and Log out focus color match with other items

Can be moved to RTBC

I personally don't think this is even a bug.

That being said, the patch in #11 uses pseudo-elements. Is there a reason why?

If we do change this, we really only need to modify the color of the current outline.

Kindly review patch for version 9.5x,,

#30 Patch applied successfully and looks good to me.

Testing Steps:
# Goto any Responsiveness devices (Mobile, iPad)
# Goto: Appearance -> Install Olivero theme.
# Go to Homepage -> Click on hamburger menu
# Press Tab key and check the focus color which is not matching for My account and a logout button.

Expected Results:
# Focus color for My account and a logout button should match with other items.

Actual Results:
# My account and logout focus color not matching with other items

Looks good to me.
Can be a move to RTBC.

Screen short for reference:

Before patch

After Patch

AFAIK this is how the focus ring in Olivero is designed to work. This is confirmed by @mherchel who is a subsystem maintainer of Olivero in #28.