Skip to content

Commit

Permalink
Added some analytics for the inline picker’s toolbar.
Browse files Browse the repository at this point in the history
  • Loading branch information
bummytime committed Aug 29, 2017
1 parent 6ee5af5 commit 2a5dad5
Show file tree
Hide file tree
Showing 3 changed files with 24 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -371,6 +371,22 @@ + (TracksEventPair *)eventPairForStat:(WPAnalyticsStat)stat
eventName = @"editor_button_tapped";
eventProperties = @{ TracksEventPropertyButtonKey : @"image" };
break;
case WPAnalyticsStatEditorMediaPickerTappedDismiss:
eventName = @"media_picker_button_tapped";
eventProperties = @{ TracksEventPropertyButtonKey : @"cancel" };
break;
case WPAnalyticsStatEditorMediaPickerTappedDevicePhotos:
eventName = @"media_picker_button_tapped";
eventProperties = @{ TracksEventPropertyButtonKey : @"device_photos" };
break;
case WPAnalyticsStatEditorMediaPickerTappedCamera:
eventName = @"media_picker_button_tapped";
eventProperties = @{ TracksEventPropertyButtonKey : @"camera" };
break;
case WPAnalyticsStatEditorMediaPickerTappedMediaLibrary:
eventName = @"media_picker_button_tapped";
eventProperties = @{ TracksEventPropertyButtonKey : @"media_library" };
break;
case WPAnalyticsStatEditorTappedItalic:
eventName = @"editor_button_tapped";
eventProperties = @{ TracksEventPropertyButtonKey : @"italic" };
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1438,10 +1438,13 @@ extension AztecPostViewController {
else if let mediaIdentifier = FormatBarMediaIdentifier(rawValue: identifier) {
switch mediaIdentifier {
case .deviceLibrary:
trackFormatBarAnalytics(stat: .editorMediaPickerTappedDevicePhotos)
presentMediaPickerFullScreen(animated: true, dataSourceType: .device)
case .camera:
trackFormatBarAnalytics(stat: .editorMediaPickerTappedCamera)
mediaPickerInputViewController?.showCapture()
case .mediaLibrary:
trackFormatBarAnalytics(stat: .editorMediaPickerTappedMediaLibrary)
presentMediaPickerFullScreen(animated: true, dataSourceType: .mediaLibrary)
}
}
Expand Down Expand Up @@ -1765,6 +1768,7 @@ extension AztecPostViewController {
changeRichTextInputView(to: nil)
updateToolbar(formatBar, forMode: .text)
restoreInputAssistantItems()
trackFormatBarAnalytics(stat: .editorMediaPickerTappedDismiss)
} else {
presentMediaPicker(fromButton: button, animated: true)
}
Expand Down
4 changes: 4 additions & 0 deletions WordPressShared/WordPressShared/Core/Analytics/WPAnalytics.h
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,10 @@ typedef NS_ENUM(NSUInteger, WPAnalyticsStat) {
WPAnalyticsStatEditorTappedHorizontalRule,
WPAnalyticsStatEditorTappedHTML,
WPAnalyticsStatEditorTappedImage,
WPAnalyticsStatEditorMediaPickerTappedDismiss,
WPAnalyticsStatEditorMediaPickerTappedDevicePhotos,
WPAnalyticsStatEditorMediaPickerTappedCamera,
WPAnalyticsStatEditorMediaPickerTappedMediaLibrary,
WPAnalyticsStatEditorTappedItalic,
WPAnalyticsStatEditorTappedLink,
WPAnalyticsStatEditorTappedMore,
Expand Down

0 comments on commit 2a5dad5

Please sign in to comment.