Defect #5775
ldap authentication in admin menu should have an icon
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Low | Due date: | ||
| Assignee: |  Eric Davis | % Done: | 0% | |
| Category: | UI | |||
| Target version: | 1.0.1 | |||
| Resolution: | Fixed | Affected version: |
Description
In app/views/admin/_menu.rhtml the entry label_ldap_authentication doesn't contain a class and therefore no icon is displayed.
The link in Administration menu was introduced in r3744.
Associated revisions
Added icon for the LDAP authentication menu item. #5775
History
#1
Updated by Holger Just over 9 years ago
- File key_go.png added
- File group_key.png added
I would propose the key_go icon of famfamfam for now.
Once we are able to define complete authentication modules in redmine and be able to configure those in a combined GUI, I would propose the group_key icon.
#2
Updated by Jan from Planio www.plan.io over 9 years ago
- Assignee set to Eric Davis
fwiw, these two from famfamfam would work as well imho. but we should definitely get one of them into the final (non RC) 1.0 release.
#3
Updated by Jan from Planio www.plan.io over 9 years ago
- File database_connect.png added
- File server_key.png added
missed the attachments...
#4
Updated by Eric Davis over 9 years ago
- Status changed from New to Resolved
- Target version set to 1.0.1
- Resolution set to Fixed
Thanks everyone, I was going to add one but forgot about it while doing the release. I ended up using the server key and giving it a css class so other server authentication types can reuse it.
#5
Updated by Eric Davis over 9 years ago
- Status changed from Resolved to Closed
Merged into 1.0-stable in r3862.