#433 Add reset layout button with confirmation modal #454

Merged

c24elipe merged 1 commit from #433-reset-layout-button into team_2_week_2

2026-04-21 08:56:09 +00:00

c24carso commented

2026-04-20 14:35:28 +00:00

Collaborator

Added resetLayout function in SaveGrid.js and a reset button with confirmation modal in Dashboard.js

Screenshot 2026-04-20 163352.png

13 KiB

Screenshot 2026-04-20 163414.png

36 KiB

c24carso added 1 commit

2026-04-20 14:35:29 +00:00

#433 Add reset layout button with confirmation modal 08a7853ad9

Added resetLayout function in SaveGrid.js and a reset button with confirmation modal in Dashboard.js

c24carso added the

needs review

label

2026-04-20 14:35:38 +00:00

c24carso was assigned by c24elipe

2026-04-21 06:58:49 +00:00

c24elipe requested review from c24danli

2026-04-21 07:22:36 +00:00

c24danli approved these changes

2026-04-21 08:37:24 +00:00

c24danli left a comment

Collaborator

Review on #433

Whats done

Tested on Windows 11 using Chrome

App starts without problem
- Tested adding and reseting using the "Reset Layout" button, looks good and works well.
- Doesn't brake any previous functionallity

Inspected following files

/Dashboard.js
- Indentation/code styling follows prettier
  - Indentation is ok no standard written in wiki but follows prettier
- Naming convention
  - Good naming on file( PascalCase) and varibles use camelCase.
- Comments
  - No comments but good naming makes the code clear
- General code quality
  - Well written code.
/SaveGrid.js
- Indentation/code styling follows prettier
  - Indentation is ok no standard written in wiki but follows prettier
- Naming convention
  - Good naming on file( PascalCase) and varibles use camelCase.
- Comments
  - Clear short comments that desctibe fuctionallity well
- General code quality
  - Well written code.

What needs to be done

See no problems with it.

Conclusion

Ready to merge

# Review on #433 ## Whats done ### Tested on Windows 11 using Chrome + App starts without problem + Tested adding and reseting using the "Reset Layout" button, looks good and works well. + Doesn't brake any previous functionallity ### Inspected following files + /Dashboard.js + Indentation/code styling follows prettier + Indentation is ok no standard written in wiki but follows prettier + Naming convention + Good naming on file( PascalCase) and varibles use camelCase. + Comments + No comments but good naming makes the code clear + General code quality + Well written code. + /SaveGrid.js + Indentation/code styling follows prettier + Indentation is ok no standard written in wiki but follows prettier + Naming convention + Good naming on file( PascalCase) and varibles use camelCase. + Comments + Clear short comments that desctibe fuctionallity well + General code quality + Well written code. ## What needs to be done + See no problems with it. ## Conclusion + Ready to merge