Skip to content

[Cases] One Attachments tab#243708

Merged
lgestc merged 35 commits intoelastic:mainfrom
lgestc:attachments_tab
Dec 1, 2025
Merged

[Cases] One Attachments tab#243708
lgestc merged 35 commits intoelastic:mainfrom
lgestc:attachments_tab

Conversation

@lgestc
Copy link
Contributor

@lgestc lgestc commented Nov 20, 2025

Summary

This PR implements #243968, moving all attachments tabs into one, with subtab navigation implemented.

Screenshot 2025-11-27 at 13 24 20

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

@lgestc lgestc marked this pull request as ready for review November 24, 2025 12:17
@lgestc lgestc requested a review from a team as a code owner November 24, 2025 12:17
@lgestc lgestc added release_note:enhancement backport:skip This PR does not require backporting Team:Cases Security Solution Cases team v9.3.0 9.3 candidate labels Nov 24, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cases (Team:Cases)

@lgestc lgestc marked this pull request as draft November 24, 2025 14:48
@lgestc lgestc marked this pull request as ready for review November 24, 2025 15:03
@delanni delanni removed the request for review from a team November 25, 2025 14:37
@lgestc lgestc requested a review from a team as a code owner November 26, 2025 15:01
@janmonschke
Copy link
Contributor

@lgestc These margins seem a little big compared to the design:

image
@janmonschke
Copy link
Contributor

@lgestc The tabs here seem out of place. They're not showing on your screenshot but they're showing for me locally 🤔

image
@elastic elastic deleted a comment from elasticmachine Nov 27, 2025
@lgestc lgestc force-pushed the attachments_tab branch 2 times, most recently from 1600625 to 0a12912 Compare November 27, 2025 12:09
@lgestc lgestc requested a review from janmonschke November 27, 2025 12:24
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
cases 1191 1193 +2

Async chunks

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

id before after diff
cases 1.4MB 1.4MB +1.4KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
cases 142.3KB 142.3KB +28.0B

History

@lgestc lgestc requested a review from a team December 1, 2025 12:02
@lgestc lgestc merged commit 6dba237 into elastic:main Dec 1, 2025
12 checks passed
tkajtoch pushed a commit to tkajtoch/kibana that referenced this pull request Dec 1, 2025
## Summary

This PR implements elastic#243968,
moving all attachments tabs into one, with subtab navigation
implemented.

<img width="1395" height="656" alt="Screenshot 2025-11-27 at 13 24 20"
src="https://github.com/user-attachments/assets/23c88857-8f5a-409b-a899-9a44135dab54"
/>

### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Dec 2, 2025
## Summary

This PR implements elastic#243968,
moving all attachments tabs into one, with subtab navigation
implemented.

<img width="1395" height="656" alt="Screenshot 2025-11-27 at 13 24 20"
src="https://github.com/user-attachments/assets/23c88857-8f5a-409b-a899-9a44135dab54"
/>

### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
JordanSh pushed a commit to JordanSh/kibana that referenced this pull request Dec 9, 2025
## Summary

This PR implements elastic#243968,
moving all attachments tabs into one, with subtab navigation
implemented.

<img width="1395" height="656" alt="Screenshot 2025-11-27 at 13 24 20"
src="https://github.com/user-attachments/assets/23c88857-8f5a-409b-a899-9a44135dab54"
/>

### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

9.3 candidate backport:skip This PR does not require backporting release_note:enhancement Team:Cases Security Solution Cases team v9.3.0

7 participants