search

mishunov / webcouturier.dropdownmenu

7 stars 24 forks source link

wrong submenu content on plone4 #1

Closed toutpt closed 13 years ago

toutpt commented 13 years ago

I have a default plone install built from the provided buildout.

With the default plone content, 'news' tab list news, events, members as submenu of news. It should list nothing except 'archive'

rpatterson commented 13 years ago

I can confirm this, both in tests and production. The culprit is 001fbfc0a7d5d765e5b17a1945a7612e1bd72bc0.

rpatterson commented 13 years ago

I can also confirm that pull request #5 fixes this. Can we merge this fork?

mishunov commented 13 years ago

I have merged the pull request. I didn't have time to check it myself, but if it works, it works. Thanks, JeanMichel and Ross.

mishunov commented 13 years ago

Hi,

I have re-opened this ticket, since it didn't really work for me properly after the merge. And, the code I saw didn't really do the thing to me. But I have got the idea and made appropriate changes to the code to work properly in the Plone 4 and play nice with the INavigationRoot. Could you, guys, please check the master to make sure things do work properly for you now?

Thanks for your input.

toutpt commented 13 years ago

master is working for me. I have just see the bug with INavigationRoot using linguaplone, sorry for that.

mishunov commented 13 years ago

Great, then I close the issue. Thanks toutpt.