Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

implement control sidebar #5536

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

jameslao
Copy link

@jameslao jameslao commented Apr 1, 2024

No description provided.

Copy link

netlify bot commented Apr 1, 2024

Deploy Preview for adminlte-v4 ready!

Name Link
🔨 Latest commit 5026bb8
🔍 Latest deploy log https://app.netlify.com/sites/adminlte-v4/deploys/664993a83e649000086e24a7
😎 Deploy Preview https://deploy-preview-5536--adminlte-v4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@danny007in
Copy link
Collaborator

where is toggle button

@jameslao
Copy link
Author

jameslao commented May 19, 2024

where is toggle button

Just added the panels. OverlayScrollbar may need some tweak and not quite functional as it seems...

@danny007in
Copy link
Collaborator

#5344 please check
I think theme generator is better that control sidebar

@jameslao
Copy link
Author

#5344 please check I think theme generator is better that control sidebar

Absolutely. I'll look at it

@jameslao
Copy link
Author

#5344 please check I think theme generator is better that control sidebar

Absolutely. I'll look at it

I'd agree. Theme generator can totally serve the purpose. Any specific help you might need before you merge?

@danny007in
Copy link
Collaborator

@jameslao i will push code and inform you please wait

@@ -4,7 +4,7 @@ const adminlteJsUrl = path + "/js/adminlte.js";
---

<!--begin::Third Party Plugin(OverlayScrollbars)-->

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<!--begin::Third Party Plugin(OverlayScrollbars)-->
<!--begin::Third Party Plugin(OverlayScrollbars)-->

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants