.on('change') event object should have a source value

I think it's out of scope of the library. All of refresh() is called by users. It means that users can always know whether it's isFromUI or isFromRefresh.

Feel free to reopen the issue if you have any updates.

I think it's out of scope of the library. All of refresh() is called by users. It means that users can always know whether it's isFromUI or isFromRefresh.

@cocopon I understand that is out scope, but still think that will improve the developer experience if it avoid unnecesary boilerplate to validate manually with externals boolean flags, more if we have a lot active panes and wanna check all of them.

My particular case is that i'm using vue and firebase, and when i'm having an update from firebase y call the .refresh() func for some active panes, the downside is that triggers the .on('change') event where i push the UI updates back to firesbase, having like 2 or more duplicated requests.

Because of that, having the source that triggers the event inside the event object itself or just the capability to disable temporarily the on('change') event if we call page.refresh(false) or something similar could improve the DX making the event litle more comprehensive

You can create a utility like this:

// This is just an example, not tested...

class RefreshContainer {
  private readonly pane: Pane;
  private refreshing_ = false;

  constructor(pane: Pane) {
    this.pane = pane;
  }

  get refreshing(): boolean {
    return this.refreshing_;
  }

  public refresh(): void {
    this.refreshing_ = true;
    this.pane.refresh();
    this.refreshing_ = false;
  }
}

// ...

const rc = new RefreshContainer(new Pane());

rc.pane.on('change', (ev) => {
  if (rc.refreshing) {
    // Do something
  }
});

// ...

rc.refresh();

You can also publish this kind of utility as a npm package to avoid copy-pasting the code.

Packages with lots of features may be convenient, but they are less maintainable and sometimes lose their concept, IMO.

Thanks for the great plugin
I too would like to have a flag like "ev.isFromRefresh".
Please kindly consider this matter.