Skip to content

[Obs AI Assistant] Fix page height of the AI Assistant page#233924

Merged
viduni94 merged 4 commits intoelastic:mainfrom
viduni94:ai-assistant-full-page
Sep 5, 2025
Merged

[Obs AI Assistant] Fix page height of the AI Assistant page#233924
viduni94 merged 4 commits intoelastic:mainfrom
viduni94:ai-assistant-full-page

Conversation

@viduni94
Copy link
Contributor

@viduni94 viduni94 commented Sep 3, 2025

Closes #231292

Summary

The AI Assistant page was not taking the full height of the viewport as only the max block size was set.
This PR fixes it by defining the height of the page (with respect to the headers in each solution view and classic view).

Before

image

After

Current layout - Classic space

image

Current layout - Obs space

image

Current layout - Search space

image

New layout - Classic space

image

New layout - Obs space

image

New layout - Search space

image

Testing instructions

  1. Current layout - Verify that the AI Assistant page takes full height in a classic space, observability space and search space.
  2. Add the following to kibana.dev.yml to enable the new grid layout
feature_flags.overrides:
  core.chrome.layoutType: 'grid'
  1. Refresh Kibana
  2. Verify that the AI Assistant page takes full height in a classic space, observability space and search space in the new grid layout as well.

Checklist

  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.

Release notes

Fixes the page height of the Observability AI Assistant page

@viduni94 viduni94 added release_note:fix backport:skip This PR does not require backporting Team:Obs AI Assistant Observability AI Assistant v9.2.0 labels Sep 3, 2025
@viduni94 viduni94 self-assigned this Sep 3, 2025
@viduni94 viduni94 marked this pull request as ready for review September 4, 2025 12:59
@viduni94 viduni94 requested a review from a team as a code owner September 4, 2025 12:59
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant)

@viduni94
Copy link
Contributor Author

viduni94 commented Sep 4, 2025

Related to #225263

@viduni94 viduni94 added the ci:project-deploy-observability Create an Observability project label Sep 4, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@viduni94
Copy link
Contributor Author

viduni94 commented Sep 4, 2025

/oblt-deploy

@elasticmachine
Copy link
Contributor

elasticmachine commented Sep 4, 2025

💚 Build Succeeded

  • Buildkite Build
  • Commit: 5806dab
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-233924-5806dabbaa1a

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observabilityAIAssistantApp 177.7KB 177.7KB -8.0B

Saved Objects .kibana field count

Every field in each saved object type adds overhead to Elasticsearch. Kibana needs to keep the total field count below Elasticsearch's default limit of 1000 fields. Only specify field mappings for the fields you wish to search on or query. See https://www.elastic.co/guide/en/kibana/master/saved-objects-service.html#_mappings

id before after diff
_data_stream_timestamp 1 - -1
_doc_count 1 - -1
_ignored_source 1 - -1
_index_mode 1 - -1
_inference_fields 1 - -1
_tier 1 - -1
apm-custom-dashboards 5 - -5
apm-server-schema 2 - -2
apm-service-group 5 - -5
application_usage_daily 2 - -2
config 2 - -2
config-global 2 - -2
coreMigrationVersion 1 - -1
created_at 1 - -1
created_by 1 - -1
entity-definition 9 - -9
entity-discovery-api-key 2 - -2
event_loop_delays_daily 2 - -2
favorites 4 - -4
file 11 - -11
file-upload-usage-collection-telemetry 3 - -3
fileShare 5 - -5
infra-custom-dashboards 4 - -4
infrastructure-monitoring-log-view 2 - -2
intercept_trigger_record 5 - -5
legacy-url-alias 7 - -7
managed 1 - -1
ml-job 6 - -6
ml-module 13 - -13
ml-trained-model 7 - -7
monitoring-telemetry 2 - -2
namespace 1 - -1
namespaces 1 - -1
observability-onboarding-state 2 - -2
originId 1 - -1
product-doc-install-status 7 - -7
references 4 - -4
sample-data-telemetry 3 - -3
security-ai-prompt 8 - -8
slo 11 - -11
space 5 - -5
synthetics-monitor 34 - -34
synthetics-monitor-multi-space 34 - -34
tag 4 - -4
type 1 - -1
typeMigrationVersion 1 - -1
ui-metric 2 - -2
updated_at 1 - -1
updated_by 1 - -1
upgrade-assistant-ml-upgrade-operation 3 - -3
upgrade-assistant-reindex-operation 3 - -3
uptime-synthetics-api-key 2 - -2
url 5 - -5
usage-counters 2 - -2
total -246

History

cc @viduni94

@viduni94 viduni94 merged commit 1e22896 into elastic:main Sep 5, 2025
12 checks passed
shahargl pushed a commit to shahargl/kibana that referenced this pull request Sep 7, 2025
…233924)

Closes elastic#231292

## Summary

The AI Assistant page was not taking the full height of the viewport as
only the max block size was set.
This PR fixes it by defining the height of the page (with respect to the
headers in each solution view and classic view).

## Before

<img width="1542" height="1161" alt="image"
src="https://github.com/user-attachments/assets/0bde604d-0877-41ec-964c-7e9005f1a46d"
/>

## After

### Current layout - Classic space

<img width="1203" height="1276" alt="image"
src="https://github.com/user-attachments/assets/771ddb30-a52b-4b64-80c3-6a4156448632"
/>

### Current layout - Obs space

<img width="1203" height="1276" alt="image"
src="https://github.com/user-attachments/assets/366efff8-2dcc-44ce-a79a-1b7145abebed"
/>

### Current layout - Search space

