Skip to content

fix: cop30 banner decentralized in different browsers#2200

Merged
caneppelevitor merged 1 commit intostagefrom
Fix-cop-30-banner-decentralized-in-different-browsers
Jan 28, 2026
Merged

fix: cop30 banner decentralized in different browsers#2200
caneppelevitor merged 1 commit intostagefrom
Fix-cop-30-banner-decentralized-in-different-browsers

Conversation

@LuizFNJ
Copy link
Collaborator

@LuizFNJ LuizFNJ commented Jan 26, 2026

Description

Fixed the alignment of the COP30 banner, which was off-center in Safari and Firefox. I implemented a Flexbox layout on the banner container to ensure consistent horizontal centering across all browsers.

Changes: Added display: flex and justify-content: center to the banner styles.

before:
Screenshot 2026-01-27 at 17 20 50

after:
Screenshot 2026-01-27 at 17 18 44

Related Ticket #2199

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Testing

  1. Open the application in Safari and Firefox (or any non-Chrome browser).
  2. Navigate in homepage.
  3. Verify that the banner is perfectly centered horizontally.
  4. Check Chrome to ensure no regressions (it should still look correct).

Developer Checklist

General

  • Code is appropriately commented, particularly in hard-to-understand areas
  • Repository documentation has been updated (Readme.md) with additional steps required for a local environment setup.
  • No console.log or related logging is added.
  • No code is repeated/duplicated in violation of DRY. The exception to this is for new (MVP/Prototype) functionality where the abstraction layer may not be clear (comments should be added to explain the violation of DRY in these scenarios).
  • Documented with TSDoc all library and controller new functions

Frontend Changes

  • No new styling is added through CSS files (Unless it's a bugfix/hotfix)
  • All types are added correctly

Backend Changes

  • All endpoints are appropriately secured with Middleware authentication
  • All new endpoints have a interface schema defined

Tests

  • All existing unit and end to end tests pass across all services
  • Unit and end to end tests have been added to ensure backend APIs behave as expected

Test IDs

  • Include the test ID when adding new tasks or components.
  • Check that test IDs are present in the modified components.

Merge Request Review Checklist

  • An issue is linked to this PR and these changes meet the requirements outlined in the linked issue(s)
  • High risk and core workflows have been tested and verified in a local environment.
  • Enhancements or opportunities to improve performance, stability, security or code readability have been noted and documented in Project do Github issues if not being addressed.
  • Any dependent changes have been merged and published in downstream modules
  • Changes to multiple services can be deployed in parallel and independently. If not, changes should be broken out into separate merge requests and deployed in order.

@LuizFNJ LuizFNJ requested a review from thesocialdev as a code owner January 26, 2026 21:49
@LuizFNJ LuizFNJ changed the title Fix: using flex display and centering the banner Fix: cop30 banner decentralized in different browsers Jan 26, 2026
@LuizFNJ LuizFNJ changed the title Fix: cop30 banner decentralized in different browsers fix: cop30 banner decentralized in different browsers Jan 26, 2026
@LuizFNJ LuizFNJ force-pushed the Fix-cop-30-banner-decentralized-in-different-browsers branch 4 times, most recently from bcd37b3 to eb093c1 Compare January 27, 2026 16:19
@sonarqubecloud
Copy link

@caneppelevitor caneppelevitor merged commit de33a5c into stage Jan 28, 2026
12 of 16 checks passed
@LuizFNJ LuizFNJ mentioned this pull request Feb 25, 2026
46 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: QA

Development

Successfully merging this pull request may close these issues.

2 participants