Project

General

Profile

Actions
Copy link

Todo #5571

closed
JP JP

Todo #5568: Convert Package PHP GUI Files to Bootstrap

Convert the pfSense package Cron (sysutils/pfSense-pkg-Cron) to Bootstrap

Todo #5571: Convert the pfSense package Cron (sysutils/pfSense-pkg-Cron) to Bootstrap

Added by Jim Pingle almost 11 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Jim Pingle
Category:
Cron
Target version:
pfSense - 2.3
Start date:
12/03/2015
Due date:
% Done:

100%

Estimated time:
Plus Target Version:

Description

Convert the pfSense package Cron (sysutils/pfSense-pkg-Cron) to Bootstrap

JD Updated by Jared Dillard almost 11 years ago Actions
Copy link
 #1

  • Status changed from New to Feedback

This is my first package to convert. Feedback appreciated.

PD Updated by Phillip Davis almost 11 years ago Actions
Copy link
 #2

@Jared - no new version is showing up. I think you need to bump the version in the various places it is recorded.

JP Updated by Jim Pingle almost 11 years ago Actions
Copy link
 #4

  • Status changed from Feedback to Assigned
A couple issues:
  • The fields on the edit page do not need htmlspecialchars on the stored values when building the form (e.g. in the addInput params), Bootstrap handles that and it will cause problems editing and saving. Edit a command with an & in it for example
  • You can build the tabs and use display_top_tabs without the extra table formatting around it.
  • Looks like the edit page has some JS in a script tag that isn't used/needed
  • The edit page is in dire need of more friendly field names and help text. Kick it over to me if you want me to do that part.

JP Updated by Jim Pingle almost 11 years ago Actions
Copy link
 #5

One more: There is a double confirmation prompt when deleting an item from the list

JD Updated by Jared Dillard almost 11 years ago Actions
Copy link
 #6

  • Assignee changed from Jared Dillard to Jim Pingle

I made all the changes except the "friendly field names and help text". I wasn't sure if I should increment the package number in this case though so I didn't.

JP Updated by Jim Pingle almost 11 years ago Actions
Copy link
 #7

  • Status changed from Assigned to Feedback
  • % Done changed from 0 to 100

New version pushed, should show up for testing in 15-20 minutes or so (0.3.5)

CB Updated by Chris Buechler over 10 years ago Actions
Copy link
 #8

  • Status changed from Feedback to Resolved

looks good

Actions
Copy link

Also available in: Atom