Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove angular-bootstrap dependency. #7591

Conversation

cjcenizal
Copy link
Contributor

Copy required code into src/ui/public/angular-bootstrap directory for future refactoring and deprecation.

Goals

Our UI components should have a coherent interface with consistent rules and patterns of use. The first step is to gain control over 100% of the components we use in our UI. This means migrating off Bootstrap and third-party UI frameworks like Angular-UI.

@cjcenizal cjcenizal force-pushed the refactor/remove-angular-bootstrap-dependency branch 2 times, most recently from 6ffe248 to 4a552b8 Compare July 1, 2016 22:13
@Bargs
Copy link
Contributor

Bargs commented Jul 1, 2016

Assuming tests pass, LGTM

@Bargs Bargs removed their assignment Jul 1, 2016
@tylersmalley
Copy link
Contributor

jenkins, test it

@tylersmalley
Copy link
Contributor

What are your thoughts on getting the src/* tree here for the modules we use? Importing a single concatenated file is going to be hard to maintain going forward unless that is not a concern.

@cjcenizal
Copy link
Contributor Author

Shoot, you're right. Editing the templates will especially be a PITA. I'll replace the compiled file with the source and original html templates.

@cjcenizal
Copy link
Contributor Author

@tylersmalley I just incorporated your suggestion. Could you take a look? Turns out the eslintignore file isn't used by our eslint task: #7622

@cjcenizal cjcenizal force-pushed the refactor/remove-angular-bootstrap-dependency branch from 83c2b4f to b08f016 Compare July 5, 2016 19:27
@tylersmalley
Copy link
Contributor

This is much more maintainable - thanks for the update.

LGTM

@cjcenizal cjcenizal merged commit f23d98e into elastic:master Jul 5, 2016
@cjcenizal cjcenizal deleted the refactor/remove-angular-bootstrap-dependency branch July 5, 2016 22:41
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
…r-bootstrap-dependency

Remove angular-bootstrap dependency.

Former-commit-id: f23d98e
cee-chen added a commit that referenced this pull request Mar 26, 2024
`v93.4.0` ⏩ `v93.5.1`

---

## [`v93.5.1`](https://github.com/elastic/eui/releases/v93.5.1)

**Bug fixes**

- Fixed unvirtualized `EuiSelectable`s to not cause Jest/jsdom errors on
active option change ([#7618](elastic/eui#7618))

## [`v93.5.0`](https://github.com/elastic/eui/releases/v93.5.0)

- `EuiHeaderLinks` now accepts a `children` render function that will be
passed a `closeMobilePopover` callback, allowing consumers to close the
mobile popover by its content
([#7603](elastic/eui#7603))
- Updated `EuiSelectable` to support scrolling list containers when
`listProps.isVirtualization` is set to `false`
([#7609](elastic/eui#7609))

**Bug fixes**

- Fixed `EuiIconTip`'s default `aria-label` text to be i18n tokenizable
([#7606](elastic/eui#7606))
- Fixed `EuiTextArea`'s CSS box model to no longer render a few extra
pixels of strut height
([#7607](elastic/eui#7607))

**Dependency updates**

- Updated `@types/refractor` to v3.4.0
([#7590](elastic/eui#7590))
- Updated `@types/lodash` to v4.14.202
([#7591](elastic/eui#7591))
- Removed `@types/resize-observer-browser` dependency. `ResizeObserver`
types should already be baked in to Typescript as of 4.2+
([#7592](elastic/eui#7592))
- Updated `classnames` to v2.5.1
([#7593](elastic/eui#7593))
- Updated `@types/numeral` to v2.0.5
([#7594](elastic/eui#7594))
- Updated `@types/react-window` to 1.8.8
([#7597](elastic/eui#7597))
- Updated `prop-types` to v15.18.1
([#7602](elastic/eui#7602))
- Removed `prop-types` as a peer dependency, per package recommendation
([#7602](elastic/eui#7602))

**Accessibility**

- `EuiIcons` no longer apply `aria-hidden` to empty icons, as long as a
valid title or label is provided to the icon. In particular, this is
intended to improve the accessibility of loading `EuiIconTip`s.
([#7606](elastic/eui#7606))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants