Skip to content
This repository has been archived by the owner on Feb 17, 2019. It is now read-only.

Added navigation divider #93

Open
wants to merge 11 commits into
base: dev
Choose a base branch
from

Conversation

samagragupta
Copy link
Member

@samagragupta samagragupta commented Dec 15, 2018

Added navigation divider and shifted the header to the right.
cerebro

@netlify
Copy link

netlify bot commented Dec 15, 2018

Deployed at -

Built with commit e7607ab

https://deploy-preview-93--cerebro-web.netlify.com

Copy link
Member

@aashutoshrathi aashutoshrathi left a comment

Choose a reason for hiding this comment

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

  • Update your dev branch.
  • Rebase your current branch
  • Don't remove particle JS as of now.

@samagragupta
Copy link
Member Author

@aashutoshrathi Done.
cerebro

Copy link
Member

@aashutoshrathi aashutoshrathi left a comment

Choose a reason for hiding this comment

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

Also, can you link them to About Page's section by using their ID's?

src/components/header/index.js Outdated Show resolved Hide resolved
src/components/index.css Outdated Show resolved Hide resolved
src/components/header/styles.css Outdated Show resolved Hide resolved
@rossoskull
Copy link
Member

Side menu isn't supposed to be shown at the root route, we'll be using fullpage there to show the tasks, and at the location of this menu will come the nav menu of fullpage. Your views @aashutoshrathi?

@anshumanv
Copy link
Member

Update

@aashutoshrathi
Copy link
Member

@anshumanv I think none of deployments are working now.

Copy link
Member

@aashutoshrathi aashutoshrathi left a comment

Choose a reason for hiding this comment

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

Waiting for deployment fix

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

Successfully merging this pull request may close these issues.

4 participants