-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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.
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
Editor: Link preview overflows with long strings and only shows postname #62414
Comments
Hey @hrkhal, In my opinion, the purpose of the link button in the post publish panel should be to display only the title of the post, rather than the entire permalink. The full link can be accessed through the dialog box that appears upon clicking the link button, making it redundant to display it initially. ![]() |
@Anuj-Rathore24 Appreciate the fix for line wrapping. It is not redundant to reduce the number of clicks to ascertain information that used to be available in 6.4 without the need for any user interaction. |
@Anuj-Rathore24 @hrkhal Ack, I was working on this too and opened a PR. I hope between the two of us it gets sorted 😊 There are some updates to make things a bit clearer about what you can edit and where in the PR I did as I found the description in the edit link dialog a bit imprecise. |
Wanted to note I can replicate this with 6.6.1: Screen.Recording.2024-08-20.at.12.30.09.PM.movAnother one to possibly consider for a point release cc @vcanales |
@annezazu I was unable to reproduce this in 6.6.1. The display of the full URL isn't fit for a point release, as I don't think it can be considered a regression; the URL hasn't been shown since 6.4. The problem in the video looks different from the one reported in this issue and is reproducible in 6.5: url-overflow-publish.movI've added a new issue for this behavior — #64730 |
Regarding the link button in the inspector, it's also discussed in #61196, including whether its label and popover content are appropriate. |
Description
The Link Preview in the unified publishing details overflows when long strings are present in the post name and it does not show the full url path.
Enterprise users would expect to see the full url path in the preview at a glance. Displaying the full path would better align with what was in 6.4 and what our users expect.
Step-by-step reproduction instructions
/%year%/%monthnum%/%day%/%postname%-%post_id%/
/helloworldhelloworldhelloworld
I would expect the link preview to a) wrap long strings and b) display
/2024/06/07/helloworldhelloworldhelloworld-1
Screenshots, screen recording, code snippet
Environment info
Please confirm that you have searched existing issues in the repo.
Yes
Please confirm that you have tested with all plugins deactivated except Gutenberg.
Yes
The text was updated successfully, but these errors were encountered: