Make WordPress Core

Opened 6 weeks ago

Last modified 4 weeks ago

#62637 reviewing defect (bug)

Misplaced filter themes wrapper (under Customize > Change Theme > .org themes browser)

Reported by: kharisblank's profile kharisblank Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.8 Priority: normal
Severity: normal Version:
Component: Customize Keywords: has-screenshots has-patch has-testing-info
Focuses: ui, css, administration Cc:

Description

Steps to reproduce the issue:

  1. Go to Customize
  2. Click on Change theme button
  3. Switch to WordPress.org themes

https://cldup.com/SKm44fh4kJ.png

Screen recording: https://cldup.com/tiBD_mLB5Q.mov

Attachments (3)

after-7942.png (798.2 KB) - added by parthvataliya 6 weeks ago.
after-patch.png (917.5 KB) - added by praful2111 4 weeks ago.
Customize_ WordPress Develop.webm (2.3 MB) - added by ruchirj 4 weeks ago.

Change History (19)

#1 @ankitkumarshah
6 weeks ago

Reproduction Report

Description

This report validates whether the issue can be reproduced.

Environment

  • WordPress: 6.7.1
  • PHP: 8.1.29
  • Server: nginx/1.16.0
  • Database: mysqli (Server: 8.0.16 / Client: mysqlnd 8.1.29)
  • Browser: Chrome 131.0.0.0
  • OS: macOS
  • Theme: Twenty Seventeen 3.8
  • MU Plugins: None activated

Actual Results

  1. ✅ Error condition occurs (reproduced).

Supplemental Artifacts

https://i.postimg.cc/kGpJdPTJ/Screenshot-2024-12-03-at-6-28-05-PM.png

#2 @benazeer
6 weeks ago

Reproduction Report

Description

This report validates whether the issue can be reproduced.

Environment

  • WordPress: 6.7.1
  • PHP: 8.1.22
  • Server: nginx/1.16.0
  • Database: mysqli (Server: 8.0.16 / Client: mysqlnd 8.1.22)
  • Browser: Chrome 131.0.0.0
  • OS: macOS
  • Theme: Twenty Seventeen 3.8
  • MU Plugins: None activated
  • Plugins:
    • Gutenberg 19.7.0-rc.1
    • Test Reports 1.2.0

Actual Results

  1. ✅ Error condition occurs (reproduced).

Screenshot:
https://postimg.cc/Y4yM7TCz
https://postimg.cc/Y4yM7TCz

Last edited 6 weeks ago by benazeer (previous) (diff)

This ticket was mentioned in PR #7939 on WordPress/wordpress-develop by @ankitkumarshah.


6 weeks ago
#3

  • Keywords has-patch added

Trac ticket: #62637

## Description
This PR addresses an issue where the "Filter themes" wrapper element was misplaced within the "Customize > Change Theme > .org themes browser" section of the application.

#4 @sainathpoojary
6 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://github.com/WordPress/wordpress-develop/pull/7939

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.26
  • Server: nginx/1.27.3
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.26)
  • Browser: Chrome 131.0.0.0
  • OS: macOS
  • Theme: Twenty Twenty-One 2.4
  • MU Plugins: None activated
  • Plugins: Test Reports 1.2.0

Actual Results

✅ Issue resolved with patch.

Supplemental Artifacts

Before:

https://utfs.io/f/PL8E4NiPUWyOSF6nzNLiQ4fwdzrJyI7ZFLWB3XOnMKUARmPD

After:

https://utfs.io/f/PL8E4NiPUWyO21fkheMrbSwhDZXGLqs4KMo8zvf5xlc7yBV1

#7 @mukesh27
6 weeks ago

  • Keywords has-testing-info added; 2nd-opinion removed
  • Milestone changed from Awaiting Review to 6.8
  • Version 6.7.1 deleted

Thanks for ticket.

I also reproduce the issue and open PR https://github.com/WordPress/wordpress-develop/pull/7942 that update the class for wrapper DIV.

Take a look and share your feedback.

Moving to 6.8 for visibility.

@ankitkumarshah commented on PR #7942:


6 weeks ago
#8

Hi @mukeshpanchal27,
I tested the changes and really like the alignment of the search bar and filter button. According to my understanding, I believe that if we remove the search-form class, it would be beneficial to also eliminate the corresponding class names from the CSS to maintain cleanliness and avoid redundancy. For example:

.customize-preview-header.themes-filter-bar .search-form{}

.customize-preview-header.themes-filter-bar .search-form
	.themes-filter-bar .themes-filter-container{}

Removing these class names from the CSS would help streamline the code since they are no longer needed. Please let me know if my understanding is correct.

@kharisblank commented on PR #7942:


6 weeks ago
#9

Hi @mukeshpanchal27,

I tested out the patch, and it looks really great.

https://github.com/user-attachments/assets/be2a2bc5-9219-430e-bc73-81ef6de08bbe

#10 @parthvataliya
6 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/7942.diff

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.26
  • Server: nginx/1.27.3
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.26)
  • Browser: Chrome 131.0.0.0
  • OS: Linux
  • Theme: Twenty Eleven 4.8
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Issue resolved with patch.

Additional Notes

  • I tested the PR opened by @mukesh27 , and it works fine with all the functionality. Adding the filter option in this place is ideal.
Last edited 6 weeks ago by parthvataliya (previous) (diff)

#11 @im3dabasia1
6 weeks ago

Reproduction Report

Description

This report validates whether the issue can be reproduced.

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.25
  • Server: nginx/1.27.2
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.25)
  • Browser: Chrome 129.0.0.0
  • OS: macOS
  • Theme: Twenty Nineteen 3.0
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Error condition occurs (reproduced).

Supplemental Artifacts

https://i.postimg.cc/QN0gGVQc/Screenshot-2024-12-04-at-7-06-04-PM.png

Last edited 6 weeks ago by im3dabasia1 (previous) (diff)

#12 @im3dabasia1
6 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://github.com/WordPress/wordpress-develop/pull/7942.diff

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.25
  • Server: nginx/1.27.2
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.25)
  • Browser: Chrome 129.0.0.0
  • OS: macOS
  • Theme: Twenty Nineteen 3.0
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Issue resolved with patch.

Supplemental Artifacts

https://i.postimg.cc/cH6xvDzx/Screenshot-2024-12-04-at-7-10-59-PM.png

#13 @sukhendu2002
6 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/7942.diff

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.26
  • Server: nginx/1.27.3
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.26)
  • Browser: Chrome 131.0.0.0
  • OS: macOS
  • Theme: Twenty Nineteen 3.0
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Issue resolved with patch.

#14 @SergeyBiryukov
6 weeks ago

  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#15 @praful2111
4 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/7942.diff

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.26
  • Server: nginx/1.27.3
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.26)
  • Browser: Chrome 131.0.0.0
  • OS: Linux
  • Theme: Twenty Twenty 2.8
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Issue resolved with patch.

#16 @ruchirj
4 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/7942.diff

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.26
  • Server: nginx/1.27.3
  • Database: mysqli (Server: 8.4.2 / Client: mysqlnd 8.2.26)
  • Browser: Chrome 131.0.0.0
  • OS: Linux
  • Theme: Twenty Thirteen 4.3
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Issue resolved with patch.
Note: See TracTickets for help on using tickets.