<Box /> responsiveBreakpoint not functional

changing the value of responsiveBreakpoint to anything different from ‘small’ generating crash.

desired behavior is to implement the borderSize, edgeSize for medium and large in base theme, similar to already implemented small https://github.com/grommet/grommet/blob/d6c3a7f83c9fc3dea641eafac16ac87ba2121890/src/js/themes/base.js#L124

const theme = {
  box: {
    responsiveBreakpoint: "medium"
  }
};

    <Grommet theme={theme}>
      <Box background="brand" pad="large" />
    </Grommet>

code sandbox https://codesandbox.io/s/pj8175227j

[atanasster]

@oorestisime the current use of global.edgeSize, global.borderSize is when responsive is false and also in several cases where it’s hard-coded, while the small breakpoint - when Box turns into the column direction from row-responsive. Imho it would be best to have those spacings/borders adapt not only when the breakpoint is switched, but for all sizes depending on the current screen size. For example use the medium sizes when the screen is in that size range, even if we have a responsiveBreakpoint of small. For now, just to avoid crashing, I think either add the exact same values to medium as already defined for small, or have checks in the code to avoid the crashing.

[jcfilben]

This issue was fixed in #4644 so I think this issue can be closed