Project

General

Profile

Actions

Defect #5775

closed

ldap authentication in admin menu should have an icon

Added by Bernhard Furtmueller almost 14 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Low
Assignee:
Category:
UI
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
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.


Files

key_go.png (744 Bytes) key_go.png Holger Just, 2010-07-03 13:20
group_key.png (813 Bytes) group_key.png Holger Just, 2010-07-03 13:20
database_connect.png (763 Bytes) database_connect.png Jan from Planio www.plan.io, 2010-07-19 19:23
server_key.png (746 Bytes) server_key.png Jan from Planio www.plan.io, 2010-07-19 19:23
Actions #1

Updated by Holger Just almost 14 years ago

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.

Actions #2

Updated by Jan from Planio www.plan.io over 13 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.

Actions #4

Updated by Eric Davis over 13 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.

r3859

Actions #5

Updated by Eric Davis over 13 years ago

  • Status changed from Resolved to Closed

Merged into 1.0-stable in r3862.

Actions

Also available in: Atom PDF