-
Notifications
You must be signed in to change notification settings - Fork 45
Navbar is now responsive #60
Conversation
|
Deploy preview ready! Built with commit f8cb13c |
|
@buoyantair amazing, i was procrastinating this for a while now. thanks for bring this up here in form of pr. |
|
@M-ZubairAhmed You were searching for Issues on |
|
@buoyantair yes it does needs tweaking a bit, Also i was thinking we could move the hamburger on the left and logo in middle. Could you do that while you are at it. |
|
@M-ZubairAhmed Yeah will def do that when I tweak them the next time (probably this night idk) |
|
@buoyantair change of plans, we want to chuck the current menu and replace it with semantic-ui menu in web and in mobile we would like to have semantic-ui sidebar instead. We are on a time constraint here, can you implement it by today, if not its ok i will complete it off |
M-ZubairAhmed
left a comment
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.
change it to semantic ui side bar in mobile
|
@M-ZubairAhmed Im busy today, you might wanna take off if theres a time constraint |
|
thank you @buoyantair |
and also