#28 Added-MapView #213

Merged

c24elipe merged 2 commits from #28-Support-for-displaying-maps into team_2_week_1

2026-04-10 11:57:54 +00:00

a24danwe commented

2026-04-09 14:52:53 +00:00

Collaborator

Add MapView component with floor images

Create MapView.js
Add floor1 and floor2 SVGs to assets

Add MapView component with floor images - Create MapView.js - Add floor1 and floor2 SVGs to assets

a24danwe added the

needs review

label

2026-04-09 14:52:53 +00:00

a24danwe added 1 commit

2026-04-09 14:52:53 +00:00

Added MapView 1b1c268bb0

Add MapView component with floor images

- Create MapView.js
- Add floor1 and floor2 SVGs to assets

a24danwe added 1 commit

2026-04-09 15:00:54 +00:00

Fix naming convention: setNewmap -> setNewMap 013a251e23

a24danwe changed title from ~~Added MapView~~ to #28 Added MapView

2026-04-09 18:01:22 +00:00

a24danwe changed title from ~~#28 Added MapView~~ to #28 Added-MapView

2026-04-09 18:01:40 +00:00

a24danwe was assigned by c24elipe

2026-04-10 07:19:19 +00:00

b25marem was assigned by c24elipe

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

b25marem commented

2026-04-10 11:02:59 +00:00

Collaborator

How I tested it:
Switched to the issue branch and pulled the latest changes
Started the frontend locally using npm start
Verified that the MapView is rendered in the application
Tested that the buttons “Våning 1” and “Våning 2” are visible and working
Confirmed that switching between floors updates the displayed map correctly

What works:
The MapView component is correctly placed under frontend/src/features/map
The component is properly imported and used in App.js
The SVG floor images are loaded and displayed correctly
The functionality for switching between floors works as expected

Code review / standards:
Component naming follows PascalCase (MapView)
Folder naming follows camelCase (map)
The implementation is simple and readable
The shared Card component is used correctly

Conclusion:
The feature works correctly and is implemented properly.

What I reviewed: I have reviewed all the changed files related to this issue, including MapView.js, App.js, and the added SVG assets Checked that the implementation follows the project structure and React coding standards How I tested it: Switched to the issue branch and pulled the latest changes Started the frontend locally using npm start Verified that the MapView is rendered in the application Tested that the buttons “Våning 1” and “Våning 2” are visible and working Confirmed that switching between floors updates the displayed map correctly What works: The MapView component is correctly placed under frontend/src/features/map The component is properly imported and used in App.js The SVG floor images are loaded and displayed correctly The functionality for switching between floors works as expected Code review / standards: Component naming follows PascalCase (MapView) Folder naming follows camelCase (map) The implementation is simple and readable The shared Card component is used correctly Conclusion: The feature works correctly and is implemented properly.

b25marem added

reviewed

and removed

needs review

labels

2026-04-10 11:03:19 +00:00