Opened 3 years ago

Last modified 3 years ago

#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?

Change History (6)

comment:1 Changed 3 years ago 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 3 years ago by lisabielik

@Mario

proposed text LGTM

comment:3 Changed 3 years ago by mario

  • Priority changed from Unknown to P3
  • Ready set

Thanks, Lisa!

comment:4 Changed 3 years ago 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 3 years ago by lisabielik

@mario:

Everything LGTM

comment:6 Changed 3 years ago by mario

  • Ready set

Thanks Lisa!

Note: See TracTickets for help on using tickets.