Project

General

Profile

Actions

Feature #7099

closed
KB

Make breadcrumbs clickable

Feature #7099: Make breadcrumbs clickable

Added by Kill Bill over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Web Interface
Target version:
Start date:
01/07/2017
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:

Description

Dunno if it's just me, but the entire feature is very much pointless when it's unusable for navigation. Seems pretty standard across implementations that those are clickable links.

PD Updated by Phillip Davis over 9 years ago Actions #1

I wondered about that also, at the time of the bootstrap conversion, but there was enough going on that I never followed it up.
Here is a bit of demo code to start discussion:
https://github.com/pfsense/pfsense/pull/3364

PD Updated by Phillip Davis over 9 years ago Actions #2

Proposed solution is out for review/test https://github.com/pfsense/pfsense/pull/3369

KB Updated by Kill Bill over 9 years ago Actions #3

Phil: That looks great, thanks! I guess the same can be used for packages (the PHP files I mean, not XML), right?

PD Updated by Phillip Davis over 9 years ago Actions #4

Yes, for packages with XML the pkg.php and pkg_edit.php try to put some reasonable breadcrumb links in (try the Notes package, which has XML that generates a base form (controlled by pkg.php) and has Edit Note (controlled by pkg_edit.php). The existing way pkg_edit.php generates the breadcrumb list is a bit odd - e.g. when editing a note the breadcrumb list is "Status:Notes / Edit / Notes" - the "Edit" ends up in the middle, and I gave that the "@self" link, and the others take you back to the main Notes page. The order of that breadcrumb list could be fixed better - but that was an existing "feature" that was out-of-scope for me to start messing with.

Packages that just have a simple "2-node" breadcrumb list ("Services / Cron") will get a self-reference link on that last entry (which effectively refreshes the page), and so need no changes.

Packages with >2-node breadcrumb lists in their PHP files will get a self-reference on the last element of each list. To make other links in the list, they need to be modified to define $pglinks appropriately.

KB Updated by Kill Bill over 9 years ago Actions #5

Works.

JP Updated by Jim Pingle over 9 years ago Actions #6

  • Status changed from New to Resolved

JP Updated by Jim Pingle over 9 years ago Actions #7

  • Target version changed from Future to 2.3.4
  • % Done changed from 0 to 100
Actions

Also available in: Atom