Override value of "bottom" CSS attribute

[brianespinosa]

I looked through the code for this component and it appears that the bottom CSS attribute is hard coded in this component with the logic for placement passed on to each notifications. Are you opposed to a solution that would allow the barStyle property to also set bottom in a dynamic way?

[pburtchaell]

Part of the problem is the dynamic position is determined using a numeric value and then the unit is concatenated with the numeric value, e.g., barPositon + 'rem'. I am not opposed to add this, but I think there would need to be two new props introduced: bottomPositionUnit and bottomPosition.

[brianespinosa]

Yeah, that was what I was thinking. It would need to be a prop specific to NotificationStack. I mainly wanted to confirm you were okay with this before I did the work and submitted a PR. I will update the README documentation for NotificationStack also to reflect that barStyle and others can now be passed to children from your last update.

[pburtchaell]

Awesome! I'll keep an eye out for the PR. 😄

[pburtchaell]

Another possible approach to this: make barStyle a function and the argument can be the index of the notification.

// if this is the 2nd notification in the stack, index will have a value of 2
barStyle={index => ({
  bottom: `${index*3}rem` // do some math to get the right position
})}

[RamonBeast]

I found this issue while looking for a solution and that's a good idea, at least placement of the notifications can be more flexible.

[pburtchaell]

@brianespinosa, any interest in making a PR for this?