Foreman

Bug #82

Re-organize "Classes" on Create Host page

Added by Josh Baird over 2 years ago. Updated over 2 years ago.

Status:

Closed

Start:

11/17/2009

Priority:

Normal

Due date:

Assigned to:

Ohad Levy

% Done:

100%

Category:

Web Interface

Target version:

0.1-3

Backlog:

Difficulity:

Votes:

Description

The "Classes" section of the "Create Host" page could use a face lift. It is hard to read, especially with a large number of classes. It would also be nice if you could somehow limit what modules are listed, perhaps modules specific to the OS? I have lots of modules such as "ntp::linux::config," "ntp::linux::service," etc but I'm really only interested in seeing/including "ntp," since everything else (ntp::linux::bleh) is included from the "ntp" class.

I think a good layout would be two boxes (Available and Selected) where you could drag (or move over with an arrow) classes from the available box to the selected box.

dragndrop.jpg (10.3 KB) Josh Baird, 11/18/2009 04:11 pm

Associated revisions

Revision 1cd4a9b62d9f71dd8661cca622b64d9b4f0ac1f7
Added by Ohad Levy over 2 years ago

fixes #82

History

#1
Updated by Josh Baird over 2 years ago

File dragndrop.jpg added

I have uploaded an example screenshot of one way to organize this.

#2
Updated by Ohad Levy over 2 years ago

Category set to Web Interface
Status changed from New to Feedback
Assigned to set to Josh Baird

diff --git a/app/controllers/hosts_controller.rb b/app/controllers/hosts_control
index b7ad2a2..25fbcc6 100644
--- a/app/controllers/hosts_controller.rb
+++ b/app/controllers/hosts_controller.rb
@@ -15,6 +15,7 @@ class HostsController < ApplicationController
     config.columns[:domain].form_ui  = :select
     config.columns[:environment].form_ui  = :select
     config.columns[:puppetclasses].form_ui  = :select
+    config.columns[:puppetclasses].options = { :draggable_lists => {}}
     config.columns[:fact_values].association.reverse = :host
     config.nested.add_link("Inventory", [:fact_values])
     config.columns[:puppetmaster].description = "leave empty if