Opened on 02/13/2017 at 12:09:54 PM

Last modified on 02/15/2017 at 02:27:59 PM

#4883 new change

event-based error reporting description misleading

Reported by: scheer Assignee:
Priority: P3 Milestone:
Module: Adblock-Browser-for-iOS Keywords:
Cc: mario, lisabielik Blocked By:
Blocking: Platform: Adblock Browser for iOS
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by mario)

With the implementation of #4672 and #3272 we introduced error and crash reporting to Adblock Browser. Both features contain a modal dialog which reads 'Adblock Unexpectedly Quit. Would you like to send a report to fix the problem?'. The headline should be reworded to read "Adblock Browser". On top of that the additional sentence appears to be misleading, as it implies that if the user clicks the 'Send Report' button, that the issue will be automatically fixed.

I would suggest to change the headline and a slightly different description elaborating that, clicking the 'Send Report' button would, in fact, help us to fix the issue, and not directly fix the issue. The suggested sentence would be 'Would you like to send a report to help us try and fix the problem?'.

Environment

ABB 1.5.2 (1409)
iPhone 6s Plus
iOS 10.2

What to change

In all three instances:

  • In the crash report notification (start ABB 1.5.1, navigate to "Settings" -> "Devbuild Settings" -> "Crash application now")
  • In the error report notification (start ABB 1.5.1, navigate to "Settings" -> "Devbuild Settings" -> "Produce error now")
  • In the start-up error report notification (start ABB 1.5.1, navigate to "Settings" -> "Devbuild Settings" -> "Next App Start Failure")

replace the following text:

  • Replace Adblock Unexpectedly Quit with Adblock Browser Unexpectedly Quit
  • Replace Would you like to send a report to fix the problem? with Would you like to send a report to help us try and fix the problem?

Attachments (0)

Change History (6)

comment:1 Changed on 02/13/2017 at 12:15:59 PM by mario

  • Cc lisabielik added
  • Description modified (diff)

@Lisa, would you mind reviewing the suggested text change? IMHO it makes sense to be more specific -- as suggested -- in order to avoid misinterpretation.

comment:2 Changed on 02/13/2017 at 01:57:17 PM by lisabielik

@Mario

proposed text LGTM

comment:3 Changed on 02/13/2017 at 02:17:33 PM by mario

  • Priority changed from Unknown to P3
  • Ready set

Thanks, Lisa!

comment:4 Changed on 02/15/2017 at 12:48:45 PM by mario

  • Description modified (diff)
  • Ready unset
  • Type changed from defect to change

Expanded the scope of this issue to fix the erroneous sentence in all occurrences. Furthermore included "Adblock" to be replaced by "Adblock Browser" now that we have the possibility to change that text. (It came from HockeyApp and wasn't changeable before)

Pending review for the changes in the headline.

comment:5 Changed on 02/15/2017 at 02:13:19 PM by lisabielik

@mario:

Everything LGTM

comment:6 Changed on 02/15/2017 at 02:27:59 PM by mario

  • Ready set

Thanks Lisa!

Add Comment

Modify Ticket

Change Properties
Action
as new .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from (none).
Next status will be 'reviewing'.
 
Note: See TracTickets for help on using tickets.