wagtailmenus icon indicating copy to clipboard operation
wagtailmenus copied to clipboard

Bootstrap4 Templates

Open PaulBrownMagic opened this issue 5 years ago • 7 comments

Bootstrap4 is now up to version 4.3 and is stable. As Bootstrap3 templates are included, perhaps providing Bootstrap4 ones could also be done?

PaulBrownMagic avatar May 12 '19 12:05 PaulBrownMagic

Hi @PaulBrownMagic. Sure, I'd be happy to have a set of templates for bootstrap 4. If you have experience with it, I'd gratefully accept a pull request?

ababic avatar May 12 '19 21:05 ababic

Sure, I'd be happy to, I think I can drag the code from an existing project. Give me a week or so to fit it into my schedule.

PaulBrownMagic avatar May 13 '19 23:05 PaulBrownMagic

Thanks @PaulBrownMagic, that'd be great. Please take a look at our contributing guidelines There are some good tips in there for developing locally, which could save you a bit of pain in getting set up :)

ababic avatar May 14 '19 06:05 ababic

Hey @PaulBrownMagic, did you manage to get anywhere with this? If not i'll pick this up as I'm keen to use them too :).

kbayliss avatar Aug 29 '19 09:08 kbayliss

Sure, I've pulled my templates out of a project and pushed them to my fork: https://github.com/PaulBrownMagic/wagtailmenus

If you'd double check them for me I'll create the pull request?

PaulBrownMagic avatar Aug 30 '19 01:08 PaulBrownMagic

Thanks for the work, I just test it and it works quite good :)

Only in main_menu_dropdown.html I needed to change <a href="{{ item.href }}" class="nav-item{% if item.has_children_in_menu %} dropdown-toggle" ... to <a href="{{ item.href }}" class="nav-link{% if item.has_children_in_menu %} dropdown-toggle" ...

linuxluigi avatar Nov 01 '19 06:11 linuxluigi

Fixed it.

PaulBrownMagic avatar Nov 01 '19 06:11 PaulBrownMagic