Skip to content
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

Canard: Fix display of dotted separator #7980

Merged
merged 1 commit into from
Jul 31, 2024
Merged

Conversation

dsas
Copy link
Contributor

@dsas dsas commented Jul 30, 2024

The dotted separator wasn't displaying because the height of the separator was too small.

Changes proposed in this Pull Request:

This change overrides the height for the dotted separator only.

Location Before After
Front-end Screenshot 2024-07-30 at 17 35 14 Screenshot 2024-07-30 at 17 34 26
Editor Screenshot 2024-07-30 at 17 35 39 Screenshot 2024-07-30 at 17 34 44

Related issue(s):

#7978

@dsas dsas self-assigned this Jul 30, 2024
Copy link
Contributor

github-actions bot commented Jul 30, 2024

Preview changes

I've detected changes to the following themes in this PR: Canard.

You can preview these changes by following the links below:

I will update this comment with the latest preview links as you push more changes to this PR.
⚠️ Note: The preview sites are created using WordPress Playground. You can add content, edit settings, and test the themes as you would on a real site, but please note that changes are not saved between sessions.

dara/blocks.css Outdated Show resolved Hide resolved
The dotted separator wasn't displaying because the height of the
separator was too small. This change overrides the height for the dotted
separator only.
@dsas dsas force-pushed the try/fix/canard-separator-dots branch from edae2ac to 4d3676d Compare July 31, 2024 10:49
@dsas dsas merged commit a4e2762 into trunk Jul 31, 2024
2 checks passed
@dsas dsas deleted the try/fix/canard-separator-dots branch July 31, 2024 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants