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

created better nav bar #14

Merged
merged 3 commits into from
May 19, 2018
Merged

created better nav bar #14

merged 3 commits into from
May 19, 2018

Conversation

xeon-zolt
Copy link
Member

@xeon-zolt xeon-zolt commented May 6, 2018

Under construction 2 icons left
#10
screenshot_20180506_234603

Copy link
Contributor

@nityanandagohain nityanandagohain left a comment

Choose a reason for hiding this comment

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

I feel like the theme is not matching . The icons look too modern in a theme which is too simple .

@nityanandagohain
Copy link
Contributor

We can also try changing the background to match it .

Copy link
Member

@gabru-md gabru-md left a comment

Choose a reason for hiding this comment

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

Added Comments

-ms-flex-pack: center;
justify-content: center;
-webkit-box-align: center;
-ms-flex-align: center;
Copy link
Member

Choose a reason for hiding this comment

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

Check Indentation here please

Copy link
Member Author

Choose a reason for hiding this comment

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

screenshot_20180512_134200

i did it so that its easy for eyes to find things

css/style.css Outdated
background-color: #f0f6ff;
border-radius: 0.25rem;
-webkit-box-shadow: 0 0.5rem 1rem rgba(0, 0, 0, .3);
box-shadow: 0 0.5rem 1rem rgba(0, 0, 0, .3);
Copy link
Member

Choose a reason for hiding this comment

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

Indentation here as well

.navigation a::after {
-webkit-box-shadow: 0 0.5rem 1rem rgba(0, 0, 0, .3);
box-shadow: 0 0.5rem 1rem rgba(0, 0, 0, .3);
Copy link
Member

Choose a reason for hiding this comment

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

.

css/style.css Outdated
/* modules/navigation.css */

.navigation {
background-color: #f0f6ff;
Copy link
Member

Choose a reason for hiding this comment

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

As @nityanandagohain said, It is looking a bit absurd upon the current color scheme. Can you change the color please?
Something like : rgba(255,255,255,0.4) Change the transparency according to need to get a beautiful looking bottombar

@gabru-md
Copy link
Member

gabru-md commented May 8, 2018

Ping @xeon-zolt

@xeon-zolt
Copy link
Member Author

xeon-zolt commented May 8, 2018 via email

@Ananthavijay
Copy link
Contributor

@xeon-zolt I have a doubt in your code. Why do u have 5 (gitter,home,githhub,telegram,join) svg icons but only 4 slots?

@xeon-zolt
Copy link
Member Author

xeon-zolt commented May 10, 2018 via email

@Ananthavijay
Copy link
Contributor

@xeon-zolt I changed some things. check it here - https://codepen.io/Ananthavijay/pen/qYoWMv

@xeon-zolt
Copy link
Member Author

xeon-zolt commented May 10, 2018 via email

@Ananthavijay
Copy link
Contributor

Hey @xeon-zolt What if we add hover effect and colour changes for github,telegram and gitter.

@xeon-zolt
Copy link
Member Author

xeon-zolt commented May 10, 2018 via email

@Ananthavijay
Copy link
Contributor

Yes..I will do it after half an hour

@Ananthavijay
Copy link
Contributor

Hey @xeon-zolt check this - https://codepen.io/Ananthavijay/pen/qYoWMv
I replaced it with another one. Thanks

@Ananthavijay
Copy link
Contributor

@xeon-zolt I have the existing join us logo which is desaturated. But I have no idea on how to add it 😄 .
This is the link for the file - http://www.mediafire.com/file/dgmqq5i8u9hzuqx/affiliate.svg

@Ananthavijay
Copy link
Contributor

@xeon-zolt can u do the colour change on hover for the icons?

@gabru-md
Copy link
Member

This looks cool @Ananthavijay

Hey @xeon-zolt check this - https://codepen.io/Ananthavijay/pen/qYoWMv
I replaced it with another one. Thanks

@Ananthavijay
Copy link
Contributor

I completed the colour change thing... What is the colour should I change to?
ping : @xeon-zolt @gabru-md

@Ananthavijay
Copy link
Contributor

completed the colour change effect.. check it here - https://codepen.io/Ananthavijay/pen/qYoWMv

@Ananthavijay
Copy link
Contributor

@gabru-md just approve this pr. i will create a new pr afterwards. 😄

@xeon-zolt
Copy link
Member Author

@Ananthavijay or let me give you rights to make changes in my pr

@Ananthavijay
Copy link
Contributor

Anything is fine @xeon-zolt

@xeon-zolt
Copy link
Member Author

@Ananthavijay added your changes sorry i took little long

@Ananthavijay
Copy link
Contributor

@gabru-md @nityanandagohain I think the issue can be closed

Copy link
Contributor

@nityanandagohain nityanandagohain left a comment

Choose a reason for hiding this comment

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

LGTM

@xeon-zolt xeon-zolt merged commit c166521 into yogdaan:master May 19, 2018
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.

4 participants