This might be slightly easier to read if the function was on the next line so that it reads like this

```js
modalService.showModal(...
  function () {
  ...
  },
  //comment
  funtion () {
  ..
  });
```

Not a big deal, just a thought I had.