Brittany

Results 28 comments of Brittany

also working on a different default so I'll push that up here soon

> > It seems weird that in this story the DataTable originally has a step size of 10 and the step selector is left blank. It feels like if the...

> > It feels like we don't need the padding to the right of the text since the button next to it has padding. This could help with displaying larger...

> I'm still seeing the extra spacing on the icon only anchor. See the blue area between the bottom of the icon and the top of the green padding. >...

> Looking good, dropped some naming suggestions. > > Also, I see we have this on TextInput and DateInput. Should this be supported on MaskedInput also? @jcfilben I'll hold off...

If I have this in my theme ``` readOnly: { background: 'light-2', border: { color: 'brand', }, }, ``` below is what I get would we not expect the first...

@sabarnix thank you for starting this PR just want to let you know that we are going to discuss this behavior a little more among our team.. [Here](https://github.com/grommet/hpe-design-system/issues/3600) is another...

@sabarnix no solution here would you mind filing an issue?

Adding to the dependencies that we can not update at this time ``` "react-dom": "^18.2.0", "react-is": "^18.2.0", ``` thrown: "Warning: `ReactDOMTestUtils.act` is deprecated in favor of `React.act`. Import `act` from...

"react-dom": "^18.2.0", "react-is": "^18.2.0", These can be updated now with no issue from `act warnings`