Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
43 commits
Select commit Hold shift + click to select a range
f58a886
Wireframe for feedback disambiguation screen flow
bwaresiak Mar 4, 2019
5002784
Tweaks and refactoring
bwaresiak Mar 4, 2019
b43edb8
AdHoc beta 1
bwaresiak Mar 4, 2019
1b968a3
UI Tweaks
bwaresiak Mar 5, 2019
51cc29a
Merge remote-tracking branch 'refs/remotes/origin/develop' into featu…
bwaresiak Mar 11, 2019
68df372
Added UserText for localization support, refactoring related to navig…
bwaresiak Mar 12, 2019
4722b03
Made Feedback form support two modes for broken website and regular f…
bwaresiak Mar 12, 2019
3d267ec
Update user texts
bwaresiak Mar 12, 2019
76d12e3
Updates to user text and UI tweaks
bwaresiak Mar 12, 2019
8530aaf
Yet another tweaks to user texts
bwaresiak Mar 13, 2019
2445698
Fine-tune message views and placeholders
bwaresiak Mar 13, 2019
0150378
Few additional tweaks
bwaresiak Mar 13, 2019
1d26159
AdHoc build for Design Review 2
bwaresiak Mar 13, 2019
fd50d78
Fix Positive feedback button actions
bwaresiak Mar 13, 2019
428ca5e
Improve UI placement when keyboard is being shown
bwaresiak Mar 13, 2019
909234e
Move Feedback to 'Help' section
bwaresiak Mar 13, 2019
41fa43b
Open review page directly in AppStore
bwaresiak Mar 13, 2019
d936a33
Apply dark theme to feedback form input fields
bwaresiak Mar 13, 2019
185e09a
Use title-case for category descriptions
bwaresiak Mar 13, 2019
531fd3b
AdHoc build for Design Review 3
bwaresiak Mar 14, 2019
4c90809
Various tweaks and cleanups
bwaresiak Mar 14, 2019
190869c
Fix file missing form target
bwaresiak Mar 18, 2019
6240064
Tweaks to scrolling behavior when keyboard is being shown
bwaresiak Mar 19, 2019
9eb0ccb
Merge branch 'develop' into feature/feedback-disambiguation
bwaresiak Mar 20, 2019
cd1a14c
Use 'Remove' image as template image to enable custom colors
bwaresiak Mar 20, 2019
cead730
Create proper enums for Subcategories
bwaresiak Mar 20, 2019
796a2e1
Update copy
bwaresiak Mar 20, 2019
00e64eb
Naming tweaks, firing Pixels
bwaresiak Mar 21, 2019
a98aa86
Rename FeedbackModel to SiteFeedbackModel
bwaresiak Mar 21, 2019
fbd21c9
Send feedback to the server
bwaresiak Mar 21, 2019
0cdb3a2
Merge branch 'develop' into feature/feedback-disambiguation
bwaresiak Mar 21, 2019
e0c3b99
Send to feedback.js only if needed, but always fire a Pixel
bwaresiak Mar 21, 2019
da13f6b
AdHoc build for Design Review 4
bwaresiak Mar 21, 2019
edba4ef
Tweaks and code refactoring
bwaresiak Mar 22, 2019
7a62c50
Tweak file names to reflect code changes, additional refactoring
bwaresiak Mar 22, 2019
b277de8
Tweak screen names on storyboard
bwaresiak Mar 22, 2019
75ceec3
Use guard instead of 'if' to reduce code indentation
bwaresiak Mar 22, 2019
ccee081
Merge branch 'develop' into feature/feedback-disambiguation
bwaresiak Mar 27, 2019
1f4fcbf
Address some of the PR feedback
bwaresiak Mar 27, 2019
f4c9c55
Define Pixel for each feedback option
bwaresiak Mar 27, 2019
82c73d8
Broken Site form should not be used to submit general feedback
bwaresiak Mar 27, 2019
83b2bfa
Improve Keyboard handling on Broken Site report view
bwaresiak Mar 28, 2019
9f6a0b4
Extract strings for future localization
bwaresiak Mar 28, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 34 additions & 2 deletions Core/Pixel.swift
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,40 @@ public enum PixelName: String {
case tabBarForwardPressed = "mt_fw"
case tabBarBookmarksPressed = "mt_bm"
case tabBarTabSwitcherPressed = "mt_tb"


case feedbackPositive = "mfbs_positive_submit"
case feedbackNegativePrefix = "mfbs_negative_"

case feedbackNegativeBrokenSites = "mfbs_negative_brokenSites_submit"
case feedbackNegativeOther = "mfbs_negative_other_submit"

case feedbackNegativeBrowserFeaturesNav = "mfbs_negative_browserFeatures_navigation"
case feedbackNegativeBrowserFeaturesTabs = "mfbs_negative_browserFeatures_tabs"
case feedbackNegativeBrowserFeaturesAds = "mfbs_negative_browserFeatures_ads"
case feedbackNegativeBrowserFeaturesVideos = "mfbs_negative_browserFeatures_videos"
case feedbackNegativeBrowserFeaturesImages = "mfbs_negative_browserFeatures_images"
case feedbackNegativeBrowserFeaturesBookmarks = "mfbs_negative_browserFeatures_bookmarks"
case feedbackNegativeBrowserFeaturesOther = "mfbs_negative_browserFeatures_other"

case feedbackNegativeBadResultsTechnical = "mfbs_negative_badResults_technical"
case feedbackNegativeBadResultsLayout = "mfbs_negative_badResults_layout"
case feedbackNegativeBadResultsSpeed = "mfbs_negative_badResults_speed"
case feedbackNegativeBadResultsLangOrRegion = "mfbs_negative_badResults_langRegion"
case feedbackNegativeBadResultsAutocomplete = "mfbs_negative_badResults_autocomplete"
case feedbackNegativeBadResultsOther = "mfbs_negative_badResults_other"

case feedbackNegativeCustomizationHome = "mfbs_negative_customization_home"
case feedbackNegativeCustomizationTabs = "mfbs_negative_customization_tabs"
case feedbackNegativeCustomizationUI = "mfbs_negative_customization_ui"
case feedbackNegativeCustomizationWhatCleared = "mfbs_negative_customization_whichDataCleared"
case feedbackNegativeCustomizationWhenCleared = "mfbs_negative_customization_whenDataCleared"
case feedbackNegativeCustomizationBookmarks = "mfbs_negative_customization_bookmarks"
case feedbackNegativeCustomizationOther = "mfbs_negative_customization_other"

case feedbackNegativePerformanceSlow = "mfbs_negative_performance_slow"
case feedbackNegativePerformanceCrash = "mfbs_negative_performance_crash"
case feedbackNegativePerformanceVideo = "mfbs_negative_performance_video"
case feedbackNegativePerformanceOther = "mfbs_negative_performance_other"
}

