Jeff Pihach

Results 68 comments of Jeff Pihach

This needs to wait until the charm conversion is done as authentication will need a slight overhaul to support that change then we can revisit this.

I'd propose that you don't modify the application name to append a type to it.

The issue that caused this PR to be created is still technically possible however the GUI has now been replaced with the Dashboard in 2.8 so it'll need some further...

To make this truly functional we would also have to take an `options` prop that would be expanded out into the `useTable` options argument so that configuration options could be...

After discussing this with @huwshimi and @Caleb-Ellis we've come to a different proposal that I think works quite well. Instead of enabling or disabling plugins and supplying configuration values as...

The default sub element (and arguably the only one ever used) is a `button` so it'll already have all the necessary elements on it. https://github.com/canonical-web-and-design/react-components/blob/master/src/components/Button/Button.tsx#L79 There has been some discussion...

When expanding the bundle/model visualization in the Juju Dashboard we use a modal to house the visualization. In the image below you can see that the title of the modal...

In the [Juju Dashboard]() We have a wrapper around the `Modal` component to do just this. You can see the [component here](https://github.com/canonical-web-and-design/jaas-dashboard/blob/a89a9301348018903f6e6ffaa42939429dd09f50/src/components/ConfirmationModal/ConfirmationModal.tsx#L4) and one example of the [usage here](https://github.com/canonical-web-and-design/jaas-dashboard/blob/6144e078cdf6bffcb14bf90f5260da32bd1d0474/src/panels/ActionsPanel/ActionsPanel.tsx#L348) What...

nah, when the work is being tackled we can have a quick pre-imp.

I use the S&F component often and I can see a world where hitting only the "x" to remove an item would not open the drop down but any other...