Opened 18 years ago
Closed 18 years ago
#4191 closed enhancement (duplicate)
CORE - Allow Removal of navigation items from within trac.ini
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | low | Milestone: | |
Component: | general | Version: | devel |
Severity: | trivial | Keywords: | |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
The goal is to allow easy customization, without alteration of code and templates.
Thus the scope of this ticket is very small.
Suggested implementation:
mainnav = wiki, -timeline, …
The timeline item would be removed in this case (prefix with "-"). Another option would be the "!", but this could be missleading with the functionality within the wiki
Attachments (2)
Change History (13)
comment:1 by , 18 years ago
comment:2 by , 18 years ago
This is also a pretty trivial plugin. Perhaps a Request-a-Hack is in order?
follow-up: 4 comment:3 by , 18 years ago
Keywords: | needinfo added |
---|---|
Priority: | normal → low |
Severity: | normal → trivial |
I think disabling modules is generally the preferable solution. Removing permissions will still leave those modules visible to users with TRAC_ADMIN permission.
Do you have a specific reason to leave a module enabled, but inaccessible through the UI? I think this is a wontfix or worksforme since disabling the module already does what you've asked for, but I'll leave it open temporarilly for feedback.
comment:4 by , 18 years ago
Replying to mgood:
I think disabling modules is generally the preferable solution. Removing permissions will still leave those modules visible to users with TRAC_ADMIN permission.
Do you have a specific reason to leave a module enabled, but inaccessible through the UI?
The reason is to provide flexibility in the interface-customization.
I could e.g. remove the "help/guide", "about" menus and enable them in another location of the UI.
The goal is to enable users to customize trac without interventions which make seperate maintenance neccessary (e.g. custom templates). This way those users can keep their "custom-trac" closer to the original one, enabling back-contribution of their code to the trac project.
I think this is a wontfix or worksforme since disabling the module already does what you've asked for, but I'll leave it open temporarilly for feedback.
I've seen several user requests asking for this functionality.
And I've seen several sites {http://code.djangoproject.com/ see example] which remove such navigation links, without disabling the content.
Trac should simplify the creation of Custom Versions
follow-up: 6 comment:5 by , 18 years ago
This has been implemented in a plugin. I would propose a wontfix for this ticket.
follow-up: 7 comment:6 by , 18 years ago
Replying to Noah Kantrowitz (coderanger) <coderanger@yahoo.com>:
This has been implemented in a plugin. I would propose a wontfix for this ticket.
What you (or any other person, even committers of trac) implements as a plugin on "trac-hacks" has no relevance to the state of this ticket, which affects the core of trac.
(sidenote: Similar tickets could e.g. ask to integrate existing plugin-functionality where it belongs to: to the core)
The Plugin mechanism is an powerfull mechanism, which (when not used correctly), can lead to a very bad design and usability (as it's currently the status with trac, which needs a hand-full of un-coordinated plugins to become operative/usable in typical environments).
Of course I am not going to install and maintain for every tiny bit of functionality a plugin.
As to your plugin description: you should at least point to this ticket (from which your implementation idea originates).
And finally: having 2 options "-" and "!" should be avoided ("!" for the reason stated, as it's missleading/conflicting with the wiki functionality).
comment:7 by , 18 years ago
Replying to ilias@lazaridis.com:
As to your plugin description: you
correction of link: http://trac-hacks.org/wiki/NavHiderPlugin
by , 18 years ago
Attachment: | EnableNavItemRemoval.diff added |
---|
comment:8 by , 18 years ago
The provided patch implements this functionality within the core, providing users with a simple intuitive way to customize the trac UI. The implementation does not break existent behaviour.
by , 18 years ago
Attachment: | EnableNavItemRemoval2.diff added |
---|
comment:9 by , 18 years ago
provided patch did not work on multiple removals.
New patch uses a duplicate of the list:
There are already ways of doing this:
If the intention is not to use a feature as the Timeline, it is much better to disable it than merely to hide the navigation.