Skip to content

Pullquote block having design issue when text-decoration is choosen strikethrough#66707

Merged
jorgefilipecosta merged 1 commit into
WordPress:trunkfrom
imrraaj:fix/pullquote-block-line-through
Nov 29, 2024
Merged

Pullquote block having design issue when text-decoration is choosen strikethrough#66707
jorgefilipecosta merged 1 commit into
WordPress:trunkfrom
imrraaj:fix/pullquote-block-line-through

Conversation

@imrraaj

@imrraaj imrraaj commented Nov 4, 2024

Copy link
Copy Markdown
Contributor

What?

Pullquote block's cite tag has design issues when the block editor selects the text decoration. Block editor adds display block property so text decoration line-through works fine in editor and only breaks on frontend.

Why?

Fixes: #66533

How?

PR adds a display property on cite tag to fix the issue.

Testing Instructions

  1. Take Pullquote block.
  2. Add some text
  3. Choose text-ecoration line-through;
  4. Now check in editor and front both side.

Screenshots or screencast

Before

Screenshot 2024-11-04 at 1 46 21 PM

After

Screenshot 2024-11-04 at 1 46 49 PM

@imrraaj imrraaj requested a review from ajitbohra as a code owner November 4, 2024 08:18
@github-actions

github-actions Bot commented Nov 4, 2024

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: imrraaj <imrraaj@git.wordpress.org>
Co-authored-by: NidhiDhandhukiya74 <nidhidhandhukiya@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@carolinan carolinan added [Type] Bug An existing feature does not function as intended [Block] Pullquote Affects the Pullquote Block labels Nov 4, 2024

@jorgefilipecosta jorgefilipecosta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @imrraaj, thank you for your contribution. This change fixed the issue in the tests I did.

@jorgefilipecosta jorgefilipecosta merged commit 11262ac into WordPress:trunk Nov 29, 2024
@github-actions github-actions Bot added this to the Gutenberg 19.9 milestone Nov 29, 2024
im3dabasia pushed a commit to im3dabasia/gutenberg that referenced this pull request Dec 4, 2024
…trikethrough (WordPress#66707)

Co-authored-by: imrraaj <imrraaj@git.wordpress.org>
Co-authored-by: NidhiDhandhukiya74 <nidhidhandhukiya@git.wordpress.org>
Co-authored-by: jorgefilipecosta <jorgefilipecosta@git.wordpress.org>
michalczaplinski pushed a commit that referenced this pull request Dec 5, 2024
…trikethrough (#66707)

Co-authored-by: imrraaj <imrraaj@git.wordpress.org>
Co-authored-by: NidhiDhandhukiya74 <nidhidhandhukiya@git.wordpress.org>
Co-authored-by: jorgefilipecosta <jorgefilipecosta@git.wordpress.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Block] Pullquote Affects the Pullquote Block [Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pullquote block having design issue when text-decoration is choosen strikethrough.

3 participants