Update Settings for Card Collections Block

[ravichdev]

Issue by jauyong
Monday Dec 07, 2020 at 14:36 GMT
Originally opened as xwp/material-design-wp-plugin#465

---

Feature description

Review Global vs child settings in the block and compare to how we’ve handled it before(list block).
See related ticket #377 #496

Note: consider using the in context menu as opposed to the side bar with options

Update the more settings side panel based on the mocks.

Design Mocks:

---

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

Side Panel settings should flow as follows:

1. Style
   1. Masonry
   2. List
   3. Grid
2. Card Style
3. Corner Radius
4. Card Settings
   1. Content Layout
   2. Cards Layout
   3. Columns
   4. Gutter
5. Content

Implementation brief

QA testing instructions

Demo

Changelog entry

[ravichdev]

Comment by jauyong
Monday Dec 07, 2020 at 17:28 GMT

---

@jauyong to update AC to indicate we will do what we did for the list block

[ravichdev]

Comment by jauyong
Friday Dec 18, 2020 at 16:52 GMT

---

@jauyong to open new discovery ticket (list block) to inform this ticket.

[ravichdev]

Comment by jwold
Thursday Jan 14, 2021 at 20:45 GMT

---

Blocked temporarily until I can test out the list block. Requested @jauyong to open a ticket to investigate.

[ravichdev]

Comment by jauyong
Friday Feb 12, 2021 at 16:44 GMT

---

@dawidmlynarz to continue reviewing Joshua's mocks on #496

[jauyong]

This will be lowest priority in 0.3

[jauyong]

Note: the blue background on the warning may no longer be part of core so Mario used yellow instead

[csossi]

@PatelUtkarsh - what was updated here? I took a look at the staging site and what I'm seeing has a number of differences with the mock-ups above

[PatelUtkarsh]

Here is before and after:

Before:

before.mp4

After:

after.mp4

I've changed card style to elevation because that's a more appropriate word for it, which fits material guidelines.

@emeaguiar The gutter in mockup is with card settings section but we have it in card style, Is that something we should change or keep it?

[jauyong]

@emeaguiar to verify if the design mock is outdated or if changes need to be made.

[emeaguiar]

@PatelUtkarsh I'm not really sure why I didn't initially moved it, probably just missed the second image, I've created #224 to handle that

[PatelUtkarsh]

@csossi This is updated, Can you verify on test site?

[jauyong]

@csossi You can reference the mockups in the description for correct positioning of the settings.

[csossi]

Verified in QA