Skip to content

Conversation

@arinaturcu
Copy link
Contributor

@arinaturcu arinaturcu commented Oct 7, 2022

Adds the new Landing Pages.

  • The new templates are shown in the gallery
  • The Landing Page category is present in the category dropdown
  • New Landing Page template in the wizard
  • The selected page has a more specific page status (instead of Maintenance Page the page will have Maintenance \ Coming soon \ Landing Page depending on the category)

Closed #327.

@github-actions
Copy link

github-actions bot commented Oct 7, 2022

Plugin build for df5e32d is ready 🛎️!

@arinaturcu arinaturcu changed the title Add landing pages [WIP] Add landing pages Oct 26, 2022
@arinaturcu arinaturcu requested a review from selul October 26, 2022 08:27
@JohnPixle
Copy link

I just tested the build on a TasteWP instance and it works perfect, nice Job @arinaturcu

I only noticed that the compression of the library images is pretty harsh. Not a critical issue, but perhaps we can do something about it (lowering the compression)? Happy to help with the screenshots if you need me to, I am not sure if these are auto-generated 🤔

Screenshot 2022-10-26 at 13 50 26

@arinaturcu
Copy link
Contributor Author

Thank you for testing @JohnPixle!
They are not auto generated, I took them. If you think they are OK in terms of structure, I can lower the compression and replace them.
Otherwise, some new ones are welcomed 🚀

@JohnPixle
Copy link

@arinaturcu structure is fine, I just think compression is a bit weird, I see there are very rough edges in them.
If you are doing it with Photoshop, I believe it must be a setting in the Image Quality dropdown in the Save for Web panel.

It looks like it is set to Nearest Neighbour (and this is preserving the rough edges). But I don't want to complicate you with this, I will be happy to send over new ones soon.

@arinaturcu
Copy link
Contributor Author

@JohnPixle Okay, I'll leave them to you then. Thank you for helping with these 🚀

@JohnPixle
Copy link

Hey again @arinaturcu

So, I took a deeper look with the thumbnails, I think the issue with the overly sharp edges was coming from the resizing of the images, and not the actual compression. The original images appear to be big, and when resized they appear with the rough edges.

I took the initiative to provide the images to you in the final dimensions as they appear in the library, 380 x 405. I hope the dimensions are correct. If not, let me know.

I also safely compressed them a bit. Now all the thumbnails of the landing pages are only 174 KB, which is significantly less than before (an original image of a single landing page template used to be about 400KB). Hope it helps 👍🏻

I did this for all 3 thumbnail types, Landing pages, Maintenance and coming soon. I am attaching the Zips here.

landing-pages.zip
Coming Soon.zip
maintenance.zip

I would also like to point out two things:

1. New templates. Apart from the landing pages, in this version of the plugin, we have one new Coming Soon template. Link here.. We also have one more Maintenance template. Link here. The thumbnails are included in the zips.

2. Template names. For the Maintenance and Coming Soon templates I would advise to start using a bit more generic names (maintenance 1, maintenance 2 etc) instead of style-related names as we do now (modern , cover etc). I am suggesting this in order to keep things easy since we are going to add more of these templates in the future, and we may not be able to define a unique style name for each of them.

Let me know what you think and always happy to help for anything you may need.

@arinaturcu
Copy link
Contributor Author

Great, @JohnPixle! 🚀
I will replace the screenshots and the names of the templates and add the new ones. I will come back with an update afterward.

Thank you!

@irinelenache
Copy link

@arinaturcu Tested and didn't find any problem in terms of functionality 🚀

After @JohnPixle's mentions are addressed and he approves it, this can be moved to Ready to merge

@arinaturcu
Copy link
Contributor Author

Hey @JohnPixle, I made the changes.
So have a look when you have some time and let me know if everything is fine 🚀

@JohnPixle
Copy link

JohnPixle commented Oct 28, 2022

@arinaturcu Thanks for the update, we are all good with the Thumbnails quality, thanks so much for taking care of it 🙂 👍🏻

I noticed two small issues though:

Wizard:

  • When I run the wizard, the image links are broken. Maybe it has to do with the renaming of the files? Tried on a new tasteWP instance.

  • Also, in the description, I would suggest that instead of pre-written templates we use the term pre-designed templates.

Screenshot 2022-10-28 at 10 54 53

Ordering of the templates

  • It's not critical, but I was wondering if we can have the maintenance / coming soon templates appear in the correct order (Maintenance 1, Maintenance 2, Maintenance 3 etc...). Landing pages are ok.

Screenshot 2022-10-28 at 10 47 33
Screenshot 2022-10-28 at 10 47 45

Let me know your thoughts and don't hesitate to ping me in case you need any clarifications on the above.

@arinaturcu
Copy link
Contributor Author

@JohnPixle Thank you for catching these 😄 .
Should be fixed now.

@JohnPixle
Copy link

@arinaturcu Just checked and all looks rock-solid. Thanks so much for taking care of them, looking forward to see them live. From my side it's a green light to merge! 🚀

@arinaturcu arinaturcu requested a review from selul November 1, 2022 15:36
Copy link
Contributor

@selul selul left a comment

Choose a reason for hiding this comment

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

🚀

@selul selul merged commit 542c488 into development Nov 1, 2022
@selul selul deleted the feat/landing-pages branch November 1, 2022 15:53
@pirate-bot
Copy link
Contributor

🎉 This PR is included in version 2.6.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 Nov 2, 2022
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.

6 participants