-
Notifications
You must be signed in to change notification settings - Fork 3
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
Geist UI redesign #11
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I test this PR but Show outdated
in the filters modal is not working. Spanish says is up to date
.
If you scroll through the table, you'll see no yellow square icon in the |
@lukbukkit But I'm seeing that |
Ahh thanks, now I've got it ^^ (I was a little blind before). There was a flaw in the pagination logic. The filtered elements were calculated correctly, but the first page of elements was skipped. |
Yeah that would be useful. I've experimented a bit, but wasn't able to find a solution until now. I'll come back to this problem in the next few days.
Good idea, now the category is shown in the sticky header instead of the generic page column name. And @navarroaxel thanks for detailed review, this is very helpful :D |
I've implemented a feature to highlight columns. The colors of those columns are more saturated. Is this difference in saturation enough? How should we present the checkboxes to select languages to highlight? I've got three different layouts in mind.
|
Really nice, the saturation is perfect. For selecting the language I prefer the second layout. |
I agree with you, the second layout is also my favorite. I've added a little bit more space between the checkboxes. |
Oh you picked the 2nd layout? Too bad, I love the 3rd 😢 |
@lukbukkit 🤔 Vercel supports that too. I saw that. Maybe the steps are here https://vercel.com/docs/git/vercel-for-github |
That's correct, Vercel also supports automatic builds from Git repositories. But as this repository belongs to a GitHub organization and not a personal GitHub account, I would have to pay in order to use this feature: https://vercel.com/pricing. I get the same feature set for free at Netlify. There's a limit of 300 build minutes per month but until now we used only 11. |
Just kidding 😆🤣 |
Thanks for all your help guys. 👍 I hope everyone is happy with the final version. I'm going to merge the changes in a few days. |
Hi, I agree as well, I prefer also the second layout. |
But when I went the the above linked page, I couldn't see the changes. Is this because I need to build the project locally at home ? |
The version on Vercel is outdated, please visit the page on Netlify: https://tldr-i18n-progress-geist-ui.netlify.app/ |
Thanks. |
I've found some minor inconsistencies in variable naming and in the color scheme while doing the final code review. Those were fixed by the last three commits. |
The latest update is now published, if any errors arise please let me know by opening a new issue or dropping a comment here. ^^ |
@lukbukkit why I can't see the new version in https://lukwebsforge.github.io/tldri18n/ yet? |
The website is updated along with the |
Hello,
I happy to present the redesign of the React frontend using the Geist UI framework. This enables us to use various prebuilt components and create a more clean and concise UI and UX.
You can test the new design on
https://tldr-i18n-progress-lukbukkit.vercel.apphttps://tldr-i18n-progress-geist-ui.netlify.app. Beware that the data may be out of date. I'm happy about any feedback especially about details.This pull request closes #9 by implementing the filter action.
The highlighting of columns (see #8) is still on the to do list for this pull request. Yet I'm not certain if I can implement this feature with good performance, because we have to watch the mouse hover event for all cells of the table.