I am creating this issue to fix some minor issues on the admin UI:
- The description of the "Enabled provider" field is a copy of the description of the "Retry sending mail on error." checkbox.
- The header of table of providers is shorter than its rows - the colspan must be increased. Also, the column should be called "Operations" as other Drupal's tables.
- Typo on the "SMTP Server Settingssss" fieldset.
- The submit button of the provider's edition form should be called "Save" instead of "Submit".
- When editing the provider's machine name, the textfield appears outside the fieldset.
- smtp_admin_provider_delete_submit() expects a third parameter which is not provided.
- After a provider is deleted, the user is not redirected back to configuration form.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | admin_ui_adjustments-2821938-5.patch | 2.84 KB | dbiscalchin |
| #2 | admin_ui_adjustments-2821938-2.patch | 2.09 KB | dbiscalchin |
Comments
Comment #2
dbiscalchin commentedPatch to fix the above mentioned issues.
Comment #3
dbiscalchin commentedComment #4
dbiscalchin commentedReopening to fix these issues too:
Comment #5
dbiscalchin commentedFixing the last two issues too.
Comment #6
dbiscalchin commentedComment #7
chris matthews commentedGreat improvements, but the 2 year old patch no longer applies to 7.x-2.x-dev
error: patch failed: smtp.admin.inc:65
error: smtp.admin.inc: patch does not apply
Comment #8
bluegeek9 commentedUnfortunatly, Drupal 7 is End of Life and no longer supported. We strondly encourage you to upgrade to a supported version of Drupal.