Skip to content

All: Drop $.ui.escapeSelector in favor of $.escapeSelector #1957

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 14, 2021

Conversation

mgol
Copy link
Member

@mgol mgol commented May 12, 2021

Fixes #14991

@mgol mgol added this to the 1.13 milestone May 12, 2021
@mgol mgol requested a review from fnagel May 12, 2021 21:18
@mgol mgol self-assigned this May 12, 2021
@mgol mgol force-pushed the escape-selector branch from 54807a4 to 6df9b67 Compare May 12, 2021 21:23
Copy link
Member

@fnagel fnagel left a comment

Choose a reason for hiding this comment

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

+1 by reading

@mgol mgol merged commit 7c6a9f0 into jquery:main May 14, 2021
@mgol mgol deleted the escape-selector branch May 14, 2021 21:37
tagliala added a commit to tagliala/activeadmin that referenced this pull request May 9, 2024
This dependency has been manually updated.

`escape-selector` has been removed because jQuery UI is now using
jQuery's built-in escape selector (available from version 3.0, which
is required by Active Admin)

- `jquery-ui-rails` is unmaintained and it is bundling 1.13.0
- An active fork is bundling 1.13.2, but there has been a more recent
  release and I've preferred to manually update

Ref activeadmin#8327

Ref:
- https://bugs.jqueryui.com/ticket/14991/
- jquery/jquery-ui#1957
tagliala added a commit to tagliala/activeadmin that referenced this pull request May 14, 2024
This dependency has been manually updated.

`escape-selector` has been removed because jQuery UI is now using
jQuery's built-in escape selector (available from version 3.0, which
is required by Active Admin)

- `jquery-ui-rails` is unmaintained and it is bundling 1.13.0
- An active fork is bundling 1.13.2, but there has been a more recent
  release and I've preferred to manually update

Ref activeadmin#8327

Ref:
- https://bugs.jqueryui.com/ticket/14991/
- jquery/jquery-ui#1957
tagliala added a commit to tagliala/activeadmin that referenced this pull request May 14, 2024
JavaScript files have been manually updated because:
- `jquery-ui-rails` is unmaintained and it is bundling 1.13.0
- An active fork of `jquery-ui-rails` is bundling 1.13.2, but there is
  a more recent version (1.13.3)
 
`escape-selector` has been removed because jQuery UI is using
jQuery's built-in escape selector. This selector is available from
version 3.0, which is required by Active Admin, so it should not be
a problem

Ref activeadmin#8327

Ref:
- https://bugs.jqueryui.com/ticket/14991/
- jquery/jquery-ui#1957
tagliala added a commit to tagliala/activeadmin that referenced this pull request May 14, 2024
JavaScript files have been manually updated because:
- `jquery-ui-rails` is unmaintained and it is bundling 1.13.0
- An active fork of `jquery-ui-rails` is bundling 1.13.2, but there is
  a more recent version (1.13.3)
 
`escape-selector` has been removed because jQuery UI is using
jQuery's built-in escape selector. This selector is available from
version 3.0, which is required by Active Admin, so it should not be
a problem

Ref activeadmin#8327

Ref:
- https://bugs.jqueryui.com/ticket/14991/
- jquery/jquery-ui#1957
tagliala added a commit to activeadmin/activeadmin that referenced this pull request May 15, 2024
Update vendored jquery-ui

JavaScript files have been manually updated because:
- `jquery-ui-rails` is unmaintained and it is bundling 1.13.0
- An active fork of `jquery-ui-rails` is bundling 1.13.2, but there is
  a more recent version (1.13.3)
 
`escape-selector` has been removed because jQuery UI is using
jQuery's built-in escape selector. This selector is available from
version 3.0, which is required by Active Admin, so it should not be
a problem

Ref #8327

Ref:
- https://bugs.jqueryui.com/ticket/14991/
- jquery/jquery-ui#1957
cmanetta pushed a commit to bugcrowd/activeadmin that referenced this pull request Jun 3, 2024
Update vendored jquery-ui

JavaScript files have been manually updated because:
- `jquery-ui-rails` is unmaintained and it is bundling 1.13.0
- An active fork of `jquery-ui-rails` is bundling 1.13.2, but there is
  a more recent version (1.13.3)
 
`escape-selector` has been removed because jQuery UI is using
jQuery's built-in escape selector. This selector is available from
version 3.0, which is required by Active Admin, so it should not be
a problem

Ref activeadmin#8327

Ref:
- https://bugs.jqueryui.com/ticket/14991/
- jquery/jquery-ui#1957
4tolexx pushed a commit to velaluqa/activeadmin that referenced this pull request Jul 6, 2024
Update vendored jquery-ui

JavaScript files have been manually updated because:
- `jquery-ui-rails` is unmaintained and it is bundling 1.13.0
- An active fork of `jquery-ui-rails` is bundling 1.13.2, but there is
  a more recent version (1.13.3)

`escape-selector` has been removed because jQuery UI is using
jQuery's built-in escape selector. This selector is available from
version 3.0, which is required by Active Admin, so it should not be
a problem

Ref activeadmin#8327

Ref:
- https://bugs.jqueryui.com/ticket/14991/
- jquery/jquery-ui#1957
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants