Andras/BoundlessFlowCampus2K

Fork 0

#217 & #218 Card layout config #240

Closed

b24idalu wants to merge 5 commits from Issue#217-Card-layout-config into team_1_week_1

b24idalu commented

2026-04-10 16:59:36 +00:00

Collaborator

Problems: 1. Save grid data when reloading browser, 2. Create a format on how the data could look when saved to DB (JSON format), 3. Create a way to easily work with grid data later on.

Issues: #217 Card layout config, #218 Load grid layout using config

Solution: 1. Added a way to store grid data through local storage, which is like a file system for the browser. Right now there are two functions that retrieves and stores data to it. Code bits inspired by: local storage example.
2. There is already a default gridsystem, see issue 189. The format will follow that, but have the key "layout" (can be changed to be clearer).
3. Following solution 1, the data stored in the local storage is in json string. Through the function, it is returned as a json object which is the goal.

Possible ToDos:

In the future it could be good to add a user key parameter to some function/s.
Could be a good idea to move the grid stuff to a separate file and keep the app file clean.

Problems: 1. Save grid data when reloading browser, 2. Create a format on how the data could look when saved to DB (JSON format), 3. Create a way to easily work with grid data later on. Issues: [#217 Card layout config](https://git.webug.se/Andras/BoundlessFlowCampus2K/issues/217), [#218 Load grid layout using config](https://git.webug.se/Andras/BoundlessFlowCampus2K/issues/218) Solution: 1. Added a way to store grid data through local storage, which is like a file system for the browser. Right now there are two functions that retrieves and stores data to it. Code bits inspired by: [local storage example](https://github.com/react-grid-layout/react-grid-layout/blob/master/test/examples/07-localstorage.jsx). 2. There is already a default gridsystem, see [issue 189](https://git.webug.se/Andras/BoundlessFlowCampus2K/pulls/210). The format will follow that, but have the key "layout" (can be changed to be clearer). 3. Following solution 1, the data stored in the local storage is in json string. Through the function, it is returned as a json object which is the goal. Possible ToDos: - In the future it could be good to add a user key parameter to some function/s. - Could be a good idea to move the grid stuff to a separate file and keep the app file clean.

b24idalu added 4 commits

2026-04-10 17:00:18 +00:00

Added grid location saving 05f1917c92

Added three functions to save card data to the local storage so that page can be refreshed and data will be saved. Also added a function to create json files but it is commented since it will not work through browser.

Added function that uses json for grid positions 6824616bee

Removed previous json file function.

added function to get json from local storage 03769f3329

added the last function that will take a layout from the local storage and return a json string.

Simplified functions 9e454690a4

Since functions were similar, combined them.

b24idalu added the

design

frontEnd

needs review

labels

2026-04-10 17:02:55 +00:00

b24idalu was assigned by c24elipe

2026-04-13 08:35:12 +00:00

b24idalu added 1 commit

2026-04-13 09:09:37 +00:00

Made code cleaner and commented more f39e47866b

Made code cleaner and commented more.

c24elipe requested review from b25marem

2026-04-14 06:59:00 +00:00

a24vinla referenced this pull request

2026-04-14 08:53:27 +00:00

Break out grid layout config saving from app/home/dashboard to its own file #279

a24vinla referenced this pull request

2026-04-14 09:00:18 +00:00

Break out grid layout config saving from app/home/dashboard to its own file #279

b24hugst was assigned by b24johka

2026-04-14 11:03:25 +00:00

b24hugst was unassigned by b24johka

2026-04-14 11:03:53 +00:00

a24vinla requested review from a22erigr

2026-04-14 13:16:52 +00:00

a22erigr commented

2026-04-14 13:39:42 +00:00

Collaborator

Unable fix merge conflicts. After consulting @a24vinla the decision was made to abort merge attempt. The reason given it that @b24idalu currently working with the code that would resolve some of the conflicts.