-
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
[ Latest Posts block] Option to always show the [Read more ...] #51213
base: trunk
Are you sure you want to change the base?
[ Latest Posts block] Option to always show the [Read more ...] #51213
Conversation
👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @ghorivipul97! In case you missed it, we'd love to have you join us in our Slack community, where we hold regularly weekly meetings open to anyone to coordinate with each other. If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ghorivipul97 Just a couple quick suggestions below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed Suggestion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ghorivipul97 Looks like PHP checks are also failing. Left a couple suggested code changes below.
7a535f4
to
3db3e4c
Compare
@alexstine can you help me to solve "puppeteer" error |
@ghorivipul97 Hi. You need to pull the latest changes from the trunk branch and update the pull request, then the tests will run again. Kindly try to solve the PHP errors. |
.trim() | ||
.split( ' ', excerptLength ) | ||
.join( ' ' ) } | ||
{ /* translators: excerpt truncation character, default … */ } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ellipsis is not used anymore so the translators comment should be removed.
I also think that read more link should always be on a separate line, what do you think? And maybe there needs to be some spacing added between the list items. Right now, the link is displayed differently in the editor and the front. |
link #22269
What?
Added Read More Link in Latest Post Block
Why?
#22269
How?
Testing Instructions
Testing Instructions for Keyboard
Screenshots or screencast