Fixes #37402 - add jquery-ui dependency

[MariaAga]

we want to remove it in core since its no longer used theforeman/foreman#10142
so adding it here as only Katello seem to use it, and hopefully it will be removed here as well

[ekohl]

Looks like an OK stopgap to me. Did you have a look at how many work it would be to actually drop it from Katello too?

[MariaAga]

yeah its used in some parts of the sync management (and maybe more, I didn't look after that). and I dont have capacity now to try to remove it from there since its spread through a few files

[ekohl]

Sounds good to me: the Katello team has plans to remove those pages anyway. It may be a waste of time trying to rework them just to drop jquery-ui while they're fully replaced soon after.

[jeremylenz]

Sync management is special, though, since it's actually ERB + vanilla JS + jQuery and doesn't have any Angular in it 😳

[MariaAga]

@jeremylenz Can we move jquery-ui to katello anyway? and makes plans to maybe refactor sync management later?

[jeremylenz]

@MariaAga If we're the only ones that use it, I'm fine with moving it to Katello for now. And yes we should also talk about redesigning sync status :)

[sjha4]

Wondering if this: https://bugzilla.redhat.com/show_bug.cgi?id=2274132 has anything to do with this change here and if it fixes this.

[jeremylenz]

@sjha4 The Foreman PR dropping it is not yet merged, so my guess is it's something else 🤷