Switch to border-box as default box-model (box-sizing)

Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,367 pass
Comment about 1 year ago →
🇮🇳India Gauravvv Delhi, India
I have attached the patch, please review
Status changed to RTBC about 1 year ago10:18pm 1 May 2023
Comment about 1 year ago →
🇺🇸United States smustgrave
Changes seems simple enough.
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,374 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,378 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,379 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,380 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,383 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,388 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,388 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,388 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
29,387 pass, 2 fail
Comment about 1 year ago →
System Message
The last submitted patch, 10: 3192130-10.patch, failed testing. View results →
Status changed to Needs review 8 months ago8:25am 12 October 2023
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 8 months ago
30,341 pass
Comment 8 months ago →
🇮🇳India swatidhurandhar
I have created a new patch since #11 was failing through some test cases.
Status changed to RTBC 8 months ago1:15pm 12 October 2023
Comment 8 months ago →
🇮🇳India viren18febS
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 8 months ago
30,394 pass
Comment 8 months ago →
🇺🇸United States smustgrave
11.x is the current development branch and #10 still applies
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 8 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 8 months ago
30,327 pass, 2 fail
Comment 8 months ago →
System Message
The last submitted patch, 13: 3192130-13.patch, failed testing. View results →
Comment 8 months ago →
🇮🇳India Gauravvv Delhi, India
Random failure, restoring status
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 8 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 8 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
30,341 pass
Status changed to Needs work 7 months ago11:44pm 10 November 2023
Comment 7 months ago →
needs-review-queue-bot
The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
First commit to issue fork.
@ameymudras opened merge request.
Status changed to RTBC 7 months ago3:41am 14 November 2023
Comment 7 months ago →
🇫🇷France nod_ Lille
restoring status, credited for target version update
Comment 7 months ago →
🇫🇷France nod_ Lille
I do not understand what this changes practically.

The IS mentions that it causes unnecessary calculations but there is no simplifications in the code at all. I'm tempted to close this issue as "works as designed" if we don't have any other change in this MR.
Status changed to Needs work 7 months ago3:46am 14 November 2023
Comment 7 months ago →
🇫🇷France nod_ Lille
Status changed to Closed: works as designed 6 months ago6:24pm 14 December 2023
Comment 6 months ago →
🇺🇸United States smustgrave
Per #22 since there hasn't been any follow up.

Switch to border-box as default box-model (box-sizing)

Problem/Motivation

Proposed resolution

Comments & Activities