-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Front-end improvements #6
Comments
@MutlaqAldhbuiub thanks for the comments, I really like your new design. You can go ahead with a PR. Do you have any suggestions for speeding up the google sheets request? |
Thanks, @zaidalyafeai for the kind answer. I think if we change the source of the data or even if we could do something like |
I prefer if we don't depend on any external databases or websites. I don't think 3 seconds is that bad? |
Thanks @MutlaqAldhbuiub for your PR and enhancements. They really look elegant. Hopefully, I can find a time soon to review. |
The results of various studies show that nearly 50% of Internet users expect web page load times of less than 2 seconds. so 3 seconds is a long time to render a page. Thanks all @zaidalyafeai @MagedSaeed |
Thanks @MutlaqAldhbuiub, btw the UI is messed up in mobile view. |
Hello @zaidalyafeai
|
Regarding #10 We were considering doing something similar to Huggingface's Datasets page. We might need to use a templating engine like |
For me, I like the current design more. If there is anything to change, maybe remove the bullets from the tasks. |
Hello @zaidalyafeai If i remove the bullet points will be hard to notice how many tasks for the row So i suggest to keep it if you don't want the new task style But how about the striped table do you like it with old task style? or should i also remove it and just update the card.js? |
In mobile view, the toggle button doesn't work. |
Problem
The problem is the website performance is so slow, and it's so hard to browse
and I think there's code that is not readable at all, for ex. in
card.js
there's a lot of code that no need for (I've deleted a little bit of it during reading and understanding the code)!Solution
I've designed the front end and made it easier to browse check it out
Notes
MaxRowLength
every single time someone enters a new data point in google sheets.Thanks to @sudomaze for helping me with the review of the design.
The text was updated successfully, but these errors were encountered: