Skip to content

fix: use latest tag line "better" -> "fast, modern"#1029

Open
shuuji3 wants to merge 3 commits intonpmx-dev:mainfrom
shuuji3:fix/new-tagline
Open

fix: use latest tag line "better" -> "fast, modern"#1029
shuuji3 wants to merge 3 commits intonpmx-dev:mainfrom
shuuji3:fix/new-tagline

Conversation

@shuuji3
Copy link
Member

@shuuji3 shuuji3 commented Feb 5, 2026

This PR replace the all previous tagline with the new one.

Before
Screenshot of home page showing current tagline - A better browser for the npm registry

After
Screenshot of home page showing current tagline - A fast, modern browser for the npm registry

TODO:

  • check if og image is properly generated
  • check if any static iamge has previous tagline if any
  • provide a way to update non-en locales (maybe drops affected keys once?)

@vercel
Copy link

vercel bot commented Feb 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs.npmx.dev Error Error Feb 6, 2026 3:20am
npmx.dev Ready Ready Preview, Comment Feb 6, 2026 3:20am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Feb 6, 2026 3:20am

Request Review

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
lunaria/files/en-GB.json Localization changed, will be marked as complete.
lunaria/files/en-US.json Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

This pull request updates the product tagline and description text across the application from "A better browser for the npm registry" to "A fast, modern browser for the npm registry". Changes appear in Vue components, email templates, and localisation files. No logic, control flow, or public API/signature changes were made.

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly describes the tagline replacement with before/after screenshots demonstrating the change from 'a better browser for the npm registry' to 'a fast, modern browser for the npm registry'.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
assets/email-templates/reset-password.html (1)

154-160: ⚠️ Potential issue | 🟡 Minor

Fix typo in the footer tagline.
The word “registryt” should be “registry”.

🔧 Proposed fix
-                                      >, a fast, modern browser for the npm registryt
+                                      >, a fast, modern browser for the npm registry
🧹 Nitpick comments (2)
i18n/locales/en.json (1)

5-5: Minor text redundancy (optional consideration).

The description now contains "modern" twice: "a fast, modern browser" and "with a modern interface". This is not a blocker, but you might consider rewording the second part to avoid repetition, e.g., "with a sleek interface" or "with a clean interface".

lunaria/files/en-US.json (1)

5-5: Same text redundancy as en.json.

This has the same "modern" repetition as i18n/locales/en.json. If you decide to address the redundancy, ensure both files are updated consistently.

@whitep4nth3r
Copy link
Collaborator

Might be a good idea to ask in the Discord for translations on this PR.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
assets/email-templates/reset-password.html (1)

154-160: ⚠️ Potential issue | 🟡 Minor

Fix typo in footer tagline (“registryt”).
Line 159 has a stray “t” at the end of “registry”.

[suggested fix]

🛠️ Proposed fix
-                                      >, a fast, modern browser for the npm registryt
+                                      >, a fast, modern browser for the npm registry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants