search

open-sauced / app

🍕 Insights into your entire open source ecosystem.
https://pizza.new
Apache License 2.0
413 stars 219 forks source link

bug: Responsive view of explore page is messy #4061

Closed jpmcb closed 1 week ago

jpmcb commented 2 weeks ago

We don't have to tackle this right away, but as a followup, this view of the table looks pretty messy. This is narrow enough where it collapses but not so narrow that it goes to the "card" view that you'd see on mobile.

Screenshot 2024-08-28 at 3 20 39 PM

Originally posted by @jpmcb in https://github.com/open-sauced/app/issues/4059#issuecomment-2316273232

We should consider either cleaning this up or removing this view in favor of the collapsible card, mobile view.

github-actions[bot] commented 2 weeks ago

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please comment on this issue.

For full info on how to contribute, please check out our contributors guide.

nickytonline commented 2 weeks ago

This is still using the old table view component which is why the repsonsive story is not great. Not sure if we're planning to plug TanStack Table here @zeucapua @isabensusan or are we going with the potential revamp soon?

isabensusan commented 1 week ago

@nickytonline I wouldn't put work into migrating this table tbh! We are most likely shipping @zeucapua's Explore page soon

nickytonline commented 1 week ago

I'm going to go ahead and close this then in favour of the work in #4053.