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 | Owned by: | 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:
- Go to Customize
- Click on Change theme button
- Switch to WordPress.org themes
Screen recording: https://cldup.com/tiBD_mLB5Q.mov
Attachments (3)
Change History (19)
#2
@
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
- ✅ Error condition occurs (reproduced).
Screenshot:
https://postimg.cc/Y4yM7TCz
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
@
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:
After:
This ticket was mentioned in PR #7940 on WordPress/wordpress-develop by @benazeer.
6 weeks ago
#5
Trac ticket: https://core.trac.wordpress.org/ticket/62637
This ticket was mentioned in PR #7942 on WordPress/wordpress-develop by @mukesh27.
6 weeks ago
#6
Trac ticket: https://core.trac.wordpress.org/ticket/62637
#7
@
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
#10
@
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
- ✅ 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.
#11
@
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
- ✅ Error condition occurs (reproduced).
Supplemental Artifacts
#12
@
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
- ✅ Issue resolved with patch.
Supplemental Artifacts
#13
@
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
- ✅ Issue resolved with patch.
#15
@
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
- ✅ Issue resolved with patch.
#16
@
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
- ✅ Issue resolved with patch.
Reproduction Report
Description
This report validates whether the issue can be reproduced.
Environment
Actual Results
Supplemental Artifacts