-
Notifications
You must be signed in to change notification settings - Fork 117
#124 - Project Details Block Pattern #189
#124 - Project Details Block Pattern #189
Conversation
The markup for this pattern was already provided by Bea but since it wasn't on a template I didn't include it in the original PR. Can you please check your markup against hers here to check that it all aligns? |
@MaggieCabrera I've updated the pattern to the provided markup. Let me know if that works. |
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.
Here are a few of my quick findings.
- Please use the spacing presets instead of arbitrary values for paddings where needed.
- Use
esc_html_x
with context instead ofesc_html
. You can check404.php
or other approved patterns for the reference.
patterns/project-details.php
Outdated
|
||
?> | ||
<!-- wp:group {"align":"full","style":{"spacing":{"padding":{"top":"8vh","bottom":"8vh","right":"6vw","left":"6vw"}}},"backgroundColor":"base","layout":{"type":"constrained"}} --> | ||
<div class="wp-block-group alignfull has-base-background-color has-background" style="padding-top:8vh;padding-right:6vw;padding-bottom:8vh;padding-left:6vw"><!-- wp:columns {"align":"wide","style":{"spacing":{"blockGap":{"top":"2em","left":"2em"}}}} --> |
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.
can we use presets instead here for the padding?
<!-- wp:group {"align":"full","style":{"spacing":{"padding":{"top":"8vh","bottom":"8vh","right":"6vw","left":"6vw"}}},"backgroundColor":"base","layout":{"type":"constrained"}} --> | ||
<div class="wp-block-group alignfull has-base-background-color has-background" style="padding-top:8vh;padding-right:6vw;padding-bottom:8vh;padding-left:6vw"><!-- wp:columns {"align":"wide","style":{"spacing":{"blockGap":{"top":"2em","left":"2em"}}}} --> | ||
<div class="wp-block-columns alignwide"><!-- wp:column {"width":"15%"} --> | ||
<div class="wp-block-column" style="flex-basis:15%"><!-- wp:paragraph {"style":{"typography":{"fontSize":"0.8rem","lineHeight":"1.4"}}} --> |
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.
same about font size presets here
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.
I changed the spacing values, let's bring this in and iterate
Description
Screenshots
![image](https://cdn.statically.io/img/private-user-images.githubusercontent.com/1948204/265520345-80c436ef-57a3-4f7b-a0ac-78316cd9ccca.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE5OTA4MjMsIm5iZiI6MTcyMTk5MDUyMywicGF0aCI6Ii8xOTQ4MjA0LzI2NTUyMDM0NS04MGM0MzZlZi01N2EzLTRmN2ItYTBhYy03ODMxNmNkOWNjY2EucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcyNiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MjZUMTA0MjAzWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9NDg2ZjdhMmVkODJiYWRmZDE4MmYyMDhhMzc5NTVkZTg3YmJmM2NmNzM3ZTkzNDk3ZDIyODZiMmFiODQyMjE4NSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ._w23xVJ1zvxkmiHxHLh5Smwnr_kBiQpv69nyS5yjDrU)
Desktop
Mobile
![image](https://cdn.statically.io/img/private-user-images.githubusercontent.com/1948204/265520773-6e4d74ce-71a4-4b43-bf89-130d579e2402.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE5OTA4MjMsIm5iZiI6MTcyMTk5MDUyMywicGF0aCI6Ii8xOTQ4MjA0LzI2NTUyMDc3My02ZTRkNzRjZS03MWE0LTRiNDMtYmY4OS0xMzBkNTc5ZTI0MDIucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcyNiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MjZUMTA0MjAzWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9Y2UxYmRjMzI3ZDNlOGE3NzA1ZDJlM2M2OGZkMWFhMGQyNjUxYjAyYmU1OTA1ZjM0NjJiZDEzNjFiNjk4ZjQ4NyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.6R5DGrCufQVjbl-tkM60p0Jl337J2Qu3WO91dXOycpY)
Testing Instructions
Contributors
khleomix