webpack/scenes/RedHatRepositories/components/EnabledRepository.js in katello-3.7.1.1 vs webpack/scenes/RedHatRepositories/components/EnabledRepository.js in katello-3.8.0.rc1

- old
+ new

@@ -1,16 +1,14 @@ import React, { Component } from 'react'; import PropTypes from 'prop-types'; -import { ListView } from 'patternfly-react'; +import cx from 'classnames'; +import { ListView, Spinner, OverlayTrigger, Tooltip } from 'patternfly-react'; import { connect } from 'react-redux'; import RepositoryTypeIcon from './RepositoryTypeIcon'; import { setRepositoryDisabled } from '../../../redux/actions/RedHatRepositories/enabled'; import api from '../../../services/api'; -import { notify } from '../../../move_to_foreman/foreman_toast_notifications'; -import { getResponseErrorMsgs } from '../../../move_to_foreman/common/helpers'; -import EnabledRepositoryContent from './EnabledRepositoryContent'; class EnabledRepository extends Component { constructor(props) { super(props); @@ -49,16 +47,13 @@ }; api .put(url, data) .then(this.setDisabled) - .catch(({ response }) => { - const errors = getResponseErrorMsgs(response); - errors.forEach((error) => { - notify({ message: error, type: 'error' }); - }); + .catch(() => { this.setState({ loading: false }); + // TODO: Add error component }); }; this.disableTooltipId = `disable-${props.id}`; } @@ -70,14 +65,28 @@ return ( <ListView.Item key={id} actions={ - <EnabledRepositoryContent - loading={this.state.loading} - disableTooltipId={this.disableTooltipId} - disableRepository={this.disableRepository} - /> + <Spinner loading={this.state.loading} inline> + <OverlayTrigger + overlay={<Tooltip id={this.disableTooltipId}>{__('Disable')}</Tooltip>} + placement="bottom" + trigger={['hover', 'focus']} + rootClose={false} + > + <button + onClick={this.disableRepository} + style={{ + backgroundColor: 'initial', + border: 'none', + color: '#0388ce', + }} + > + <i className={cx('fa-2x', 'fa fa-minus-circle')} /> + </button> + </OverlayTrigger> + </Spinner> } leftContent={<RepositoryTypeIcon id={id} type={type} />} heading={name} description={`${arch} ${releasever || ''}`} stacked