🌐 AI搜索 & 代理 主页
Skip to content

CSS: Make the reliableTrDimensions support test work with Bootstrap CSS #5278

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.

Sign up for GitHub

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

Jump to bottom
Merged
mgol merged 1 commit into from
Jul 10, 2023
Merged

CSS: Make the reliableTrDimensions support test work with Bootstrap CSS #5278

mgol merged 1 commit into from
Jul 10, 2023

Conversation

@mgol
Copy link
Member

@mgol mgol commented Jun 26, 2023 •
edited
Loading

Summary

Bootstrap 5 includes the following CSS on the page:

*,
*::before,
*::after {
  box-sizing: border-box;
}

That threw our reliableTrDimensions support test off. This change fixes the support test and adds a unit test ensuring support test values on a page including Bootstrap 5 CSS are the same as on a page without it.

Fixes gh-5270

+10 bytes

3.x version: #5279

Checklist

@mgol mgol added CSS Needs review Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. labels Jun 26, 2023
@mgol mgol added this to the 4.0.0 milestone Jun 26, 2023
@mgol mgol self-assigned this Jun 26, 2023
mgol added a commit to mgol/jquery that referenced this pull request Jun 26, 2023
@mgol
CSS: Make the reliableTrDimensions support test work with Bootstrap CSS
8fea6a7 
Bootstrap 5 includes the following CSS on the page:

```css
*,
*::before,
*::after {
  box-sizing: border-box;
}
```

This threw our `reliableTrDimensions` support test off. This change fixes the
support test and adds a unit test ensuring support test values on a page
including Bootstrap 5 CSS is the same as on a page without it.

Fixes jquerygh-5270
Ref jquerygh-5278
This was referenced Jun 26, 2023
Merged
Closed
mgol added a commit to mgol/jquery that referenced this pull request Jun 26, 2023
@mgol
CSS: Make the reliableTrDimensions support test work with Bootstrap CSS
90d30bd 
Bootstrap 5 includes the following CSS on the page:

```css
*,
*::before,
*::after {
  box-sizing: border-box;
}
```

That threw our `reliableTrDimensions` support test off. This change fixes the
support test and adds a unit test ensuring support test values on a page
including Bootstrap 5 CSS is the same as on a page without it.

Fixes jquerygh-5270
Ref jquerygh-5278
mgol added a commit to mgol/jquery that referenced this pull request Jun 29, 2023
@mgol
CSS: Make the reliableTrDimensions support test work with Bootstrap CSS
b57bd18 
Bootstrap 5 includes the following CSS on the page:

```css
*,
*::before,
*::after {
  box-sizing: border-box;
}
```

That threw our `reliableTrDimensions` support test off. This change fixes the
support test and adds a unit test ensuring support test values on a page
including Bootstrap 5 CSS is the same as on a page without it.

Fixes jquerygh-5270
Ref jquerygh-5278
mgol added a commit to mgol/jquery that referenced this pull request Jul 7, 2023
@mgol
CSS: Make the reliableTrDimensions support test work with Bootstrap CSS
26b3cc2 
Bootstrap 5 includes the following CSS on the page:

```css
*,
*::before,
*::after {
  box-sizing: border-box;
}
```

That threw our `reliableTrDimensions` support test off. This change fixes the
support test and adds a unit test ensuring support test values on a page
including Bootstrap 5 CSS are the same as on a page without it.

Fixes jquerygh-5270
Ref jquerygh-5278
@mgol mgol force-pushed the tr-height-fix branch from 7fbf15f to aad9c0f Compare July 7, 2023 10:16
@timmywil timmywil removed the Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. label Jul 10, 2023
mgol added a commit that referenced this pull request Jul 10, 2023
@mgol
CSS: Make the reliableTrDimensions support test work with Bootstrap C…
a288838 
…SS (3.x version)

Bootstrap 5 includes the following CSS on the page:

```css
*,
*::before,
*::after {
  box-sizing: border-box;
}
```

That threw our `reliableTrDimensions` support test off. This change fixes the
support test and adds a unit test ensuring support test values on a page
including Bootstrap 5 CSS are the same as on a page without it.

Fixes gh-5270
Closes gh-5279
Ref gh-5278
@mgol
CSS: Make the reliableTrDimensions support test work with Bootstrap CSS
60eacd9 
Bootstrap 5 includes the following CSS on the page:

```css
*,
*::before,
*::after {
  box-sizing: border-box;
}
```

That threw our `reliableTrDimensions` support test off. This change fixes the
support test and adds a unit test ensuring support test values on a page
including Bootstrap 5 CSS are the same as on a page without it.

Fixes jquerygh-5270
Ref jquerygh-5279
@mgol mgol removed the Needs review label Jul 10, 2023
@mgol mgol merged commit 65b8503 into jquery:main Jul 10, 2023
@mgol mgol deleted the tr-height-fix branch July 10, 2023 16:33
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@timmywil timmywil timmywil approved these changes

Assignees

@mgol mgol

Labels

CSS

Milestone

4.0.0

Development

Successfully merging this pull request may close these issues.

Breaking change on height(Upgrading from 3.5.1 to 3.6/3.7)

2 participants

@mgol @timmywil