solved the responsiveness issue#1702
Conversation
Co-authored-by: Copilot <copilot@github.com>
MImran2002
left a comment
There was a problem hiding this comment.
Working perfectly, the only fix left is the responsiveness of the table. I wanted to see if it is necessary and it the development it is responsive:

But in the branch it is not working. This is the bug: in the first populated table when you first get there the responsiveness is good then you deselect the checkmarks and the table loses the columns and retain the width but when you add the checkmark again the responsiveness break. I think the columns are added again on top of the total width breaking the responsiveness aside. This could exist in the development too because in the development the checkmark itself is breaking so this responsiveness bug was hidden. Aside from this the code working:

potential solution could be to wrapped the table with a fixed responsive div and allow the table to expand and contract by 100%, this could be a solution there can be more solutions too.
|
For some reason I am not able to reproduce this bug on my branch. Did you pull the changes? |
…table remove the columsn but when the checkmark are ticekd back the action broke the table's responsivness. This isn't a bug created by Sara but it was responsive bug that already exist but was not discover because the checkmark was broken. The fixed now is to have a wrapper around the table to maintain its responsivness while allowing the table to be responsive wihtin the wrapper.
MImran2002
left a comment
There was a problem hiding this comment.
the table's responsiveness breaks when the checkmarks are toggle the table remove the columns but when the checkmark are ticked back the action broke the table's responsiveness. This isn't a bug created by Sara but it was responsive bug that already exist but was not discover because the checkmark was broken. The fixed now is to have a wrapper around the table to maintain its responsiveness while allowing the table to be responsive within the wrapper.




Issue Description
Fixes #1689
Changes
Testing
The checkboxes are now working!
