Project

General

Profile

Actions

Todo #5492

closed

Correcting the names of available widgets

Added by Ivor Kreso almost 10 years ago. Updated almost 10 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 almost 10 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 almost 10 years ago

  • Status changed from New to Feedback

Pull request has been merged

Actions #3

Updated by Chris Buechler almost 10 years ago

  • Status changed from Feedback to Resolved

these are all fixed

Actions #4

Updated by Phillip Davis almost 10 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