Skip to content

[Dashboard] Fix panel borders#232676

Merged
Heenawter merged 1 commit intoelastic:mainfrom
Heenawter:fix-panel-borders_2025-08-22
Aug 22, 2025
Merged

[Dashboard] Fix panel borders#232676
Heenawter merged 1 commit intoelastic:mainfrom
Heenawter:fix-panel-borders_2025-08-22

Conversation

@Heenawter
Copy link
Contributor

@Heenawter Heenawter commented Aug 22, 2025

Closes #232671

Summary

The PresentationPanelHoverActionsWrapper styles were hard coded to be in edit mode, which meant that the panel border wasn't changing to solid in view mode as it should. This PR fixes that by properly passing the view mode to the panel styles.

Before After
Aug-22-2025 09-23-48 Aug-22-2025 09-23-02

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.
@Heenawter Heenawter self-assigned this Aug 22, 2025
@Heenawter Heenawter added Feature:Dashboard Dashboard related features release_note:fix Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// loe:small Small Level of Effort impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. backport:skip This PR does not require backporting labels Aug 22, 2025
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

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
presentationPanel 99.0KB 99.0KB +37.0B

cc @Heenawter

@Heenawter Heenawter marked this pull request as ready for review August 22, 2025 17:54
@Heenawter Heenawter requested a review from a team as a code owner August 22, 2025 17:54
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for reporting and fixing issue
code review only

@Heenawter Heenawter merged commit edc1981 into elastic:main Aug 22, 2025
26 checks passed
@Heenawter Heenawter deleted the fix-panel-borders_2025-08-22 branch August 22, 2025 18:07
qn895 pushed a commit to qn895/kibana that referenced this pull request Aug 26, 2025
Closes elastic#232671

## Summary

The `PresentationPanelHoverActionsWrapper` styles were hard coded to be
in edit mode, which meant that the panel border wasn't changing to solid
in view mode as it should. This PR fixes that by properly passing the
view mode to the panel styles.

| Before | After |
|--------|--------|
| ![Aug-22-2025
09-23-48](https://github.com/user-attachments/assets/800d70d7-7e0b-4fb0-b18d-33701f8d2c9a)
| ![Aug-22-2025
09-23-02](https://github.com/user-attachments/assets/fb3b8987-d437-4b41-a404-f60af691deed)
|




### 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.
KodeRad pushed a commit to KodeRad/kibana that referenced this pull request Aug 28, 2025
Closes elastic#232671

## Summary

The `PresentationPanelHoverActionsWrapper` styles were hard coded to be
in edit mode, which meant that the panel border wasn't changing to solid
in view mode as it should. This PR fixes that by properly passing the
view mode to the panel styles.

| Before | After |
|--------|--------|
| ![Aug-22-2025
09-23-48](https://github.com/user-attachments/assets/800d70d7-7e0b-4fb0-b18d-33701f8d2c9a)
| ![Aug-22-2025
09-23-02](https://github.com/user-attachments/assets/fb3b8987-d437-4b41-a404-f60af691deed)
|




### 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.
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 Feature:Dashboard Dashboard related features impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort release_note:fix Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v9.2.0

4 participants