Skip to content

Changed slideover to popover and dialog - #46

Open
jordythevulder wants to merge 12 commits into
masterfrom
feature/popover-slideover
Open

Changed slideover to popover and dialog#46
jordythevulder wants to merge 12 commits into
masterfrom
feature/popover-slideover

Conversation

@jordythevulder

@jordythevulder jordythevulder commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

The new slideovers are here. Fully made with the popover and dialog functionality. No more magic and full control over each element.

Notes

  • This is a breaking change!
  • All chrome animations are working
  • All safari animations are working except ::backdrop fade out animation (not disturbing)
  • All firefox animations are working except all fade out animations (not disturbing)
  • When browsers add more compatibility in the future, they will automatically work
  • Nested slideovers are also supported
  • Internal REF: RAP-1974

Options

  • Position left or right (defaults to left)

Known issues

  • Nested dialog slideovers can't close parent dialogs without extra JS (Popovers can)

Variants

  • Slideover mobile (Show the content on desktop, hide content within slideover on mobile)

Questions

  • Can the label support be dropped within the resources/views/components/button/tag.blade.php?
  • Do i need to redirect my PR to a different 3.0 branch?
  • Should the slideover header be text-centered just like the old slideover? > i did change it

Usage
image

Slideover mobile usage
image

Nested usage
image

New
image

Old
image

@jordythevulder jordythevulder changed the title RAP-1974 - Changed slideover to popover Changed slideover to popover Jul 23, 2026
@Roene-JustBetter

Copy link
Copy Markdown
Member

Looking good 🚀

Can the label support be dropped within the resources/views/components/button/tag.blade.php?

I think we should keep this support maybe in some cases we still want a button as a label.

Do i need to redirect my PR to a different 3.0 branch?

Breaking change so I think this should be a 3.0 release.

Should the slideover header be text-centered just like the old slideover?

I don't think it is necessary as we discussed this f2f.

@jordythevulder

Copy link
Copy Markdown
Contributor Author

Edit:

I've added dialog aswell. Also i did change the text-center within the slideover header. Makes it easier to update and takes less time when project use default styling.
image

@jordythevulder jordythevulder changed the title Changed slideover to popover Changed slideover to popover and dialog Jul 27, 2026
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