Skip to content

Commit

Permalink
remove unused EventsSelect component and the associated `showEvents…
Browse files Browse the repository at this point in the history
…Select` prop
  • Loading branch information
janmonschke committed Mar 12, 2024
1 parent f90040e commit 82bf54d
Show file tree
Hide file tree
Showing 16 changed files with 0 additions and 281 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ export interface HeaderActionProps {
isEventViewer?: boolean;
isSelectAllChecked: boolean;
onSelectAll: ({ isSelected }: { isSelected: boolean }) => void;
showEventsSelect: boolean;
showSelectAllCheckbox: boolean;
sort: SortColumnTable[];
tabType: string;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ export interface HeaderActionProps {
isEventViewer?: boolean;
isSelectAllChecked: boolean;
onSelectAll: ({ isSelected }: { isSelected: boolean }) => void;
showEventsSelect: boolean;
showSelectAllCheckbox: boolean;
sort: SortColumnTable[];
tabType: string;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ describe('checkbox control column', () => {
columnHeaders: [],
isSelectAllChecked: true,
onSelectAll: jest.fn(),
showEventsSelect: true,
showSelectAllCheckbox: true,
sort: [],
tabType: TimelineTabs.query,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ export const transformControlColumns = ({
isEventViewer={false}
isSelectAllChecked={isSelectAllChecked}
onSelectAll={onSelectPage}
showEventsSelect={false}
showSelectAllCheckbox={showCheckboxes}
sort={sort}
tabType={tabType}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ const defaultProps: HeaderActionProps = {
columnHeaders: [],
isSelectAllChecked: false,
onSelectAll: jest.fn(),
showEventsSelect: false,
showSelectAllCheckbox: false,
sort: [],
tabType: TimelineTabs.query,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ import { DEFAULT_ACTION_BUTTON_WIDTH } from '.';
import { EventsTh, EventsThContent } from '../../../timelines/components/timeline/styles';
import { StatefulRowRenderersBrowser } from '../../../timelines/components/row_renderers_browser';
import { EXIT_FULL_SCREEN } from '../exit_full_screen/translations';
import { EventsSelect } from '../../../timelines/components/timeline/body/column_headers/events_select';
import * as i18n from './translations';

const SortingColumnsContainer = styled.div`
Expand Down Expand Up @@ -71,7 +70,6 @@ const HeaderActionsComponent: React.FC<HeaderActionProps> = ({
isEventViewer = false,
isSelectAllChecked,
onSelectAll,
showEventsSelect,
showSelectAllCheckbox,
sort,
tabType,
Expand Down Expand Up @@ -264,14 +262,6 @@ const HeaderActionsComponent: React.FC<HeaderActionProps> = ({
</EventsThContent>
</EventsTh>
)}

{showEventsSelect && (
<EventsTh role="button">
<EventsThContent textAlign="center" width={DEFAULT_ACTION_BUTTON_WIDTH}>
<EventsSelect checkState="unchecked" timelineId={timelineId} />
</EventsThContent>
</EventsTh>
)}
</ActionsContainer>
);
};
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

This file was deleted.

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ describe('ColumnHeaders', () => {
isSelectAllChecked: false,
onSelectAll: jest.fn,
show: true,
showEventsSelect: false,
showSelectAllCheckbox: false,
sort,
tabType: TimelineTabs.query,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ export interface ColumnHeadersComponentProps {
isSelectAllChecked: boolean;
onSelectAll: OnSelectAll;
show: boolean;
showEventsSelect: boolean;
showSelectAllCheckbox: boolean;
sort: Sort[];
tabType: TimelineTabs;
Expand Down Expand Up @@ -92,7 +91,6 @@ export const ColumnHeadersComponent = ({
isSelectAllChecked,
onSelectAll,
show,
showEventsSelect,
showSelectAllCheckbox,
sort,
tabType,
Expand Down Expand Up @@ -217,7 +215,6 @@ export const ColumnHeadersComponent = ({
isEventViewer={isEventViewer}
isSelectAllChecked={isSelectAllChecked}
onSelectAll={onSelectAll}
showEventsSelect={showEventsSelect}
showSelectAllCheckbox={showSelectAllCheckbox}
sort={sort}
tabType={tabType}
Expand All @@ -239,7 +236,6 @@ export const ColumnHeadersComponent = ({
isEventViewer,
isSelectAllChecked,
onSelectAll,
showEventsSelect,
showSelectAllCheckbox,
sort,
tabType,
Expand All @@ -266,7 +262,6 @@ export const ColumnHeadersComponent = ({
isEventViewer={isEventViewer}
isSelectAllChecked={isSelectAllChecked}
onSelectAll={onSelectAll}
showEventsSelect={showEventsSelect}
showSelectAllCheckbox={showSelectAllCheckbox}
sort={sort}
tabType={tabType}
Expand All @@ -288,7 +283,6 @@ export const ColumnHeadersComponent = ({
isEventViewer,
isSelectAllChecked,
onSelectAll,
showEventsSelect,
showSelectAllCheckbox,
sort,
tabType,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,6 @@ export const StatefulBody = React.memo<Props>(
isSelectAllChecked={isSelectAllChecked}
onSelectAll={onSelectAll}
show={show}
showEventsSelect={false}
showSelectAllCheckbox={false}
sort={sort}
tabType={tabType}
Expand Down
6 changes: 0 additions & 6 deletions x-pack/plugins/translations/translations/fr-FR.json
Original file line number Diff line number Diff line change
Expand Up @@ -35635,12 +35635,6 @@
"xpack.securitySolution.timeline.descriptionTooltip": "Description",
"xpack.securitySolution.timeline.destination": "Destination",
"xpack.securitySolution.timeline.EqlQueryBarLabel": "Requête EQL",
"xpack.securitySolution.timeline.eventsSelect.actions.pinSelected": "Épingler la sélection",
"xpack.securitySolution.timeline.eventsSelect.actions.selectAll": "Tous",
"xpack.securitySolution.timeline.eventsSelect.actions.selectNone": "Aucune",
"xpack.securitySolution.timeline.eventsSelect.actions.selectPinned": "Épinglé",
"xpack.securitySolution.timeline.eventsSelect.actions.selectUnpinned": "Désépinglé",
"xpack.securitySolution.timeline.eventsSelect.actions.unpinSelected": "Désépingler la sélection",
"xpack.securitySolution.timeline.expandableEvent.alertTitleLabel": "Détails de l'alerte",
"xpack.securitySolution.timeline.expandableEvent.closeEventDetailsLabel": "fermer",
"xpack.securitySolution.timeline.expandableEvent.eventTitleLabel": "Détails de l'événement",
Expand Down
Loading

0 comments on commit 82bf54d

Please sign in to comment.