Project

General

Profile

Actions

Todo #5492

closed

Correcting the names of available widgets

Added by Ivor Kreso over 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Category:
Web Interface
Target version:
Start date:
11/19/2015
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
Release Notes:

Description

The names of widgets are capitalized and not written in the correct way. Please change the names to follow their correct way of writing:

Here are the corrections:

- gmirror status
- IPsec
- OpenVPN
- SMART status
- CARP status
- Dynamic DNS (DynDNS is a service from Dyn, so it's more appropriate to dynamic DNS)
- RSS
- NTP status

To my understanding the widget names in the available widget list are also applied to actual widgets.

Thanks!

Actions #1

Updated by Phillip Davis over 8 years ago

Here is the way I thought of to do it. Maybe there is a better way?
https://github.com/pfsense/pfsense/pull/2093

Actions #2

Updated by Renato Botelho over 8 years ago

  • Status changed from New to Feedback

Pull request has been merged

Actions #3

Updated by Chris Buechler about 8 years ago

  • Status changed from Feedback to Resolved

these are all fixed

Actions #4

Updated by Phillip Davis about 8 years ago

And for the record there were later changes so that the widget display titles come from the var defined by each widget in the respective include file - e.g. https://github.com/pfsense/pfsense/blob/master/src/usr/local/www/widgets/include/gmirror_status.inc etc.
So each widget can choose the text of its title, like this was used in 2.2.* already.

Actions

Also available in: Atom PDF