<img width="1203" height="1275" alt="image"
src="https://github.com/user-attachments/assets/3fc3cc9d-d5d1-4a2e-8bf0-4e7d76f2675d"
/>

### New layout - Classic space

<img width="1206" height="1274" alt="image"
src="https://github.com/user-attachments/assets/b7f25bba-08d4-46a8-b2f9-582555e4220f"
/>

### New layout - Obs space

<img width="1204" height="1274" alt="image"
src="https://github.com/user-attachments/assets/c0efb081-a68e-4982-b14c-f8746239cedd"
/>

### New layout - Search space

<img width="1201" height="1274" alt="image"
src="https://github.com/user-attachments/assets/4e33aa5b-d1a0-4a6d-84c6-93b601101632"
/>

## Testing instructions

1. Current layout - Verify that the AI Assistant page takes full height
in a classic space, observability space and search space.
2. Add the following to `kibana.dev.yml` to enable the new grid layout
```
feature_flags.overrides:
  core.chrome.layoutType: 'grid'
```
3. Refresh Kibana
4. Verify that the AI Assistant page takes full height in a classic
space, observability space and search space in the new grid layout as
well.

### Checklist

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

### Release notes

Fixes the page height of the Observability AI Assistant page
KodeRad pushed a commit to KodeRad/kibana that referenced this pull request Sep 15, 2025
…233924)

Closes elastic#231292

## Summary

The AI Assistant page was not taking the full height of the viewport as
only the max block size was set.
This PR fixes it by defining the height of the page (with respect to the
headers in each solution view and classic view).

## Before

<img width="1542" height="1161" alt="image"
src="https://github.com/user-attachments/assets/0bde604d-0877-41ec-964c-7e9005f1a46d"
/>

## After

### Current layout - Classic space

<img width="1203" height="1276" alt="image"
src="https://github.com/user-attachments/assets/771ddb30-a52b-4b64-80c3-6a4156448632"
/>

### Current layout - Obs space

<img width="1203" height="1276" alt="image"
src="https://github.com/user-attachments/assets/366efff8-2dcc-44ce-a79a-1b7145abebed"
/>

### Current layout - Search space

<img width="1203" height="1275" alt="image"
src="https://github.com/user-attachments/assets/3fc3cc9d-d5d1-4a2e-8bf0-4e7d76f2675d"
/>

### New layout - Classic space

<img width="1206" height="1274" alt="image"
src="https://github.com/user-attachments/assets/b7f25bba-08d4-46a8-b2f9-582555e4220f"
/>

### New layout - Obs space

<img width="1204" height="1274" alt="image"
src="https://github.com/user-attachments/assets/c0efb081-a68e-4982-b14c-f8746239cedd"
/>

### New layout - Search space

<img width="1201" height="1274" alt="image"
src="https://github.com/user-attachments/assets/4e33aa5b-d1a0-4a6d-84c6-93b601101632"
/>

## Testing instructions

1. Current layout - Verify that the AI Assistant page takes full height
in a classic space, observability space and search space.
2. Add the following to `kibana.dev.yml` to enable the new grid layout
```
feature_flags.overrides:
  core.chrome.layoutType: 'grid'
```
3. Refresh Kibana
4. Verify that the AI Assistant page takes full height in a classic
space, observability space and search space in the new grid layout as
well.

### Checklist

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

### Release notes

Fixes the page height of the Observability AI Assistant page
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Sep 24, 2025
…233924)

Closes elastic#231292

## Summary

The AI Assistant page was not taking the full height of the viewport as
only the max block size was set.
This PR fixes it by defining the height of the page (with respect to the
headers in each solution view and classic view).

## Before

<img width="1542" height="1161" alt="image"
src="https://github.com/user-attachments/assets/0bde604d-0877-41ec-964c-7e9005f1a46d"
/>

## After

### Current layout - Classic space

<img width="1203" height="1276" alt="image"
src="https://github.com/user-attachments/assets/771ddb30-a52b-4b64-80c3-6a4156448632"
/>

### Current layout - Obs space

<img width="1203" height="1276" alt="image"
src="https://github.com/user-attachments/assets/366efff8-2dcc-44ce-a79a-1b7145abebed"
/>

### Current layout - Search space

<img width="1203" height="1275" alt="image"
src="https://github.com/user-attachments/assets/3fc3cc9d-d5d1-4a2e-8bf0-4e7d76f2675d"
/>

### New layout - Classic space

<img width="1206" height="1274" alt="image"
src="https://github.com/user-attachments/assets/b7f25bba-08d4-46a8-b2f9-582555e4220f"
/>

### New layout - Obs space

<img width="1204" height="1274" alt="image"
src="https://github.com/user-attachments/assets/c0efb081-a68e-4982-b14c-f8746239cedd"
/>

### New layout - Search space

<img width="1201" height="1274" alt="image"
src="https://github.com/user-attachments/assets/4e33aa5b-d1a0-4a6d-84c6-93b601101632"
/>

## Testing instructions

1. Current layout - Verify that the AI Assistant page takes full height
in a classic space, observability space and search space.
2. Add the following to `kibana.dev.yml` to enable the new grid layout
```
feature_flags.overrides:
  core.chrome.layoutType: 'grid'
```
3. Refresh Kibana
4. Verify that the AI Assistant page takes full height in a classic
space, observability space and search space in the new grid layout as
well.

### Checklist

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

### Release notes

Fixes the page height of the Observability AI Assistant page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting ci:project-deploy-observability Create an Observability project release_note:fix Team:Obs AI Assistant Observability AI Assistant v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Obs AI Assistant] The AI Assistant page does not utilize the full height of the viewport

3 participants