Skip to content

Update JQuery to 4.0.0 - #1420

Merged
ByteHamster merged 14 commits into
sabre-io:masterfrom
dertuxmalwieder:patch-1
Aug 3, 2026
Merged

ByteHamster merged 14 commits into
sabre-io:masterfrom
dertuxmalwieder:patch-1

Conversation

@dertuxmalwieder

Copy link
Copy Markdown
Contributor

Bootstrap 5 kicked older browsers in the bin, so we have no reason to support them with an older JQuery version anymore either.

@ByteHamster

Copy link
Copy Markdown
Member

Can we get this through composer as well?

@dertuxmalwieder

dertuxmalwieder commented Jul 28, 2026 •

Copy link
Copy Markdown
Contributor Author

Perhaps (components/jquery), but we still need to place jquery-color somewhere, and that's not on Composer as far as I can see.

(Do we actually use jquery-color?) edit: yes we do, but I wonder whether we could have that without another dependency...

@ByteHamster

Copy link
Copy Markdown
Member

(Do we actually use jquery-color?) edit: yes we do, but I wonder whether we could have that without another dependency...

I must admit I don't have an overview there. If it is not really needed, I'm open to removing it :)

@dertuxmalwieder

Copy link
Copy Markdown
Contributor Author

The only place where it seems to be used is the copyToClipboard function. JQuery Color "adds" the possibility to animate() background colours.

(FWIW, that could be updated as well.)

@ByteHamster

Copy link
Copy Markdown
Member

Ah, I think we can probably rebuild this using CSS transitions.

@dertuxmalwieder

Copy link
Copy Markdown
Contributor Author

Do you need me for that? I'm somewhat busy this week, I'm afraid.

@ByteHamster

Copy link
Copy Markdown
Member

No worries, I probably won't have time for it either in the next couple of days

@dertuxmalwieder dertuxmalwieder mentioned this pull request Jul 29, 2026
4 tasks done
Fixed paths, removed reference to jquery color
got rid of the jquery color reference
This should do?
@dertuxmalwieder

Copy link
Copy Markdown
Contributor Author

Updated JQuery to use the Composer., reworked copytoclipboard, left a todo for jquery color (I tried yesterday before bed, but I failed)

@ByteHamster
ByteHamster merged commit 8879886 into sabre-io:master Aug 3, 2026
10 checks passed
@dertuxmalwieder
dertuxmalwieder deleted the patch-1 branch August 3, 2026 07:44
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.

2 participants