Closed GoogleCodeExporter closed 8 years ago
i used it in base.html that used global site
Original comment by xur...@gmail.com
on 15 Jun 2009 at 12:36
can menu_as_ul tag use menu_group name directly?
Original comment by xur...@gmail.com
on 15 Jun 2009 at 12:40
Hi,
At the moment it does not take the menu_group name directly and requires an
instance
of it to be passed into the context.
It does however make sense that it should take the name as a string (or at
least
allow both options) so I'll open a feature request to get that added.
--
G
Original comment by greatlemer@gmail.com
on 15 Jun 2009 at 6:52
Original comment by greatlemer@gmail.com
on 15 Jun 2009 at 6:54
thanks for your reply,expecting name directly feature in term of global menu and
performance.
thus far,gtd is suitable for submenu or appmenu.
thanks a lot for your work,benefited me so much.
Original comment by xur...@gmail.com
on 15 Jun 2009 at 2:22
Hi,
I've updated the code to allow strings to be entered as well as menu_group
instances.
These strings will then be used to look for a menu_group that has that name.
Hopefully this will fulfill your use case.
Unfortunately you may need to change the name of the directory you have the app
stored as it is now be required to be named 'gdt_nav' (if you had called it
something
else previously then you may need to dump the current data to a fixture then re-
import it again afterwards). This had to be enforced due to an issue with the
way
django imports template tags not allowing me to use a relative import for the
models
and as such I had to hard code the app name in.
--
G
Original comment by greatlemer@gmail.com
on 15 Jun 2009 at 10:39
surprised by your efficiency.i'll try now.thank a lot.
Original comment by xur...@gmail.com
on 16 Jun 2009 at 4:42
[deleted comment]
[deleted comment]
great job.thanks a lot.
a I18N feature for menu options name will be better.:-)
Original comment by xur...@gmail.com
on 16 Jun 2009 at 5:20
added I18N support for menu options
Original comment by xur...@gmail.com
on 16 Jun 2009 at 5:38
Attachments:
Hi,
Thank you for the patch to enable I18N on the menu options. I've now applied
that to
the code so the feature should be there for everyone. I18N was something on my
list
of things to implement/add but I've not yet found the time to look at it
properly
yet.
Does everything else seem to be working now?
--
G
Original comment by greatlemer@gmail.com
on 16 Jun 2009 at 7:02
[deleted comment]
it sounds everything go as we want.maybe more tests needed.
Original comment by xur...@gmail.com
on 17 Jun 2009 at 2:01
Original issue reported on code.google.com by
xur...@gmail.com
on 15 Jun 2009 at 12:35