Opened 14 years ago
Closed 14 years ago
#8257 closed enhancement (fixed)
Display PasswordStore Option Docs on Configuration Page
| Reported by: | David A. Riggs | Owned by: | Steffen Hoffmann |
|---|---|---|---|
| Priority: | normal | Component: | AccountManagerPlugin |
| Severity: | normal | Keywords: | display configuration doc-string |
| Cc: | Trac Release: | 0.12 |
Description
The AccountManager "Configuration" page (template 'admin_accountsconfig.html') renders a form for editing the options of enabled PasswordStore implementations. This page does not currently render the docstrings for Options.
I have attached an initial patch (a whopping two lines) which renders the docstrings on this form.
Attachments (2)
Change History (4)
Changed 14 years ago by
| Attachment: | accountmanager_doc_patch_r9560.diff added |
|---|
Changed 14 years ago by
| Attachment: | accountmanager_doc_patch_screenshot.png added |
|---|
Screenshot showing docstrings rendered on Configuration form
comment:1 Changed 14 years ago by
| Keywords: | display configuration doc-string added |
|---|---|
| Status: | new → assigned |
Nice, I've just tested your modification. Looks really useful, and strings display done with CSS style selected by Trac's special class="help" integrates seamlessly with current page style.
I'll add i18n support and push it out afterwards. Thank you very much for your valuable contribution.
comment:2 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
(In [9587]) AccountManagerPlugin: Add option doc-strings to configuration admin page, closes #8257.
I've extended the changes suggested by David A. Riggs with i18n markup and will require future doc-strings to be translatable before adding them to the AcctMgr source. There's no sane way to force the same on IPasswordStore's provided as external plugin, but I'm asking very kindly to follow that policy.
Patch to r9560 which renders PasswordStore docstrings on the Configuration form