Project

General

Profile

Actions

Bug #5047

closed
JP CB

system_authservers.php LDAP "Search Scope" fields are not formatted properly

Bug #5047: system_authservers.php LDAP "Search Scope" fields are not formatted properly

Added by Jim Pingle about 11 years ago. Updated about 11 years ago.

Status:
Resolved
Priority:
Low
Target version:
Start date:
08/28/2015
Due date:
% Done:

100%

Estimated time:

Description

On 2.2.x the LDAP Search Scope setting has two fields "Level" And "Base DN", on bootstrap these seem to have lost some formatting.

2.2.x:

2.3:

The Base DN box should move below the Level drop-down. The "Level" drop-down may be confusing without a label on it as well.

Updated by Anonymous about 11 years ago Actions #1

  • Status changed from Assigned to Feedback
  • Assignee changed from Anonymous to Chris Buechler

Layout changed as requested

Updated by Anonymous about 11 years ago Actions #2

  • % Done changed from 0 to 100

Applied in changeset pfsense:commit:df8fca9d1505f856512e23889fa11cddc31a8aaa.

CB Updated by Chris Buechler about 11 years ago Actions #3

  • Status changed from Feedback to Confirmed
  • Assignee changed from Chris Buechler to Anonymous

this still isn't laid out clearly vs. how it was. Both those fields are for defining "Search scope", which is clear in the original. Current bootstrap doesn't clearly show the Base DN has any relation to the search scope.

Updated by Anonymous about 11 years ago Actions #4

  • Status changed from Confirmed to Feedback
  • Assignee changed from Anonymous to Chris Buechler

LDAP Search scope items now grouped horizontally again, but labels beneath each input show the function.
This is about the only way of grouping input elements with the current Bootstrap framework.

Updated by Anonymous about 11 years ago Actions #5

Applied in changeset pfsense:commit:905f61195954ef93a019d89ec4ffd8045ce2d253.

JP Updated by Jim Pingle about 11 years ago Actions #6

  • Status changed from Feedback to Resolved

Looks like it was adjusted again, has them each on one row with a label, looks much nicer now, IMO. Thanks!

Actions

Also available in: Atom