Skip to content
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

Merged
merged 49 commits into from
May 29, 2021
Merged

Geist UI redesign #11

merged 49 commits into from
May 29, 2021

Conversation

lukbukkit
Copy link
Member

@lukbukkit lukbukkit commented May 8, 2021

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.app https://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.

@lukbukkit lukbukkit added the enhancement New feature or request label May 8, 2021
@lukbukkit lukbukkit self-assigned this May 8, 2021
Copy link

@navarroaxel navarroaxel left a 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.

resources/src/AppLoader.tsx Outdated Show resolved Hide resolved
@lukbukkit
Copy link
Member Author

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 es column, so it works as intended.
If we want to catch more outdated pages, we have to improve the data generation logic (see #10). The front end just displays the data it receives through the data.json file from the Go application.

grafik

@navarroaxel
Copy link

@lukbukkit But I'm seeing that nano is outdated for the es translation 🤔

@lukbukkit
Copy link
Member Author

@lukbukkit But I'm seeing that nano is outdated for the es translation 🤔

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.

@marchersimon
Copy link

Looks good. Something which annoyed me a bit was that you cannot exit pop-up dialogs with esc. Also, it would be nice to see in which section (platform) you are. This would be especially useful if you search for a page and don't know in which section you are. Maybe replacing page at the top left by the platform would work.
image

resources/src/Data.tsx Outdated Show resolved Hide resolved
resources/src/IconActionFilter.tsx Outdated Show resolved Hide resolved
resources/src/Table.tsx Outdated Show resolved Hide resolved
resources/src/index.tsx Outdated Show resolved Hide resolved
resources/src/App.css Show resolved Hide resolved
@lukbukkit
Copy link
Member Author

Looks good. Something which annoyed me a bit was that you cannot exit pop-up dialogs with esc

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.

Also, it would be nice to see in which section (platform) you are. This would be especially useful if you search for a page and don't know in which section you are. Maybe replacing page at the top left by the platform would work.

Good idea, now the category is shown in the sticky header instead of the generic page column name.
I've updated the live example with the latest build: https://tldr-i18n-progress-lukbukkit.vercel.app/

And @navarroaxel thanks for detailed review, this is very helpful :D

@marchersimon
Copy link

marchersimon commented May 10, 2021

Another thing I noticed: The link when loading the page isn't clickable. (I assume it should be a link because of the symbol)
image

@lukbukkit
Copy link
Member Author

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.

  1. One option per line (currently live)

  1. Multiple options per line (ordered)

  1. Unordered

@marchersimon
Copy link

Really nice, the saturation is perfect. For selecting the language I prefer the second layout.

@lukbukkit
Copy link
Member Author

lukbukkit commented May 22, 2021

I agree with you, the second layout is also my favorite. I've added a little bit more space between the checkboxes.

@lukbukkit lukbukkit marked this pull request as ready for review May 22, 2021 17:43
@bl-ue
Copy link
Contributor

bl-ue commented May 22, 2021

Oh you picked the 2nd layout? Too bad, I love the 3rd 😢

Dockerfile Outdated Show resolved Hide resolved
@navarroaxel
Copy link

navarroaxel commented May 23, 2021

I've moved the preview page to Netlify, because they support automatic builds for GitHub organizations. As soon as a commit is up, you can view it on tldr-i18n-progress-geist-ui.netlify.app.

@lukbukkit 🤔 Vercel supports that too. I saw that. Maybe the steps are here https://vercel.com/docs/git/vercel-for-github

@lukbukkit
Copy link
Member Author

🤔 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.

@bl-ue
Copy link
Contributor

bl-ue commented May 23, 2021

Oh you picked the 2nd layout? Too bad, I love the 3rd 😢

Just kidding 😆🤣

@lukbukkit
Copy link
Member Author

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.

@patricedenis
Copy link

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.

Hi, I agree as well, I prefer also the second layout.

@patricedenis
Copy link

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 ?

@lukbukkit
Copy link
Member Author

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/

@patricedenis
Copy link

Thanks.
So when I went looking into this Netlify page, it looked really great !!
Again, good job !!
It feels really nicer than the original page.

@lukbukkit lukbukkit linked an issue May 26, 2021 that may be closed by this pull request
@lukbukkit
Copy link
Member Author

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.

@lukbukkit lukbukkit merged commit db68a05 into main May 29, 2021
@lukbukkit
Copy link
Member Author

The latest update is now published, if any errors arise please let me know by opening a new issue or dropping a comment here. ^^

@navarroaxel
Copy link

@lukbukkit why I can't see the new version in https://lukwebsforge.github.io/tldri18n/ yet?

@lukbukkit
Copy link
Member Author

The website is updated along with the data.json file. This update happens every day at midnight (UTC).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a filter for outdated translations Highlight columns when hovering over the table
6 participants