Inconsistent sub-menu on Browse Source comparing with View Tickets

[pkou at ua.fm]

Menu View Tickets contains two sub-items:

• Available Reports
• Custom Query

An item are not clickable when active.

In opposite, Browse Source contains one sub-item that changes its value depending on the active selection:

• Revision Log
• View Latest Revision

This needs to be unified in either way.

[Christian Boos]

If this needs unification, I would propose to modify the View Tickets to match Browse Source: only active links should be shown.

In the View Tickets, there's no point to show the inactive link, as this is redundant with the page title.

See also TracProject/UiGuidelines.

[Manuzhai]

I think it's good to show the inactive link. That way, links are always in the same place, allowing for effective muscle memory clicking.

[Christopher Lenz]

I agree with Manuzhai that showing the inactive link is good. It indicates where you currently are, as opposed to showing only where you can go.

The navigation links in the browser are not exactly the same story, however. View Latest Revision would need to be renamed, at least, because it would not make sense to show that label as inactive link when looking at an older revision.

Anyway, this can wait until after 0.9 IMO.

(And also IMO, the whole subnavigation story needs to be completely redesigned… it's a mess).

[Christian Boos]

Milestone 1.0 deleted

[Christian Boos]

With the changes in attachment:custom-query-link.patch:ticket:9604​, the "inactive" link is no longer inactive, but goes back to a "new" default custom query (kind of "Reset").

[Christian Boos]

Which leaves us with not showing the Available Reports contextual link as inactive when viewing the list of Available Reports. I'll fix that if nobody objects.