Created a new custom button #612

Closed
c22abdal wants to merge 0 commits from #605-new-custom-button into team_2_week_3
Collaborator

Created a custom button to be used universally within cards.
the button has varaiants that can be inserted when defining a buttons such as

                     <Button variant='primary' onClick={() => setNewMap(floor1)}>
						Floor 1
					</Button>
					<Button variant='secondary' onClick={() => setNewMap(floor2)}>
						{' '}
						Floor 2
					</Button>
					<Button variant='danger'>Danger</Button>

The variants will change how the button behaves and looks.
GIF 2026-04-28 11-31-38

Created a custom button to be used universally within cards. the button has varaiants that can be inserted when defining a buttons such as ``` <Button variant='primary' onClick={() => setNewMap(floor1)}> Floor 1 </Button> <Button variant='secondary' onClick={() => setNewMap(floor2)}> {' '} Floor 2 </Button> <Button variant='danger'>Danger</Button> ``` The variants will change how the button behaves and looks. ![GIF 2026-04-28 11-31-38](/attachments/4984490a-1028-45d6-8a36-8622f723e4f1)
Collaborator

Review on PR ( Created a new custom button )

What was done

Added reusable custom button

Tested on Mac using chrome/firefox

  • Application starts without errors
  • Button renders correctly in the UI
  • onClick functionality works(switching floors).

Inspected the following files

Button.js

  • Indentation = ok
  • Naming convention = ok
  • Comments = Not required due to simplicity and clarity.
  • Code quality = clean a readable

What needs to be done

Nothing

Conclusion

Ready for merge

**Review on** PR ( Created a new custom button ) **What was done** Added reusable custom button **Tested on Mac using chrome/firefox** - Application starts without errors - Button renders correctly in the UI - onClick functionality works(switching floors). **Inspected the following files** Button.js - Indentation = ok - Naming convention = ok - Comments = Not required due to simplicity and clarity. - Code quality = clean a readable **What needs to be done** Nothing **Conclusion** Ready for merge
Collaborator

Good job everyone, will merge manually!

Good job everyone, will merge manually!
a24vinla closed this pull request 2026-04-29 07:18:06 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
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!612
No description provided.