Closed PixievoltNo1 closed 7 years ago
Anything I can do to help get this merged?
Thank you for the reminder. Looks good. Merging in. @timmfin can rebuild the docs and we don't even need an npm publish. Thanks so much Pikadude 👍
I've rebuilt and the docs, including https://github.com/HubSpot/vex/commit/ff15cf55dc9d92f24b3a7b1d32c3c4dbeb5202b5 📖 🆕
With these edits to the doc pages:
vex.defaultOptions
is introduced to developers who, for whatever reason, skipped the Basic page and went right to AdvancedcloseAllOnPopState
option is clarified, including the formerly-unmentioned wrinkle that it works only as a default optionSomeone at HubSpot, perhaps @timmfin, will need to update vex's HubSpot site after this PR goes through.