broulik created this revision.
broulik added reviewers: Plasma, fvogt.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  When the extension is reloaded, its context is invalidated and any call to an 
extension API would just throw.
  Catch the most important bits of error reporting on save, so you're not left 
wondering why your settings don't change.

TEST PLAN
  Was working on some settings bits, had to reload the extension often to test 
things and was always wondering why it didn't do anything when I forgot to also 
reload the settings page
  
  Now at least I get an error of sorts:
  
  > Saving settings failed (Error: Extension context invalidated.)

REPOSITORY
  R856 Plasma Browser Integration

REVISION DETAIL
  https://phabricator.kde.org/D20819

AFFECTED FILES
  extension/options.js

To: broulik, #plasma, fvogt
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to