Skip to content

Conversation

@ek-so
Copy link
Contributor

@ek-so ek-so commented Sep 9, 2025

Summary

Some tweaks proposed for #234187

@elasticmachine
Copy link
Contributor

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!

@ek-so
Copy link
Contributor Author

ek-so commented Sep 9, 2025

Hey @angeles-mb, thanks for a great work on fb snippet 🙏
I think the duration of animation is good!
However, I noticed that button sizes were too large in some cases (fb buttons in snippet itself and in the menu later).
And also after looking at it live, I guess we better add a divider to the button that is placed inside the navigation panel too (I had some doubts about it before).
Lmk please what you think, and please feel free to implement it differently in your PR

angeles-mb added a commit that referenced this pull request Sep 18, 2025
Closes #234863

## Summary

- Added feedback snippet to our Storybook with custom controls to test
props
- Integrated some UI tweaks suggested by @ek-so
#234499
  - A top divider for the feedback button
  - Smaller buttons

## Testing

Tested divider on scrollable submenus:

<img width="743" height="1051" alt="Screenshot 2025-09-16 at 12 05 47"
src="https://github.com/user-attachments/assets/75589637-9adc-4bfb-99a1-ccccf50b00e9"
/>

Tested divider on non-scrollable submenus:

<img width="743" height="1043" alt="Screenshot 2025-09-16 at 13 06 05"
src="https://github.com/user-attachments/assets/69ba7b47-b3e0-4624-aa1a-0d1ea7f32e6b"
/>

Storybook looks:

<img width="895" height="1019" alt="Screenshot 2025-09-16 at 12 07 13"
src="https://github.com/user-attachments/assets/dba37ae4-9055-4b6b-8494-870c35f81890"
/>

---------

Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Sep 24, 2025
Closes elastic#234863

## Summary

- Added feedback snippet to our Storybook with custom controls to test
props
- Integrated some UI tweaks suggested by @ek-so
elastic#234499
  - A top divider for the feedback button
  - Smaller buttons

## Testing

Tested divider on scrollable submenus:

<img width="743" height="1051" alt="Screenshot 2025-09-16 at 12 05 47"
src="https://github.com/user-attachments/assets/75589637-9adc-4bfb-99a1-ccccf50b00e9"
/>

Tested divider on non-scrollable submenus:

<img width="743" height="1043" alt="Screenshot 2025-09-16 at 13 06 05"
src="https://github.com/user-attachments/assets/69ba7b47-b3e0-4624-aa1a-0d1ea7f32e6b"
/>

Storybook looks:

<img width="895" height="1019" alt="Screenshot 2025-09-16 at 12 07 13"
src="https://github.com/user-attachments/assets/dba37ae4-9055-4b6b-8494-870c35f81890"
/>

---------

Co-authored-by: kibanamachine <[email protected]>
niros1 pushed a commit that referenced this pull request Sep 30, 2025
Closes #234863

## Summary

- Added feedback snippet to our Storybook with custom controls to test
props
- Integrated some UI tweaks suggested by @ek-so
#234499
  - A top divider for the feedback button
  - Smaller buttons

## Testing

Tested divider on scrollable submenus:

<img width="743" height="1051" alt="Screenshot 2025-09-16 at 12 05 47"
src="https://github.com/user-attachments/assets/75589637-9adc-4bfb-99a1-ccccf50b00e9"
/>

Tested divider on non-scrollable submenus:

<img width="743" height="1043" alt="Screenshot 2025-09-16 at 13 06 05"
src="https://github.com/user-attachments/assets/69ba7b47-b3e0-4624-aa1a-0d1ea7f32e6b"
/>

Storybook looks:

<img width="895" height="1019" alt="Screenshot 2025-09-16 at 12 07 13"
src="https://github.com/user-attachments/assets/dba37ae4-9055-4b6b-8494-870c35f81890"
/>

---------

Co-authored-by: kibanamachine <[email protected]>
@ek-so ek-so closed this Oct 6, 2025
@ek-so ek-so deleted the ekso-fb-snippet branch October 6, 2025 09:57
rylnd pushed a commit to rylnd/kibana that referenced this pull request Oct 17, 2025
Closes elastic#234863

## Summary

- Added feedback snippet to our Storybook with custom controls to test
props
- Integrated some UI tweaks suggested by @ek-so
elastic#234499
  - A top divider for the feedback button
  - Smaller buttons

## Testing

Tested divider on scrollable submenus:

<img width="743" height="1051" alt="Screenshot 2025-09-16 at 12 05 47"
src="https://github.com/user-attachments/assets/75589637-9adc-4bfb-99a1-ccccf50b00e9"
/>

Tested divider on non-scrollable submenus:

<img width="743" height="1043" alt="Screenshot 2025-09-16 at 13 06 05"
src="https://github.com/user-attachments/assets/69ba7b47-b3e0-4624-aa1a-0d1ea7f32e6b"
/>

Storybook looks:

<img width="895" height="1019" alt="Screenshot 2025-09-16 at 12 07 13"
src="https://github.com/user-attachments/assets/dba37ae4-9055-4b6b-8494-870c35f81890"
/>

---------

Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants