Issues found with new settings pages

[mikemurray]

• Couldn't save the main shop settings form without adding a shop email. Initially, it didn't seem to give me an error. After entering the email, the forms save consistently. Even if I remove the shop email.

• Can scroll the layout container. This is hard reproduce, and may just be a general layout issue. Double-check that no other settings subpage or template does overflow scroll other than the ContentViewPrimaryDetailLayout component.
  

• Some pages need bottom padding
  

• Payment (maybe but it too short to matter)

• Taxes

• Shipping

• System information

• Address validation

Each card either needs margin-bottom, or you can do this to make it automatic

                <Blocks region="RegionName">
                  {(blocks) =>
                    blocks.map((block, index) => (
                      <Box paddingBottom={2} key={index}>
                        {block}
                      </Box>
                    ))
                  }
                </Blocks>

• All "save" button should say "save changes" i18nkey "app.saveChanges". Some do, but others don't

• Consider disabling save buttons if the form isn't dirty. isDirty is available from the useReactoForm hook. disabled={!isDirty || isSubmitting}

• Cancel button in address dialog should be primary. The secondary blue pops

• The loading spinners need some height.

• Sidebar list should highlight the active item

[vandernorth]

Additionally: I wasn't able to open the email-logs.

[willopez]

@vandernorth can you provide more details? I see the email logs under the "Email" link