Skip to content
This repository was archived by the owner on Jan 24, 2024. It is now read-only.

Fixed Pulloquote Block border issue#550

Merged
luminuu merged 4 commits intoWordPress:trunkfrom
viralsampat-multidots:fixed_533_pullquote_block_border_issue
Oct 4, 2023
Merged

Fixed Pulloquote Block border issue#550
luminuu merged 4 commits intoWordPress:trunkfrom
viralsampat-multidots:fixed_533_pullquote_block_border_issue

Conversation

@viralsampat-multidots
Copy link
Copy Markdown
Contributor

Description

Hello Team,

I have worked on issue #533 and resolved it from my end. Here, I have added my PR.

Screenshots

Back-end
backend

Front-end
frontend

Testing Instructions

  • Go to page Type / to choose a block
  • Select "Pullquote" block
  • Add dummy text
  • Click on the block setting and try to set the border.

Contributors

theme.json Outdated
"bottom": {
"style": "none",
"width": "0px"
"style": "solid",
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.

In order to make the borders work again and keep the layout as in the design, the bottom, left, right and top attributes should be completely removed. This way there's no border when using the block as default, and borders can be added if the block settings are updated.

Please remove the mentioned attributes, but keep the radius attribute.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hello @luminuu

I have added my new PR.

Please review it and let me know your feedback for the same.

Thanks,

theme.json Outdated
"style": "none",
"width": "0px"
}
"style": "solid",
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.

This does not need to be added, as the default design of the pullquote block does not have a border. Please remove these two lines.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @luminuu

Done,

Please check it

Thanks,

theme.json Outdated
"style": "none",
"width": "0px"
}
"style": "solid"
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.

Please remove the "style": "solid" too, as it creates a border that is not in the design:

CleanShot 2023-10-04 at 13 07 38@2x

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @luminuu

I have updated my PR and removed the above-mentioned line.

Please review it.

Thanks,

Copy link
Copy Markdown
Member

@luminuu luminuu left a comment

Choose a reason for hiding this comment

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

Tested. By default, the pullquote has no border as intended in the design, and borders can be added via the block settings if needed.

@luminuu luminuu merged commit f8c275c into WordPress:trunk Oct 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants