613-fix-logo-chrome-edge #614

Merged

b24johka merged 1 commit from 613-fix-logo-chrome-edge into team_1_week_3

2026-04-28 12:37:16 +00:00

b24erika commented

2026-04-28 10:07:21 +00:00

Collaborator

There was an issue in chrome/edge where logo wasent properly displayed, this was a minor fix which was adding align-item in the css file.

I have tested this on firefox/chrome/edge and works as it should.

There was an issue in chrome/edge where logo wasent properly displayed, this was a minor fix which was adding align-item in the css file. I have tested this on firefox/chrome/edge and works as it should.

b24erika added the

labels

2026-04-28 10:07:21 +00:00

b24erika self-assigned this

2026-04-28 10:07:22 +00:00

b24erika added 1 commit

2026-04-28 10:07:23 +00:00

fixed issue with logo displayed wrong 89669a28db

chrome/edge needed align items in order to have the same output.

b24erika referenced this pull request

2026-04-28 10:08:24 +00:00

Fix issue where center logo is much smaller on edge/chrome #613

a24timsv requested review from b24krila

2026-04-28 11:30:35 +00:00

b24krila commented

2026-04-28 11:48:31 +00:00

Collaborator

Review on #614

What has been Done.

Reviewed code and tested so that the navbar is displayed correctly.

Tested On Windows using Google Chrome and Microsoft edge.

App starts without error
Center logo is shown and looks good.

Some slight problems with the css based on our conventions but that is another issue.

What Needs to be done.

Nothing

Conclusion

The issue is fixed and can merged.

### Review on #614 ### What has been Done. Reviewed code and tested so that the navbar is displayed correctly. ### Tested On Windows using Google Chrome and Microsoft edge. - App starts without error - Center logo is shown and looks good. ### Inspected the following file Navbar.module.css - Some slight problems with the css based on our conventions but that is another issue. ### What Needs to be done. - Nothing ### Conclusion - The issue is fixed and can merged.