Open Klap-in opened 4 years ago
I started the first work on replacing dTree by FancyTree. First results:
I wonder if I should readd the contextmenu again, but my impression is that there is a smaller group, that really appreciates this functionality.
Please see #244 for detailed discussion whether the contextmenu should return.
Please share your preference: :+1: = keep the contextmenu feature, :-1: = I do not use the contextmenu feature
indexmenu plugin is still the best plugin to display menu at the sidebar. Now i recognized, using the ckgedit editor we have to decide for defer_js or against defer_js option. the indexmenu requires to disable it, the ckgedit requires to enable it :-O i am not skilled enough to add the required changes to indexmenu to make it work with the defer_js option. seens there is no more progress yet by anyone. did not find any pull requets or forks :O
I started with implementing the Fancytree javascript tree. But I need to perform a large refactoring of the code as well. Because it is still quite broken, I did not publish it yet.
First version of the new js tree based on Fancytree is available: #254
Temporary extra settings: treenew
, treeold
and treeboth
.
Any feedback is welcome. Please note it is still subject of change.
Not yet tested with the following settings, if you use these, please share your experience:
Just one thing I noticed when using the deferjs
branch: Icons do not load because I have DokuWiki in a subdirectory. I manually amended the all.less
file on line 13 to add my subfolder before the prefix and then it seems to work.
Oh and another: The URLs to click on didn't work for me because they didn't include doku.php?id=
which is needed in our wiki. I went to script.js and added that manually in the the function for enhanceTitle
property of fancytree object, but this would probably need to get auto-detected somehow.
I also removed the warning about using the js option while having defer_js enabled. In our internal wiki we just have a single page with a simple indexmenu showing all pages, so I'm satisfied.
Thanks for your feedback! It confirms that the different urls due to the different rewrite settings still needs more coding and testing. If I have an update I would like to ask you for new feedback.
I expect I have a rather okay state of the plugin again. The treenew
is not feature complete, and is a bit different that the dTree based one. Therefore, I do not make it default yet.
Everybody who like to try is invited to do so! Any feedback is welcome. If this is stable enough, I will release it for everybody on short notice.
@jayna37 I think I have solved your issues for your case where you do not use urlrewriting. You are welcome to test.
Please see #269 for the latest version of the development version.
You can also download it from the devel
branch (https://github.com/samuelet/indexmenu/archive/devel.zip)
@Klap-in Sorry, but I left that company in the meanwhile. Would've been happy to test for you with that specific installation.
I will remove the old JavaScript tree because its maintenance is too much work. It is not doable to keep it.
I guess now that most remaining issues/questions will be related to the new skins.
I find that our index menu is no longer working properly, and it has performed flawlessly for years. No indexmenu is unresponsive when I remove a page, or change the title. What has happened here? https://opencpn.org/wiki/dokuwiki/doku.php?id=opencpn:supplementary_software:signalk:sk-windows
https://opencpn.org/wiki/dokuwiki/doku.php?id=opencpn:supplementary_software:signalk:a1
Refactor the javascript code:
Implementing new menutree with FancyTree:
treenew
,treeold
andtreeboth
option. (First to test both, if stable, swap to defaulttreenew
In e.g. after one year these options are removed.)max#n[#m]
(number of levels retrieved by AJAX at initial loading #n and lazy loading #m). Max on initial loading (probably) only applied ifnavbar
option was used. Proposal: keep option, #n default 1, #m also default 1. (level set number of opened levels, overrules this setting of course)navbar
(tree opens itself in current namespace)nons
(exclude ns nodes)nopg
(exclude page nodes)noscroll
(lets tree menu scroll in small widths) Proposal: removeid#[random|ns|number]
(used for storing opened/closed nodes) Proposal: removecookie
(stores opened/closed nodes). Proposal: removemaxjs#n
(number of nodes rendered) Proposal: remove (not needed with todays javascript/not supported by Fancytree)To check:
idfilter()
(for:
,;
,/
)