-
Notifications
You must be signed in to change notification settings - Fork 6
[Gutenberg] Identify areas of the Travel theme’s homepage that can live as discrete theme blocks and document these for future creation. #9
Comments
We went through the homepage of Travel theme with @DavidCramer to define the custom Gutenberg blocks needed. Gutenberg custom blocks to be developed for homepage:
Question: Does this make sense for these two sections from the editor's point of view?
Question: does it seem reasonable to use the same block for these two sections? On the "Featured destinations" -- what would be the sorting logic of the Cities?
Question: Gutenberg already has a Heading block and a Subhead block -- should we add a new custom block that would allow adding a heading + subheading within the same block or are the current blocks sufficient? Thoughts? |
In general with the blocks, I would say we should implement them according to how you see them in the AMP Start templates as-is to start. Once the blocks are built so that the templates can be re-created, then at that point we can look at ways that they can have customizations to showcase Gutenberg editing. I think that will be more important to focus on later, since the primary thing right now is ensuring we have the frontend rendering faithfully according to the templates. |
Okay, sounds good. In this case the blocks could be:
In this case creating a skeleton/base for all of these blocks that as a placeholder would just display the content from the current homepage template could be perhaps done in one ticket, and then these blocks could be made configurable and functional one by one in separate tickets. @postphotos / @ThierryA Does this sound reasonable? I'd be happy to add the base for Gutenberg blocks if this sounds OK as one ticket. Thoughts? |
Actually taking it as-is for now, maybe the first "Hero" block could be just one -- consisting of the header + search block. |
Hi @miina, yes I think that makes perfect sense.
Yes I agree, the Hero block should have the header and search block (with date picker and CTA). Arguably it could even contain the subheader with the Title, Subtitle and From the blog section. |
Hi @postphotos, on the issues: |
I'm checking to see if Maxim has availability to support this. |
Maxim has availability to help with this and will update with outlines of what design elements are affected by what blocks. |
Linking new Trello card outlining the design task for this issue: |
TLDR: MVP should be AMP Start in WP, but if we plan to distribute we should tool for Gutenberg's abilities @miina - Just noticed I didn't respond to your questions here and wanted to capture thoughts. I agree with @westonruter and believe we should shoot for an MVP of a working theme that is as close to AMP start as possible. I've written these to capture thoughts for future tickets based on your suggestions. Beyond MVP, especially if this theme will actually be used by users, you've raise some good questions here. I'm responding inline to your message from a few days back.
Agreed here. Dynamic layouts should be part of the thought pattern. A user should be able to adjust this section based on their need.
In reality, a site should have the ability to select n number of cities by default here and sort on the fly. Perhaps it pulls a listing and a user can manipulate the display if they choose to do so. I think having a category works as one option, and another would be a "Zoninator" type listing: Allow the users set the positions if they choose to do so, or hide one they don't want to see. They'd likely be promoting a specific area, or through some algorithm this might be populated on a per user basis (based on your location, your recent interest level of a place combined with history and seasonality). I'm not sure what is technically possible within creating transient lists in the way I'm describing, but I think it could be done. Further, this could be a cool interface to design for as a prototype of what Gutenberg could do... cc @jwold. |
Question About Moving To "Ready For Merging" Hi @postphotos,
Do you think we could move this into "Ready For Merging" to show that it's not awaiting review? |
No description provided.
The text was updated successfully, but these errors were encountered: