Skip to content

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

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