search

learnhubmk / app

The frontend app for LearnHub.mk
https://learnhub.mk
MIT License
6 stars 0 forks source link

Reusable table #412

Closed Mario-k4 closed 4 months ago

Mario-k4 commented 4 months ago

281

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2024 10:37pm
edichoska commented 4 months ago

@Mario-k4 Please resolve the conflicts! Also please look into this, the table rows are not showing on the table image

edichoska commented 4 months ago

cc: @aleksandarmicev please look into the issue as well

edichoska commented 4 months ago

@Mario-k4 @aleksandarmicev Can we make this the highest priority? It is still not working properly

edichoska commented 4 months ago

Consider either:

  1. Adding min-height to table body, and displaying message: No data available
  2. If no data, don't display the whole table. Right now we are showing only the thead
edichoska commented 4 months ago

@aleksandarmicev The sort is not working! Can you please take a look?

aleksandarmicev commented 4 months ago

@aleksandarmicev The sort is not working! Can you please take a look?

With the latest updates, sorting in ascending and descending order is now working. If anything else needs to be changed, we are here to do it.

edichoska commented 4 months ago

When I hover over the Table head it changes the cursor, indicating I can type. Please look into that as well. And resolve the conflicts. @Mario-k4 @aleksandarmicev