#732-footer-centering #735

Merged

a22erigr merged 1 commit from #732-footer-centered into team_2_week_4 2026-05-05 10:01:26 +00:00

Conversation 1 Commits 1 Files changed 2 +6 -2

c24carso commented 2026-05-05 08:36:40 +00:00

Collaborator

Copy link

Centered the footer columns by changing text align from left to center. Added scroll to when clicking the About Us link.

Centered the footer columns by changing text align from left to center. Added scroll to when clicking the About Us link. 

Screenshot 2026-05-05 102838.png

24 KiB

c24carso added the

needs review

label 2026-05-05 08:36:40 +00:00

c24carso added 1 commit 2026-05-05 08:36:41 +00:00

#732-footer-centering ... 1fe839977f

Centered the footer columns by changing text align from left to center. Added scroll to when clicking the About Us link.

c24carso referenced this pull request 2026-05-05 08:37:03 +00:00

The content in the footer it not centerd #732

a24nedka was assigned by a22erigr 2026-05-05 08:41:42 +00:00

a24nedka was unassigned by a22erigr 2026-05-05 08:41:50 +00:00

a22erigr requested review from a24nedka 2026-05-05 08:41:58 +00:00

a24nedka commented 2026-05-05 09:54:21 +00:00

Collaborator

Copy link

Review on #735

Whats done

Tested on Windows 11 using firefox

• Tested footer layout in app
• No errors detected, and app starts.
• Footer content is now centered correctly
  +Previous issue with left-aligned content is resolved
  +Clicking "About Us" scrolls to top as expected
  +No visual or functional issues observed

Inspected following files

• Footer.js
• Footer.module.css
• No issues found , the implementation follows our documentation.

What needs to be done

The issue is solved , and fulfills the requirement for the issue.

Conclusion

• My conclusion is that the issue is solved and can be merged.

# Review on #735 ## Whats done ### Tested on Windows 11 using firefox + Tested footer layout in app + No errors detected, and app starts. + Footer content is now centered correctly +Previous issue with left-aligned content is resolved +Clicking "About Us" scrolls to top as expected +No visual or functional issues observed ### Inspected following files + Footer.js + Footer.module.css + No issues found , the implementation follows our documentation. ## What needs to be done The issue is solved , and fulfills the requirement for the issue. ## Conclusion + My conclusion is that the issue is solved and can be merged. 

image.png

37 KiB

a24nedka added

reviewed

and removed

needs review

labels 2026-05-05 09:55:09 +00:00

a22erigr merged commit 58bfd088e2 into team_2_week_4 2026-05-05 10:01:26 +00:00

a22erigr referenced this pull request from a commit 2026-05-05 10:01:27 +00:00

Merge pull request '#732-footer-centering' (#735) from #732-footer-centered into team_2_week_4

a22erigr referenced this pull request 2026-05-06 08:04:51 +00:00

The content in the footer it not centerd #732

b25marem referenced this pull request from a commit 2026-05-15 08:56:03 +00:00

Merge pull request '#732-footer-centering' (#735) from #732-footer-centered into team_2_week_4

Sign in to join this conversation.

Reviewers

No reviewers

a24nedka

Labels

Clear labels   

_CRITICAL_

Only use this label for critical tasks that are project-scope,and must be addressed ASAP.

  

API

  

app

  

backEnd

  

Blocked-waiting-for-further-changes

  

bug

Something is not working

  

bug-only-on-server-for-mobile-not-webpage

  

Bug-Report-After-Merge

  

cleanup

Issues to cleanup the codebase (restructure folders, remove deprecated or smelly code)

  

close

Issue to close

  

design

  

duplicate

This issue or pull request already exists

  

enhancement

New feature

  

feature request

  

frontEnd

  

further-changes-needed

  

future-problem-not-fixint-this-period

  

help wanted

Need some help

  

invalid

Something is wrong

  

last-week-issue-to-fix

  

library

  

low-priority

  

needs input

Issue needs input from product owner or customer

  

needs review

issue is in need of review

  

not-implemented.

  

project documentation

  

question

More information is needed

  

research

  

reviewed

This Issue has been reviewed

  

Script

  

security

use this for security-related issues

  

SQL

  

style

  

Team 1

Team 1 is responsible for this issue

  

Team 2

Team 2 is responsible for this issue

  

team leaders

  

test-creation

  

testing

  

topLevel

Reserved for top-level, project-wide issues.

  

unassigned

  

Under-review

  

wontfix

This won't be fixed

No labels

_CRITICAL_

API

app

backEnd

Blocked-waiting-for-further-changes

bug

bug-only-on-server-for-mobile-not-webpage

Bug-Report-After-Merge

cleanup

close

design

duplicate

enhancement

feature request

frontEnd

further-changes-needed

future-problem-not-fixint-this-period

help wanted

invalid

last-week-issue-to-fix

library

low-priority

needs input

needs review

not-implemented.

project documentation

question

research

reviewed

Script

security

SQL

style

Team 1

Team 2

team leaders

test-creation

testing

topLevel

unassigned

Under-review

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

Andras/BoundlessFlowCampus2K!735

No description provided.