Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Added New Private Tab QuickAction. #1259

Merged
merged 4 commits into from
Jul 22, 2019

Conversation

Brandon-T
Copy link
Collaborator

@Brandon-T Brandon-T commented Jul 16, 2019

Fixes: #1258

Added New Private Tab quick action:

image

Quick actions are dynamic.

If the application has NEVER launched, the actions will only show "Share Brave".
However, after the first launch, the actions will depend on what mode you're browsing in:

Private Browsing Only Mode:
New Private Tab -> Launches only private tabs.

Any Other Mode:
New Tab -> Launches a regular tab.
New Private Tab -> Launches a new private tab.

Submitter Checklist:

  • Submitted a ticket for my issue if one did not already exist.
  • My patch or PR title has a standard commit message that looks like Fix #123: This fixes the shattered coffee cup! (or No Bug: <message> if no relevant ticket)
  • Unit Tests are updated to cover new or changed functionality
  • User-facing strings use NSLocalizableString()
  • New files have MPL-2.0 license header.

Test Plan:

Screenshots:

Reviewer Checklist:

  • PR is linked to an issue via Zenhub.
  • Issues are assigned to at least one epic.
  • Issues include necessary QA labels:
    • QA/(Yes|No)
    • release-notes/(include|exclude)
    • bug / enhancement
  • Necessary security reviews have taken place.
  • Adequate test coverage exists to prevent regressions.
  • Adequate test plan exists for QA to validate (if applicable)

Client/Application/AppDelegate.swift Show resolved Hide resolved
Client/Application/AppDelegate.swift Show resolved Hide resolved
@Brandon-T Brandon-T requested a review from iccub July 16, 2019 15:52
@Brandon-T Brandon-T force-pushed the feature/QuickAction3DTouch branch 2 times, most recently from cedb575 to 8b55dbe Compare July 16, 2019 17:39
@Brandon-T Brandon-T force-pushed the feature/QuickAction3DTouch branch from 8b55dbe to 2ba9cf4 Compare July 17, 2019 16:33
@Brandon-T Brandon-T force-pushed the feature/QuickAction3DTouch branch from 2ba9cf4 to 05fb4af Compare July 20, 2019 15:01
Copy link
Contributor

@iccub iccub left a comment

Choose a reason for hiding this comment

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

looks good, only need to remove extra lines

BraveShared/BraveStrings.swift Outdated Show resolved Hide resolved
public static let QuickActionNewTab = NSLocalizedString("ShortcutItemTitleNewTab", tableName: "BraveShared", bundle: Bundle.braveShared, value: "New Tab", comment: "Quick Action for 3D-Touch on the Application Icon")

public static let QuickActionNewPrivateTab = NSLocalizedString("ShortcutItemTitleNewPrivateTab", tableName: "BraveShared", bundle: Bundle.braveShared, value: "New Private Tab", comment: "Quick Action for 3D-Touch on the Application Icon")

Copy link
Contributor

Choose a reason for hiding this comment

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

remove extra line

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed.

@Brandon-T Brandon-T requested a review from iccub July 22, 2019 19:13
@kylehickinson kylehickinson merged commit 326b421 into development Jul 22, 2019
@kylehickinson kylehickinson deleted the feature/QuickAction3DTouch branch July 22, 2019 19:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quick Action for New Private Tab
4 participants