chatGPTBox icon indicating copy to clipboard operation
chatGPTBox copied to clipboard

tweak english, move tabs in Popup.jsx

Open iamsirsammy opened this issue 2 years ago • 4 comments
trafficstars

This has a few small tweaks to the english localization, mostly renaming Feature Pages to history, and same for Conversation Page/Window. The localization is pretty good now, but I have two questions:

  • Should I be editing the english language from src/_locales/en/main.json?
  • What does the option "Always Create New Conversation Window" do? Does it save every conversation or something else

Thanks

iamsirsammy avatar Jun 20 '23 02:06 iamsirsammy

/review

PeterDaveHello avatar Jun 02 '25 18:06 PeterDaveHello

Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Tab Order Change

The PR moves the "Feature Pages" tab to the end of the tab list, but also renames it to "History" in the localization file. The tab name should be updated to use the new localization.

  <Tab className="popup-tab">{t('Feature Pages')}</Tab>
</TabList>

qodo-merge-pro[bot] avatar Jun 02 '25 18:06 qodo-merge-pro[bot]

/improve

PeterDaveHello avatar Jun 02 '25 18:06 PeterDaveHello

Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

PR Code Suggestions ✨

No code suggestions found for the PR.

qodo-merge-pro[bot] avatar Jun 02 '25 18:06 qodo-merge-pro[bot]