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

docs(pt-br): Update pt-br/index.mdx #2139

Merged
merged 2 commits into from
Mar 25, 2024
Merged

docs(pt-br): Update pt-br/index.mdx #2139

merged 2 commits into from
Mar 25, 2024

Conversation

Exerea
Copy link
Contributor

@Exerea Exerea commented Mar 20, 2024

Summary

Update pt-br homepage.

Related: #1958

Test Plan

Run pnpm start locally and check/compare the translated homepage on the dev server with the default one.

@github-actions github-actions bot added the A-Website Area: website label Mar 20, 2024
Copy link

netlify bot commented Mar 20, 2024

Deploy Preview for biomejs ready!

Name Link
🔨 Latest commit 4df04ac
🔍 Latest deploy log https://app.netlify.com/sites/biomejs/deploys/65fd89d75c8b5800085ef358
😎 Deploy Preview https://deploy-preview-2139--biomejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (🟢 up 4 from production)
Accessibility: 97 (no change from production)
Best Practices: 100 (no change from production)
SEO: 93 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@unvalley unvalley requested a review from victor-teles March 20, 2024 19:19
@@ -4,11 +4,15 @@ head:
- tag: title
content: Biome, conjunto de ferramentas da web
template: splash
description: Formate, verifique erros e muito mais em uma fração de segundo.
description: Formato, fiapos e muito mais em uma fração de segundo.
Copy link
Contributor

@victor-teles victor-teles Mar 20, 2024

Choose a reason for hiding this comment

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

@Exerea This change doesn't look good to me (as a native speaker)

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, you need to keep the same sentence on the site hero section

@@ -190,7 +188,7 @@ import LinterExample from "@/components/linter/example.md";
<h2 class="head">Comunidade</h2>
<div class="community-connect">
<div class="community-description">
Desenvolvido pela nossa incrível comunidade
Alimentado por nossos colaboradores de código aberto
Copy link
Contributor

Choose a reason for hiding this comment

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

@Exerea I think in this case "Mantido" or "Desenvolvido" is more appropriate here

hero:
title: Um conjunto de ferramentas para seu projeto web
title: Cadeia de ferramentas integrada para o desenvolvimento de projetos da Web.
Copy link
Contributor

Choose a reason for hiding this comment

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

@Exerea You need to keep the same sentence on the site hero section

@Exerea
Copy link
Contributor Author

Exerea commented Mar 22, 2024

@victor-teles Thank you for your feedback! I have updated it.

@ematipico ematipico requested a review from victor-teles March 22, 2024 13:43
@ematipico ematipico merged commit 8af7c78 into biomejs:main Mar 25, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Website Area: website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants