Skip to content

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
@cloudflare-workers-and-pages
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

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

3 participants