#289-fix-interactivity-with-cards #398

Closed

a24julot wants to merge 0 commits from #289-fix-interactivity-with-cards into team_1_week_2

a24julot commented

2026-04-17 08:52:43 +00:00

Collaborator

Moves card without collision, layout it saved so the user can add more cards without disturbing their layout. No overlapping. Issue #289 .

Moves card without collision, layout it saved so the user can add more cards without disturbing their layout. No overlapping. Issue https://git.webug.se/Andras/BoundlessFlowCampus2K/issues/289 .

nocollision.png

77 KiB

a24julot added 4 commits

2026-04-17 08:52:44 +00:00

Updated Dashboard.js 0402e4bea2

Added code to fix so that cards cant be placed on top of each other. And so that when adding a new view, it spawn underneath other cards.

Fixed some logic, more readable 8078c43b02

Cleaned up code and commented 3180e23fb1

Removed console.log ff58629def

b23albst referenced this pull request

2026-04-17 10:37:06 +00:00

Certain views/cards scale with text when added, but moving them scales them to default size #374

b24johka requested review from a24hamrh

2026-04-17 11:27:12 +00:00

a24hamrh commented

2026-04-17 13:21:29 +00:00

Collaborator

Review on #398

Whats done

Reviewed #398

Tested on Windows 11 using Firefox

App starts
- Added two views from the menu and tried to drag one on top of the other. The card i wasn't dragging moved away.

Inspected following files Dashboard.js

files Dashboard.js
- Indentation/code styling
  - Indentation is ok
- Naming convention
  - Naming convention is followed
- Comments
  - Good amount of comments. Easy to follow and understand
- General code quality
  - The general code quality is good.

Conclusion

To code works as intended and solves the problem. Cards cannot be placed on top of each other.

# Review on #398 ## Whats done Reviewed [#398](https://git.webug.se/Andras/BoundlessFlowCampus2K/pulls/398) ### Tested on Windows 11 using Firefox + App starts + Added two views from the menu and tried to drag one on top of the other. The card i wasn't dragging moved away. ### Inspected following files Dashboard.js + files Dashboard.js + Indentation/code styling + Indentation is ok + Naming convention + Naming convention is followed + Comments + Good amount of comments. Easy to follow and understand + General code quality + The general code quality is good. ## Conclusion + To code works as intended and solves the problem. Cards cannot be placed on top of each other.