Details
Description
(This is a list of issues I'm familiar with. Feel free to add/remove/break up items.)
1. Spacing/Sizing: There seem to be some issues related to font-size,
padding, alignment, etc. For example, the styling of From/Reply-to/Recipients/Subject seems a bit different when the content appears inside a tab (eg default layout) versus outside a tab (eg "wizard" or "unified2" layout).
2. DateTime: The scheduling UI currently uses date/time formats which match the underlying API ("YYYY-MM-DD" and "HH:MM"). Ideally, it should default to localized formats (e.g. "M D, Y" and "HH::MM A") based on Civi's configuration. The main places to change are directives crmUiDate and crmUiTime. (crmUiDateTime and EditScheduleCtrl are tangentially involved; however, they should continue working as long as crmUiDate and crmUiTime still use YYYY-MM-DD and HH:MM as the internal data model.)
3. Popups: There are popups for previewing the recipient list, previewing mail content, and previewing mailing-components (automated-messages). Ideally, they should appear bigger and support scrolling.
4. Tabs: Tabs are used in the default layout and unified layout. The coloring/styling of the tabs appears incongruous (eg with white background/white trim) and should "fit" with the other UI elements (e.g. accordions).
5. Accordion: In some cases (such as "Preview", "Review", and "Schedule" in the default layout), fields are grouped together using crmUiAccordion for visual reasons – even though there's no need to collapse the group. These are currently expanded ("<crm-accordion collapsed=false>"); ideally, they could locked open (eg "<crm-accordion collapsible=false>").
6. (Towards the end, as things stablize) There may be a few spots which use inline CSS styling instead of classes. Clean these up.
(Note: There are multiple layouts – e.g. "/civicrm/a/#/mailing2/new". "/civicrm/a/#/mailing2/new/wizard", "/civicrm/a/#/mailing2/new/unified", "/civicrm/a/#/mailing2/new/unified2". Most of the issues should be the same regardless of the layout chosen.)
7. A few things from the blog discussion https://civicrm.org/blogs/totten/civimail-ui-refresh-v46 – in the wizard layout, combine steps 1 and 2. In the compromise layout, move "Mailing Name" so that it's inline with the other options (instead of its own block).
8. Buttons - add icons and style appropriately
9. Clicking 'New A/B Test' from the Mailings menu loads the form (which is a 'single page app') BUT the dropdown menu remains visible, overlaying the form.