Project

General

Profile

Actions

Feature #5608

closed
IK JD

Clicking on widget name does not open widget function

Feature #5608: Clicking on widget name does not open widget function

Added by Ivor Kreso almost 11 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
Low
Assignee:
Jared Dillard
Category:
Dashboard
Target version:
Start date:
12/07/2015
Due date:
% Done:

90%

Estimated time:
Plus Target Version:
Release Notes:

Description

On versions prior to 2.3 clicking on widget name would open widget function. For example, if clicked on IPsec widget name, it would automatically open IPsec status. Same with OpenVPN.

I'd suggest you return that feature or provide a icon next to the widget name.


Files

Dashboard225.png (42.3 KB) Dashboard225.png Phillip Davis, 12/08/2015 10:06 AM

Updated by Anonymous almost 11 years ago Actions #1

  • Status changed from New to Feedback

So clicking the widget name would have the same effect as clicking the '+' icon?

IK Updated by Ivor Kreso almost 11 years ago Actions #2

Not sure if you understood me. On 2.2+ if you click on widget name (so not from the widget list, but when you add the widget) it will open the status or options for that widget. Let me know if you need me to provide more info.

Updated by Anonymous almost 11 years ago Actions #3

Which is the same thing that happens when you click the '+' sign to the far right of the widget name (in the title bar) right?

IK Updated by Ivor Kreso almost 11 years ago Actions #4

Are you talking about this ? https://i.imgur.com/0pt7bbp.png

Because I'm talking on about this, where there is no + sign on the widget https://i.imgur.com/VpXk3oM.png

Updated by Anonymous almost 11 years ago Actions #5

Because in your example the widget is already open, so it show a minus sign rather than a plus sign.

IK Updated by Ivor Kreso almost 11 years ago Actions #6

Click on + sign while it's being closed / minimized only opens the widget, it doesn't go to status or function. I think that's what confusing you. When the widget is open, when you click on widget name, for example on IPsec it goes to IPsec status page. Same happens with OpenVPN, CARP and other widgets. It opens an additional function.

PD Updated by Phillip Davis almost 11 years ago Actions #7

Here is a 2.2.5 dashboard screen shot. "Gateways" is underlined. When you click it, you are taken to status_gateways. Similar for most widgets.

Updated by Anonymous almost 11 years ago Actions #8

Ah. "Open the widget function" doesn't exactly convey that. So you want to open the status page that is related to the widget, where applicable. The architecture of the dashboard may make that something of a challenge, but I'll take a look.

IK Updated by Ivor Kreso almost 11 years ago Actions #9

Yes, that's it. Glad we're on the same page now. That was functional on versions prior to 2.3, a lot of people are used to it I suppose. It's quite convenient.

Updated by Anonymous almost 11 years ago Actions #10

Added as requested.
JDillard will refine link styling

IK Updated by Ivor Kreso almost 11 years ago Actions #11

  • % Done changed from 0 to 90

This is now working flawlessly, should we close this ticket?

PD Updated by Phillip Davis almost 11 years ago Actions #12

Not quite flawless - I noticed that Dynamic DNS Status widget does not have a link. That is because the displayed name of the widget is "Dynamic DNS Status" but the underlying file names and var names for this widget are based on dyn_dns_status.
https://github.com/pfsense/pfsense/pull/2197
This makes the code simpler and based on the underlying file name that the widget uses, rather than the displayed name.

PD Updated by Phillip Davis almost 11 years ago Actions #13

Also https://github.com/pfsense/pfsense/pull/2198 eliminates the need to generate the widget title text by auto-capitalization and a loop of special cases.
I think this is all working now - until the next special case is found :)

Updated by Anonymous almost 11 years ago Actions #14

Agreed. I was about to address exactly this but you beat me to it :) Thanks.

Updated by Anonymous almost 11 years ago Actions #15

  • Status changed from Feedback to Assigned
  • Assignee set to Jared Dillard

Need to style the linked titles (probably white, underlined on pfSense.css). The default blue link color is not so good on a gray title bar.

Updated by Anonymous over 10 years ago Actions #16

  • Status changed from Assigned to Closed

Completed as requested

Actions

Also available in: Atom