In forced color active: Toolbar navbar should be separate from each other

created a patch 3377198-2.patch for the issue.
Need review.

Testing Steps:
1. set Drupal with 11.x and enable claro as backend theme.
2. enable the forced color mode active in browser, and check the issue in toolbar.

After patch:

Note: In pcss file hex color code are used however use the follow the same not used variables.

wrong patch commit by mistake in last patch #2, commit the correct one 3378190-2.patch,
Needs Review

Hi @Harish1688
Verified your #3 patch on Drupal 9.5.8 and Drupal 11.x, the issue is fixed now. Added after and before screenshots.

@Harish1688, I applied your patch in Drupal 11.x. It is successfully applied. The border issue is now resolved. Attaching screenshot for the same. Moving to RTBC.

1. +++ b/core/themes/claro/css/theme/toolbar.theme.css
   @@ -175,3 +175,12 @@
   +  .toolbar-horizontal .toolbar .toolbar-toggle-orientation button.toolbar-icon {
   

   Can we remove the tag (button) from the selector? We usually try to avoid introducing additional dependencies on things like the tag when possible.

2. +++ b/core/themes/claro/css/theme/toolbar.theme.css
   @@ -175,3 +175,12 @@
   +    border-top: 1px solid #aaa;
   ...
   +    border-bottom: 1px solid #aaa;
   

   Nit: We could set the borders as transparent

Tried to address the pointers in #6.
Attaching patch as well as interdiff.

Hi@_utsavsharma, I have review your patch, applied successfully. It works fine and also button tag is removed as suggested by @lauriii attaching screenshot for the same.
moving it to RTBC.

The last submitted patch, 3: 3378190-2.patch, failed testing. View results →

The last submitted patch, 3: 3378190-2.patch, failed testing. View results →

Committed 83c6291 and pushed to 11.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.