AdminLTEBundle icon indicating copy to clipboard operation
AdminLTEBundle copied to clipboard

Adding some customizations

Open loicallsoftware opened this issue 4 years ago • 7 comments

Hi, Here my pull request :

  • Able to custom your navbar and sidebar with navbar_class and sidebar_class in admin_lte_yaml
  • Wrap content-header row in a content_header block

Thanks for your accept. Regards, Loïc

loicallsoftware avatar Jan 17 '21 22:01 loicallsoftware

Thank you @loicallsoftware 👍
At a first glance this looks good, but ... please add some docs for the next user: https://github.com/kevinpapst/AdminLTEBundle/blob/master/Resources/docs/bundle_options.md https://github.com/kevinpapst/AdminLTEBundle/blob/master/Resources/docs/layout.md#layout-blocks then I will give it a try.

kevinpapst avatar Jan 17 '21 22:01 kevinpapst

Hi @kevinpapst . Ok, I will make a new pull request now. Because I think I can't update it !?

loicallsoftware avatar Jan 22 '21 18:01 loicallsoftware

Sure you can and you did with your last push 👍

kevinpapst avatar Jan 22 '21 18:01 kevinpapst

Ignore the failure, it is not related to your change!

kevinpapst avatar Jan 22 '21 18:01 kevinpapst

Hoo yes, OK, I didn't know that. Thank you ;-) Hope my documentation updates are well. Let me know.. Thanks for your good work on this project. Loïc

loicallsoftware avatar Jan 22 '21 18:01 loicallsoftware

Hi @kevinpapst , are you OK to merge it ? Or, are there something I have to do ?

loicallsoftware avatar Feb 09 '21 21:02 loicallsoftware

This here is a tiny side-project and changes like yours take a long time to properly review. It works for you, but I want to test it with 3 projects of mine.

Or, are there something I have to do

Yes: wait 😁 I will come back to it.

kevinpapst avatar Feb 10 '21 11:02 kevinpapst

Closing: I will archive this repo now

kevinpapst avatar Dec 07 '23 12:12 kevinpapst