Here different variables are given to `stylePicker.setValue` and `openStyle`, while in the two other cases the same variable is passed, slightly confuses when reading the code. Any reason for doing so? Otherwise might be better to streamline things here as well. Possibly there should be even a single method
```
function setStyle(styleId) {
    stylePicker.setValue(styleId);
    openStyle(styleId);
}
```
which is called in all three places. Avoid code duplication and makes sure future changes will not miss any of the three spots.