public class Pixel {
Expand Down Expand Up @@ -113,7 +146,6 @@ public class Pixel {
withAdditionalParameters params: [String: String?] = [:],
withHeaders headers: HTTPHeaders = APIHeaders().defaultHeaders,
onComplete: @escaping (Error?) -> Void = {_ in }) {

let formFactor = deviceType == .pad ? Constants.tablet : Constants.phone
let url = appUrls
.pixelUrl(forPixelNamed: pixel.rawValue, formFactor: formFactor)
Expand Down
80 changes: 66 additions & 14 deletions DuckDuckGo.xcodeproj/project.pbxproj

Large diffs are not rendered by default.

3 changes: 3 additions & 0 deletions DuckDuckGo/Assets.xcassets/Remove.imageset/Contents.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,8 @@
"info" : {
"version" : 1,
"author" : "xcode"
},
"properties" : {
"template-rendering-intent" : "template"
}
}
12 changes: 12 additions & 0 deletions DuckDuckGo/Assets.xcassets/ReportIcon.imageset/Contents.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
{
"images" : [
{
"idiom" : "universal",
"filename" : "iconReport.pdf"
}
],
"info" : {
"version" : 1,
"author" : "xcode"
}
}
Binary file not shown.
4 changes: 2 additions & 2 deletions DuckDuckGo/AutoClearSettingsViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ class AutoClearSettingsViewController: UITableViewController {
cell.backgroundColor = theme.tableCellBackgroundColor

// Checkmark color
cell.tintColor = theme.toggleSwitchColor
cell.tintColor = theme.buttonTintColor

if let settings = clearDataSettings,
indexPathOf(action: settings.action) == indexPath || indexPathOf(timing: settings.timing) == indexPath {
Expand Down Expand Up @@ -185,7 +185,7 @@ extension AutoClearSettingsViewController: Themable {
label.textColor = theme.tableCellTintColor
}

clearDataToggle.onTintColor = theme.toggleSwitchColor
clearDataToggle.onTintColor = theme.buttonTintColor

tableView.backgroundColor = theme.backgroundColor
tableView.separatorColor = theme.tableCellSeparatorColor
Expand Down
113 changes: 46 additions & 67 deletions DuckDuckGo/Base.lproj/Settings.storyboard

Large diffs are not rendered by default.

10 changes: 9 additions & 1 deletion DuckDuckGo/DarkTheme.swift
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,11 @@ struct DarkTheme: Theme {
var tableCellAccessoryTextColor = UIColor.greyish
var tableHeaderTextColor = UIColor.lightGreyish

var toggleSwitchColor = UIColor.cornflowerBlue
var buttonTintColor = UIColor.cornflowerBlue
var placeholderColor = UIColor.greyish

var textFieldBackgroundColor = UIColor.nearlyBlackLight
var textFieldFontColor = UIColor.white

var homeRowPrimaryTextColor = UIColor.white
var homeRowSecondaryTextColor = UIColor.lightMercury
Expand All @@ -61,5 +65,9 @@ struct DarkTheme: Theme {
var faviconBackgroundColor = UIColor.charcoalGrey
var favoriteTextColor = UIColor.greyish

var feedbackPrimaryTextColor = UIColor.white
var feedbackSecondaryTextColor = UIColor.lightGreyish
var feedbackSentimentButtonBackgroundColor = UIColor.charcoalGrey

var activityStyle: UIActivityIndicatorView.Style = .white
}
Loading