Skip to content

Conversation

@canerakdas
Copy link
Member

Description

This PR provides the ability to add icons to buttons by default

Affected components;

  • Codebox
  • Button
  • Pagination
  • DownloadButton

Affected pages;

  • Not Found
  • Error
  • Global Error
  • Home
  • Blog Listing

Validation

The pages/components should work and look same on local deployments with npm run serve:redesign

Related Issues

fixes #6247, #6246

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run npx turbo lint to ensure the code follows the style guide. And run npx turbo lint:fix to fix the style errors if necessary.
  • I have run npx turbo format to ensure the code follows the style guide.
  • I have run npx turbo test to check if all tests are passing.

@canerakdas canerakdas added enhancement website redesign Issue/PR part of the Node.js Website Redesign labels Jan 22, 2024
@canerakdas canerakdas requested a review from a team as a code owner January 22, 2024 17:58
@vercel
Copy link

vercel bot commented Jan 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 22, 2024 6:24pm

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2024

Lighthouse Results

URL Performance Accessibility Best Practices SEO Report
/en 🟠 87 🟢 100 🟢 92 🟢 92 🔗
/en/about 🟢 100 🟢 97 🟢 92 🟢 92 🔗
/en/about/previous-releases 🟢 98 🟢 96 🟢 92 🟢 92 🔗
/en/download 🟢 100 🟢 97 🟢 92 🟢 92 🔗
/en/blog 🟢 100 🟢 97 🟢 92 🟢 92 🔗

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2024

Unit Test Coverage Report

Lines Statements Branches Functions
Coverage: 90%
86.49% (397/459) 77.33% (116/150) 77.77% (70/90)

Unit Test Report

Tests Skipped Failures Errors Time
72 0 💤 0 ❌ 0 🔥 4.397s ⏱️

Copy link
Member

@AugustinMauroy AugustinMauroy left a comment

Choose a reason for hiding this comment

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

LGTM !

@ovflowd ovflowd added the github_actions:pull-request Trigger Pull Request Checks label Jan 23, 2024
@github-actions github-actions bot removed the github_actions:pull-request Trigger Pull Request Checks label Jan 23, 2024
@ovflowd ovflowd added this pull request to the merge queue Jan 23, 2024
Merged via the queue into nodejs:main with commit 7a89269 Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

website redesign Issue/PR part of the Node.js Website Redesign

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants