-
Notifications
You must be signed in to change notification settings - Fork 73
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
Added simple navigation menu #5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hizkifw good start!
We'll focus on the design part in this task and I will create a new task to improve the django side (user permissions etc) after we are done with this.
Can you add a menu icon that is now very common among that type of menu?
I would like to keep only the links to the following items (in this order):
- Home (link to index)
- Device
- Templates
- VPN Servers
- Certification Authorities (CAs, but we can use the extended form here)
- Certificates
- Users
- Groups
- Organizations
Can you make each link like a square button? Like this website here (resize the window to make it narrow and the menu on the right will show up).
Any other improvement that will make the menu similar to the example I provided is welcome!
padding: 0 20px; | ||
float: left; | ||
background-color: #353535; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add blank line
#nav-activate:hover { | ||
background-color: #222; | ||
} | ||
#nav-activate:focus { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add blank line
#nav-activate:focus { | ||
outline: none; | ||
} | ||
#nav-top { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add blank line
background-color: #353535; | ||
transition: all .5s; | ||
z-index: 999; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add blank line
font-weight: bold; | ||
} | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove blank line
3baab92
to
0cd8267
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hizkifw could you squash the commits into two (one for frontend and one for backend) please?
@nemesisdesign Commits squashed |
openwisp_utils/admin.py
Outdated
@@ -1,8 +1,8 @@ | |||
from django.contrib import admin | |||
from django.contrib.admin import * |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Redundant. As we've already imported admin from django.contrib
openwisp_utils/admin.py
Outdated
@@ -96,3 +96,44 @@ class TimeReadonlyAdminMixin(object): | |||
def __init__(self, *args, **kwargs): | |||
self.readonly_fields += ('created', 'modified',) | |||
super(TimeReadonlyAdminMixin, self).__init__(*args, **kwargs) | |||
|
|||
class MyAdminSite(AdminSite): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can make it admin.AdminSite
This PR adds a simple collapsible menu, styled with some animations and support for tab navigation.