Results 198 comments of Cory Simmons

```js set tutsBaseURL(url) { console.log(111111) this._tutsBaseURL = url this.setup() } ``` How come my console.log doesn't work? :\ The setter is getting triggered in inject.js right? `visbug.tutsBaseURL = src_path.slice(0, src_path.lastIndexOf('/'))`

This fixes it: ```js ReactModal.defaultStyles = {} // Flushes all of react-modal's styles ``` ```css .ReactModal__Html--open, .ReactModal__Body--open { overflow: hidden; /* prevents background page from scrolling when the modal is...

@diasbruno Yup sure can. Busy the next few days but might have some time over the weekend to prepare something. It's really not much more than what I posted above,...

Sorry, still working on work-work stuff... 😩 Here's what I've got: Code I'm using: https://gist.github.com/corysimmons/c1adcc4af71e7d9eb9ced0437bd208d6 The relevant bit: ```css .ReactModal__Html--open, .ReactModal__Body--open { overflow: hidden; } .ReactModal__Overlay { position: fixed; z-index:...

@justinledelson It's insane you commented on this. I literally just implemented this and was looking at this issue for my Gist after years. Anyway, my Gist still works great. Just...

@diasbruno I definitely think the ability to scroll the content area should be the responsibility of react-modal and there should be a fix & probably a major version bump (since...

Fair enough. It's also possible you don't need to do a major bump and just keep the slightly bloated markup (it's really not bad, we're talking removing a single wrapping...

That's totally doable but now you're bloating the API with "Oh, be sure to read this part of the docs, and add this setting to your project to be able...

The measure animation bug on web doesn't seem to happen on small-width viewports. As the viewport gets wider, the popover changes the side it appears on (going from bottom to...

I'm prrretty lazy. Anyone suggest a Mac client that supports this?