added scroll to the Name column of the ClassTable

[ShivaniNR]

Currently, a scroll bar has been added to the cells of the 'Name' column, as reported for scroll or wrap by @dtemkin1 in #146. Please have a look. I am ready to make any adjustments if needed!

Referring to: #146 issue.

[dtemkin1]

Thank you so much for the PR! Would it be possible to add a minimum width to the class name column? On really small screens, it seems like it's just getting shrunk to the point of not being able to see anything. Also, what are thoughts on adding a horizontal scroll bar to the class links section towards the bottom?

[ShivaniNR]

Hi @dtemkin1,
Thanks for the feedback!

To address the clipping issue on smaller screens, I added the overflow property directly to the cells in the 'Name' column, this helps ensure the class names remain visible across screen sizes.

As for the idea of adding a horizontal scroll bar to the class links section at the bottom, that’s a great suggestion! I’ll need to look into how feasible it is with the current AgGridReact setup, but I’ll definitely explore it.

Also, regarding setting a minimum width for the class name column, do you have a preferred value in mind that you think would work well across devices?

[ShivaniNR]

Hi @dtemkin1,
Just following up to see if you had a chance to review my previous comment.

Regarding the suggestion to add a horizontal scroll bar to the class links section at the bottom, I’ve been exploring it, and I believe that adding a scrollbar to just one column may not provide the best visual experience. Instead, we might need to apply horizontal scrolling to the entire AgGrid to maintain consistency. However, since the AgGridReact component has quite restrictive styling, I’d love to hear your thoughts on this approach before proceeding further.

Thanks again for your feedback!

[dtemkin1]

Ah thanks for the bump! I can take a look into how styling AgGridReact works and what we can do, and hopefully there's something that works across the board. We might have to change column properties, do you have any thoughts @psvenk?

[jophy-ye]

Would it make more sense to just wrap the links to a new line, esp for mobile devices? Everything else is wrapped.

[dtemkin1]

wait yeah @jophy-ye I think wrapping to a new line would be great :o