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

WebTorrent: Rename "Save File" to "Save .torrent File" #3036

Merged
merged 1 commit into from
Jul 27, 2019

Conversation

feross
Copy link
Contributor

@feross feross commented Jul 27, 2019

Really trivial PR.

Fixes brave/brave-browser#5354

Submitter Checklist:

Test Plan:

  1. Visit https://webtorrent.io/free-torrents
  2. Click Big Buck Bunny (magnet link)
  3. Start downloading torrent
  4. Ensure the top right button says "Save .torrent File"

Reviewer Checklist:

  • New files have MPL-2.0 license header.
  • Request a security/privacy review as needed.
  • Adequate test coverage exists to prevent regressions
  • Verify test plan is specified in PR before merging to source

After-merge Checklist:

  • The associated issue milestone is set to the smallest version that the
    changes has landed on.
  • All relevant documentation has been updated.

@feross feross added the feature/webtorrent Label for webtorrent related issues label Jul 27, 2019
@feross feross added this to the 0.70.x - Nightly milestone Jul 27, 2019
@feross feross requested a review from yrliou July 27, 2019 00:07
@feross feross self-assigned this Jul 27, 2019
@yrliou yrliou merged commit 2bcc6c8 into master Jul 27, 2019
@yrliou yrliou deleted the webtorrent-save branch July 27, 2019 01:10
@@ -54,7 +54,7 @@ export default class TorrentViewerHeader extends React.PureComponent<
const mainButtonText = torrent ? 'Stop Torrent' : 'Start Torrent'
const copyButtonText = this.props.torrentId.startsWith('magnet:')
? 'Copy Magnet Link'
: 'Save File'
: 'Save .torrent File'
Copy link
Contributor

Choose a reason for hiding this comment

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

probably we would like to add translations to torrent page

cc @rebron

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/webtorrent Label for webtorrent related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WebTorrent: Rename "Save File" button to make functionality clearer
4 participants