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

[ML][UX]: Consistent Layout and UI Enhancements for ML Pages #203813

Merged
merged 28 commits into from
Dec 23, 2024
Merged
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
6a0d029
anomaly timeline use eui button for add to actions popover
rbrtj Dec 11, 2024
28d27b0
timeseries explorer controls layout update
rbrtj Dec 11, 2024
924c396
log rate analysis use eui button for add to actions popover
rbrtj Dec 11, 2024
a3b2030
log rate analysis use eui button for add to actions popover & auto sa…
rbrtj Dec 11, 2024
02aa1f3
remove unused import
rbrtj Dec 11, 2024
3ae73e8
change point detection use eui button for add to actions popover
rbrtj Dec 11, 2024
0e0f1de
update button colors to text
rbrtj Dec 12, 2024
9960e5c
single metric viewer embeddable color
rbrtj Dec 12, 2024
a0bb316
update buttons to button icon with boxes horizontal icon
rbrtj Dec 12, 2024
18fe658
use size s as default size for add to button
rbrtj Dec 13, 2024
9142f10
keep consistent spaces between controls around ml pages
rbrtj Dec 13, 2024
a0bea28
fix overflowing date picker inside various ml pages
rbrtj Dec 13, 2024
9aa70f6
fix data visualizer layout
rbrtj Dec 13, 2024
63c3193
update toasts
rbrtj Dec 16, 2024
4db66c5
Merge remote-tracking branch 'upstream/main' into ml-add-to-action-ui…
rbrtj Dec 17, 2024
24eaf3d
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine Dec 17, 2024
e9108d6
update anomaly panel menu button
rbrtj Dec 17, 2024
4c193b7
add icons for attachements actions
rbrtj Dec 18, 2024
7f408c6
internationalization for cases toast messages && plural for anomaly c…
rbrtj Dec 19, 2024
cd56f03
timeseriesexplorer detector dropdown min width && wrap controls
rbrtj Dec 19, 2024
dfd2494
Merge remote-tracking branch 'upstream/main' into ml-add-to-action-ui…
rbrtj Dec 19, 2024
847d00e
different message for change point table and chart attachments && plu…
rbrtj Dec 19, 2024
d73ab5d
revert accidentaly removed comment
rbrtj Dec 19, 2024
7f47fad
rename titles variable to cases_toast_messages_titles
rbrtj Dec 19, 2024
e679f8e
data visualizer view revert time range selector id
rbrtj Dec 19, 2024
c9bf590
update cases functional tests to respect new toast title
rbrtj Dec 19, 2024
1368a37
update more tests to respect new toast title
rbrtj Dec 19, 2024
f54f968
Merge branch 'main' into ml-add-to-action-ui-update
rbrtj Dec 23, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix overflowing date picker inside various ml pages
rbrtj committed Dec 13, 2024
commit a0bea28864bf0c3c1df06a4ba5bc65ac4a813916
Original file line number Diff line number Diff line change
@@ -90,10 +90,6 @@ interface DatePickerWrapperProps {
/**
* Boolean flag to set use of flex group wrapper
*/
flexGroup?: boolean;
/**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like you accidentaly removed the comment for the next prop and not flexGroup.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, done in: #d73ab5d

* Boolean flag to disable the date picker
*/
isDisabled?: boolean;
/**
* Boolean flag to force change from 'Refresh' to 'Update' state
@@ -123,7 +119,6 @@ export const DatePickerWrapper: FC<DatePickerWrapperProps> = (props) => {
isLoading = false,
showRefresh,
width,
flexGroup = true,
isDisabled = false,
needsUpdate,
onRefresh,
@@ -363,6 +358,8 @@ export const DatePickerWrapper: FC<DatePickerWrapperProps> = (props) => {
</>
);

const flexGroup = !isTimeRangeSelectorEnabled || isAutoRefreshOnly === true;

const wrapped = flexGroup ? (
<EuiFlexGroup gutterSize="s" alignItems="center">
{flexItems}
Original file line number Diff line number Diff line change
@@ -115,7 +115,6 @@ export const PageHeader: FC<PageHeaderProps> = ({ onRefresh, needsUpdate }) => {
isAutoRefreshOnly={!hasValidTimeField}
showRefresh={!hasValidTimeField}
width="full"
flexGroup={!hasValidTimeField}
onRefresh={onRefresh}
needsUpdate={needsUpdate}
/>,
Original file line number Diff line number Diff line change
@@ -80,7 +80,6 @@ export const PageHeader: FC = () => {
isAutoRefreshOnly={!hasValidTimeField}
showRefresh={!hasValidTimeField}
width="full"
flexGroup={!hasValidTimeField}
/>,
hasValidTimeField && (
<FullTimeRangeSelector