Use default exports by default

[turnerhayes]

For many components, we are currently using named exports instead of default exports (e.g. import { MyButton } from './MyButton; rather than import MyButton from './MyButton';.) I feel like we should be using default exports for components, with named exports only if it needs to export additional symbols.

This is a somewhat stylistic question; thoughts? Anyone feel strongly otherwise?

[knod]

I'd err more in the opposite direction. I feel that the more consistent we can be, the easier it is for new people getting started. Having both import MyButton from './MyButton'; and import { Button1, Button2 } from './Buttons'; seems like two behaviors to keep track of instead of one.

[knod]

What are the pros to using default exports?

[turnerhayes]

Mainly terseness, I suppose. I'm not clear why import { Button1, Button2 } from './Buttons'; is the pattern we'd use; I'm inclined to tend towards one component (or at least one exported component) per file, so I would probably tend to use

import Button1 from './Button1'; // or 'Buttons/Button1', if we want to group them together
import Button2 from './Button2';

[knod]

So is this more of a conversation about one abstracted component per file?

[turnerhayes]

I think that plays into it, certainly, but there are components (I believe) that only have one component in a file and still use named exports.

[knod]

Maybe we'll do some real-time talking about this and put those conclusions in here.