React-beautiful-dnd: Please remove noisy console message in development.

[iMerica]

Hi,

Thanks for building this, I'm really liking it so far except for one thing:

It's not appropriate to pollute everyones console with noisy messages like this. According to my count (find . -name 'package.json' | wc -l), I have 1,855 dependencies in my project and out of all of these dependencies, only ONE is polluting my console with messages by default: react-beautiful-dnd.

I shouldn't have to enable production builds in my webpack configuration just to make one dependency not emit a noisy marketing banner 😃 .

Thank you,

Michael

[alexreardon]

Here you go: https://github.com/atlassian/react-beautiful-dnd#disable-warnings

[alexreardon]

Also, it is not a marketing banner. Nested scroll containers is not supported and can lead to bugs 👍

[alexreardon]

I linked to the issue in the message to show that we plan on overcoming the limitation

[alexreardon]

Perhaps we could add a link to the disable warnings guide in the messages? Thoughts @iMerica ?

[iMerica]

Maybe the warnings could look a little less banner-y and more warning-y

For example, in React:

Proposed changes

• Remove the "react-beautiful-dnd" green text so this warning looks less like an advertisement and more like a warning.
• Remove "This is a development only message."
• Example:
  

[iMerica]

Looks like the next version of React will include a native way of reporting these things so they look consistent (the crux of this issue).

https://github.com/facebook/react/pull/15170

cc/ @alexreardon