Skip to content

Commit

Permalink
remove Alert component
Browse files Browse the repository at this point in the history
There are a lot of unknowns and context dependendant questions to
implement Alerts in a good way. The current Alert component just had a
role set, and it had no JS attached.

We will revisit this, once we start working on Alert Dialogs,
Notification center notifications (dismissable, hide after x time, ...)
  • Loading branch information
RobinMalfait committed Mar 31, 2021
1 parent c6c4cd9 commit ec6f1d6
Show file tree
Hide file tree
Showing 13 changed files with 0 additions and 417 deletions.
2 changes: 0 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Add `Popover` component ([#220](https://github.com/tailwindlabs/headlessui/pull/220))
- All components that accept a `className`, can now also receive a function with the renderProp argument ([#257](https://github.com/tailwindlabs/headlessui/pull/257))
- Add `RadioGroup` component ([#274](https://github.com/tailwindlabs/headlessui/pull/274))
- Add `Alert` component ([#274](https://github.com/tailwindlabs/headlessui/pull/274))

## [Unreleased - Vue]

Expand All @@ -40,7 +39,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Added

- Add `SwitchDescription` component, which adds the `aria-describedby` to the actual Switch ([#220](https://github.com/tailwindlabs/headlessui/pull/220))
- Add `Alert` component ([#279](https://github.com/tailwindlabs/headlessui/pull/279))

## [@headlessui/react@v0.3.1] - 2021-02-11

Expand Down
1 change: 0 additions & 1 deletion packages/@headlessui-react/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ _This project is still in early development. New components will be added regula
- [Dialog](./src/components/dialog/README.md)
- [Popover](./src/components/popover/README.md)
- [Radio Group](./src/components/radio-group/README.md)
- [Alert](./src/components/alert/README.md)

### Roadmap

Expand Down
126 changes: 0 additions & 126 deletions packages/@headlessui-react/pages/dialog/dialog-with-alert.tsx

This file was deleted.

52 changes: 0 additions & 52 deletions packages/@headlessui-react/src/components/alert/README.md

This file was deleted.

31 changes: 0 additions & 31 deletions packages/@headlessui-react/src/components/alert/alert.test.tsx

This file was deleted.

48 changes: 0 additions & 48 deletions packages/@headlessui-react/src/components/alert/alert.tsx

This file was deleted.

1 change: 0 additions & 1 deletion packages/@headlessui-react/src/index.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import * as HeadlessUI from './index'
*/
it('should expose the correct components', () => {
expect(Object.keys(HeadlessUI)).toEqual([
'Alert',
'Dialog',
'Disclosure',
'FocusTrap',
Expand Down
1 change: 0 additions & 1 deletion packages/@headlessui-react/src/index.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
export * from './components/alert/alert'
export * from './components/dialog/dialog'
export * from './components/disclosure/disclosure'
export * from './components/focus-trap/focus-trap'
Expand Down
51 changes: 0 additions & 51 deletions packages/@headlessui-vue/src/components/alert/README.md

This file was deleted.

Loading

0 comments on commit ec6f1d6

Please sign in to comment.