Skip to content

fix: add horizontal padding to navbar toolbar to prevent navbar elements from sticking to edges (#230)#260

Merged
dricazenck merged 2 commits intoWomen-Coding-Community:mainfrom
IrynaLypnyk:fix/230-logo-padding
Mar 28, 2026
Merged

fix: add horizontal padding to navbar toolbar to prevent navbar elements from sticking to edges (#230)#260
dricazenck merged 2 commits intoWomen-Coding-Community:mainfrom
IrynaLypnyk:fix/230-logo-padding

Conversation

@IrynaLypnyk
Copy link
Copy Markdown
Contributor

Description

Add horizontal padding to navbar toolbar to prevent elements from sticking to screen edges.

Type

  • Bug Fix
  • New Feature
  • Code Refactor
  • Documentation
  • Other

Related Issue

Closes #230

Screenshots

Before:
Screenshot 2026-03-24 at 15 54 18
Screenshot 2026-03-24 at 15 54 46
Screenshot 2026-03-24 at 15 54 55

After:
Screenshot 2026-03-24 at 15 51 45
Screenshot 2026-03-24 at 15 51 30
Screenshot 2026-03-24 at 15 51 17

Testing

Tested locally.

Pull request checklist

Please check if your PR fulfills the following requirements:

  • I checked and followed the contributor guide
  • I have tested my changes locally.
  • I have added a screenshot from the website after I tested it locally

@IrynaLypnyk
Copy link
Copy Markdown
Contributor Author

Visual tests are failing due to the intentional logo position update in the mobile header.

I’ve verified the new UI locally and will update the Playwright snapshots accordingly.

@sonarqubecloud
Copy link
Copy Markdown

@IrynaLypnyk
Copy link
Copy Markdown
Contributor Author

I updated the visual snapshot after the intentional logo position change.

I also investigated the failing tests and ran them on the main branch — the same failures are reproducible there.

The affected navigation still works manually, but it is noticeably slow, and the current Playwright assertions appear to be timing out because of that.

So this does not appear to be introduced by this change.

@nora-weisser
Copy link
Copy Markdown
Contributor

@IrynaLypnyk thank you for fixing screenshot for the visual test! I will investigate other issues later and fix them.

@dricazenck dricazenck merged commit bcee015 into Women-Coding-Community:main Mar 28, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: WCC logo padding issue on mobile devices

4 participants