1154-fix-add-card-modal #1165

Merged

a24julot merged 1 commit from 1154-fix-add-card-modal into both_teams_week_8

2026-05-27 11:31:14 +00:00

b24erika commented

2026-05-26 10:41:24 +00:00

Collaborator

Added flex so every selector in "AddCardModal" has the same size.

b24erika added the

frontEnd

needs review

labels

2026-05-26 10:41:24 +00:00

b24erika added 1 commit

2026-05-26 10:41:24 +00:00

added flex 1 to card selector 9f01841e49

To make each selection have the same size

b24erika self-assigned this

2026-05-26 10:41:28 +00:00

a24julot

2026-05-27 08:45:14 +00:00

removed the
needs review
label
requested review from a24hirsa

a24hirsa commented

2026-05-27 09:07:39 +00:00

Collaborator

Review on issue #1154, PR #1165

Whats done

Checked code and running branch to see if add card model selectors are of the same size

Tested on Linux using Firefox

App starts

Inspected following files

AddCardModal.module.css
- Indentation/code styling
- CSS standard
- General code quality

What needs to be done

nothing

Conclusion

Code is a simple addition and code strucutre was fixed which is good.
The add card model now has the same size for all selectors and is ready to be merged

# Review on issue #1154, PR #1165 ## Whats done Checked code and running branch to see if add card model selectors are of the same size ### Tested on Linux using Firefox + App starts ### Inspected following files + AddCardModal.module.css + Indentation/code styling + CSS standard + General code quality ## What needs to be done + nothing ## Conclusion + Code is a simple addition and code strucutre was fixed which is good. + The add card model now has the same size for all selectors and is ready to be merged

Screenshot_20260527_110314.png

110 KiB

🎉 1

b24erika commented

2026-05-27 09:12:29 +00:00

Author

Collaborator

@a24hirsa wrote in #1165 (comment):

Code is a simple addition and code strucutre was fixed which is good.

The add card model now has the same size for all selectors and is ready to be merged

Thanks for the review! If you consider that the solution is sufficient and your review is done, please mark it as "reviewed", thanks!

@a24hirsa wrote in https://git.webug.se/Andras/BoundlessFlowCampus2K/pulls/1165#issuecomment-14548: > Code is a simple addition and code strucutre was fixed which is good. > > The add card model now has the same size for all selectors and is ready to be merged Thanks for the review! If you consider that the solution is sufficient and your review is done, please mark it as "reviewed", thanks!