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

New patterns #87

Merged
merged 53 commits into from
Dec 8, 2023
Merged

New patterns #87

merged 53 commits into from
Dec 8, 2023

Conversation

JohnPixle
Copy link
Contributor

Summary

  • Added most of the new patterns as php templates
  • Adjusted/added string variables
  • Adjusted / added pattern categories

All strings and images in these patterns reference the variables we have created for strings.

Will affect visual aspect of the product

NO

Test instructions

  • Please review the pattern PHPs
  • Install the build and in the pattern inserter see if all the patterns appear and are being imported correctly.

Closes #.

@pirate-bot
Copy link
Contributor

Plugin build for de2f4e4 is ready 🛎️!

@pirate-bot
Copy link
Contributor

Plugin build for 6ff6c1d is ready 🛎️!

Had uploaded the pattern in the wrong location: inc/title-background.php
@pirate-bot
Copy link
Contributor

Plugin build for b84d628 is ready 🛎️!

@pirate-bot
Copy link
Contributor

Plugin build for b21ef65 is ready 🛎️!

update pattern with strings and spacing
updated pattern with strings and spacing
@pirate-bot
Copy link
Contributor

Plugin build for 8a6b10c is ready 🛎️!

@pirate-bot
Copy link
Contributor

Plugin build for bc57c52 is ready 🛎️!

update pattern with strings and spacing
@pirate-bot
Copy link
Contributor

Plugin build for 7c56e58 is ready 🛎️!

updated strings and layout
@pirate-bot
Copy link
Contributor

Plugin build for 2bd6511 is ready 🛎️!

@pirate-bot
Copy link
Contributor

Plugin build for 1bef484 is ready 🛎️!

@pirate-bot
Copy link
Contributor

Plugin build for 8f7dac5 is ready 🛎️!

@pirate-bot
Copy link
Contributor

Plugin build for e5b5c80 is ready 🛎️!

fixed typo
@pirate-bot
Copy link
Contributor

Plugin build for 693c22e is ready 🛎️!

@pirate-bot
Copy link
Contributor

Plugin build for e8e58cb is ready 🛎️!

@JohnPixle
Copy link
Contributor Author

Hey @HardeepAsrani , just wanted to set some perspective here if possible. I think we should prioritise with the review and merging of this PR, it has a bunch of changes, mainly the addition of new patterns and pages.

We need to start consolidating things and shape-up the theme with all the final material (patterns/templates/pages) included, so we can test holistically with the onboarding.

I need you to review overall, and investigate some rendering issues with 4 patterns that I have faced problems with. Other than that, things appear to be working well.

Testimonial Columns with Rating

Feature Cards

Testimonial Cards

Pricing Plans 4

rendering

Pattern Library at the Demo

If you want, you can see the full pattern library in the demo site.

Magic login here, and then go to pattern library. You should see all the patterns there,

Check also the full-page patterns in the pages category. I made them composite, referencing other patterns (wherever possible), like you suggested. Homepage 2 Page Example

Last but not least, I think this PR here is important to merge first, if we want to finalise the templates in this one: #85

Thanks! Please don't hesitate to ping me for any help or clarifications at any point.

@pirate-bot
Copy link
Contributor

Plugin build for 5461d14 is ready 🛎️!

@HardeepAsrani
Copy link
Member

@JohnPixle I've fixed the resolving errors in blocks. Let me know if all is good.

@HardeepAsrani HardeepAsrani merged commit b8e5eaf into development Dec 8, 2023
8 checks passed
@JohnPixle
Copy link
Contributor Author

@HardeepAsrani Thanks, looking perfect now. Just out of curiosity, what was the problem with these patterns? I almost fried my head to debug them, but eventually gave up 😆 . Thanks anyway!

@HardeepAsrani
Copy link
Member

@JohnPixle God knows, I just copied and pasted them back from the patterns website that you have over at instawp. So might have been something during the exporting/importing.

@pirate-bot
Copy link
Contributor

🎉 This PR is included in version 1.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Indicate that an issue has been resolved and released in a particular version of the product.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants