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

Add website translations #96

Merged
merged 19 commits into from
Jan 17, 2025
Merged

Add website translations #96

merged 19 commits into from
Jan 17, 2025

Conversation

Carlgo11
Copy link
Member

@Carlgo11 Carlgo11 commented Nov 21, 2024

Added the ability to serve translated versions of the site.
Also made the following UI changes:

The following translations have been verified to be correct:

  • Swedish
  • French
  • Spanish

@Carlgo11 Carlgo11 added the UI change Enhancements to the user interface label Nov 21, 2024
Signed-off-by: Carl <Carlgo11@users.noreply.github.com>
Copy link

cloudflare-workers-and-pages bot commented Nov 21, 2024

Deploying 2fa with  Cloudflare Pages  Cloudflare Pages

Latest commit: b103415
Status: ✅  Deploy successful!
Preview URL: https://08013bfa.2fa.pages.dev
Branch Preview URL: https://i18n.2fa.pages.dev

View logs

src/components/table.jsx Outdated Show resolved Hide resolved
lang/en.json Outdated Show resolved Hide resolved
@Carlgo11 Carlgo11 marked this pull request as ready for review November 24, 2024 14:04
Copy link
Member

@hkamran80 hkamran80 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Were the icons intentionally made larger?

@Carlgo11
Copy link
Member Author

Carlgo11 commented Dec 19, 2024

@hkamran80 No the size change of the icons was a mistake. Previously there was a padding of 9 pixels that I accidentally removed. Do you think it looks better with the padding?
Screenshot_20241219_160300
Screenshot_20241219_160319

@hkamran80
Copy link
Member

@hkamran80 No the size change of the icons was a mistake. Previously there was a padding of 9 pixels that I accidentally removed. Do you think it looks better with the padding?

Both look good. I like the larger icons. I was just wondering if it was intentional or not.

@hkamran80
Copy link
Member

Is this ready to be merged?

@Carlgo11
Copy link
Member Author

Yeah. The build works and I've verified the translations as best I can.

@hkamran80 hkamran80 merged commit 92d6904 into master Jan 17, 2025
4 checks passed
@hkamran80 hkamran80 deleted the i18n branch January 17, 2025 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI change Enhancements to the user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants