Fix MapView scaling on smaller card sizes #969

Merged

a22erigr merged 2 commits from 955-fix-mapview-responsiveness into team_2_week_6

2026-05-19 14:20:59 +00:00

b25marem commented

2026-05-15 10:46:02 +00:00

Collaborator

Fixed MapView responsiveness issue.

Changes:

Improved image scaling inside smaller card sizes
Prevented image clipping when resizing
Removed unnecessary internal scrolling
Updated layout styling for better responsiveness

Fixed MapView responsiveness issue. Changes: * Improved image scaling inside smaller card sizes * Prevented image clipping when resizing * Removed unnecessary internal scrolling * Updated layout styling for better responsiveness

b25marem added 1 commit

2026-05-15 10:46:03 +00:00

Fix MapView scaling on smaller card sizes 4d7d7d23e8

b25marem added the

needs review

label

2026-05-15 10:46:21 +00:00

a22erigr

2026-05-18 08:27:12 +00:00

added the
Under-review
label
requested review from c24danli

c24danli commented

2026-05-18 10:04:37 +00:00

Collaborator

Review on #969

Tested on Windows using Chrome

App starts
- it scales better doesn´t cut of any more, depending on size you get scrollbars

Inspected following files

frontend/src/features/map/MapView.js
- Indentation/code styling
  - ok
- CSS standard
  - In line style exist + its redundant as overflow and height already exist in the css for map-container.
```
  className={styles['map-container']}
  style={{
  	height: '100%',
  	overflow: 'hidden',
  }}
```
- Naming convention
  - looks good no changes
- Comments
  - no need as the code is readable
- General code quality
  - ok except inline css
frontend/src/features/map/MapView.module.css
- Indentation/code styling
  - ok
- CSS standard
  - ok
- Naming convention
  - ok
- Comments
  - no need as the code is readable
- General code quality
  - ok

What needs to be done

Remove inline css

Conclusion

Remove inline css before merge

# Review on #969 ### Tested on Windows using Chrome + App starts + it scales better doesn´t cut of any more, depending on size you get scrollbars ### Inspected following files + frontend/src/features/map/MapView.js + Indentation/code styling + ok + CSS standard + In line style exist + its redundant as overflow and height already exist in the css for map-container. className={styles['map-container']} style={{ height: '100%', overflow: 'hidden', }} + Naming convention + looks good no changes + Comments + no need as the code is readable + General code quality + ok except inline css + frontend/src/features/map/MapView.module.css + Indentation/code styling + ok + CSS standard + ok + Naming convention + ok + Comments + no need as the code is readable + General code quality + ok ## What needs to be done + Remove inline css ## Conclusion + Remove inline css before merge

c24danli requested changes

2026-05-18 10:05:13 +00:00

c24danli left a comment

Collaborator

See review above

b25marem added 1 commit

2026-05-18 13:29:30 +00:00

Remove redundant inline styles from MapView c32a7f46eb

b25marem added

and removed

needs review

Under-review