Opened 2 months ago
Closed 26 hours ago
#62380 closed defect (bug) (fixed)
About page: 'See everything new' link can extend outside its container
Reported by: | franciscabusas22 | Owned by: | ryelle |
---|---|---|---|
Milestone: | 6.7.2 | Priority: | normal |
Severity: | minor | Version: | 6.6 |
Component: | Help/About | Keywords: | has-patch has-testing-info has-screenshots fixed-major dev-reviewed |
Focuses: | ui, css | Cc: |
Description
Mobile View Issue: The button is overlapping in parent container in Brave browser.
Attachments (3)
Change History (24)
#2
@
2 months ago
- Component changed from General to Help/About
- Milestone changed from Awaiting Review to 6.7.1
- Version changed from 6.6.2 to 6.6
#3
@
8 weeks ago
- Summary changed from Mobile Responsive: Overlapping Button in WordPress Dashboard to About page: 'See everything new' link can extend outside its container
#4
@
8 weeks ago
The default English 'hero' link is 280 pixels wide (with Segoe UI in Windows). Other languages can be even wider:
- Tamil (573px) எல்லாவற்றையும் புதிதாகப் பாருங்கள்
- Malayalam (461px) എല്ലാം പുതിയതായി കാണുക
- French (422px) Découvrez toutes les nouveautés
This ticket was mentioned in PR #7769 on WordPress/wordpress-develop by @yogeshbhutkar.
8 weeks ago
#5
- Keywords has-patch added
This ticket was mentioned in PR #7771 on WordPress/wordpress-develop by @sainathpoojary.
8 weeks ago
#6
This PR addresses the issue of button text overlapping in mobile view About page. The following responsive styles have been added for a viewport width of 782px or less:
- Set text-wrap: balance for optimal text distribution
- Adjust line-height to 1.7
- Configure dynamic padding using --gap variable
- Update max-width properties
Preview after fixing:
Trac ticket: #62380
#7
@
8 weeks ago
I've added a patch fixing the link on mobile and tablet devices and attached screenshots of it in the PR. It's been tested for all the locales mentioned above and is found to render the link appropriately.
This ticket was mentioned in Slack in #core by desrosj. View the logs.
8 weeks ago
#10
@
8 weeks ago
- Milestone changed from 6.7.1 to 6.7.2
- Severity changed from normal to minor
Because of a few bug reports opened since 6.7 was released, the Core team is evaluating the need for a short 6.7.1 cycle (possibly next week).
To help prepare for this scenario in case it's decided to move forward, I'm going to punt this to the 6.7.2 milestone. This issue was not introduced in 6.7, so it now falls outside of the focus for 6.7.1 as currently defined.
#11
@
7 weeks ago
- Keywords has-testing-info has-screenshots added
Test Report
Description
This report validates whether the indicated patch works as expected.
Patch tested: https://github.com/WordPress/wordpress-develop/pull/7769.diff
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 129.0.0.0
- OS: macOS
- Theme: Twenty Twenty-Five 1.0
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
- WordPress Beta Tester 3.6.1
Actual Results
- ✅ Issue resolved with patch.
Additional Notes
- Tested the patch in Safari, Chrome and Brave
Supplemental Artifacts
Brave: https://postimg.cc/gnzkjvDs
Safari: https://postimg.cc/RWqSQ42n
Chrome: https://postimg.cc/56s9C6KV
This ticket was mentioned in Slack in #core by jorbin. View the logs.
3 weeks ago
This ticket was mentioned in PR #8027 on WordPress/wordpress-develop by @ryelle.
3 weeks ago
#13
On some screen sizes and languages, the "See everything new" button expands out of the content area. This change allows the button to wrap at all screen sizes.
This pulls CSS from both #7769 and #7771, the changes I made were to fix some padding, and apply to all screen sizes.
Trac ticket: https://core.trac.wordpress.org/ticket/62380
Screenshots
1200px | 800px | 400px | |
---|---|---|---|
English | |||
French | |||
Japanese | |||
Uighur (RTL) |
@audrasjb commented on PR #8027:
3 weeks ago
#15
Thanks @ryelle I can confirm this changeset solves the issue in fr_FR 👍
5e479dce-183e-4086-aaff-a6294403d8d3.webm
#17
@
27 hours ago
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening to backport [59580] to the 6.7 branch.
#18
@
27 hours ago
- Keywords dev-feedback added
Adding dev-feedback
for the backport per these instructions.
#62379 was marked as a duplicate.