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

Updated - Navigation Bar Styling, animation, highlights active section and page smooth scrolling etc. #32

Closed
wants to merge 7 commits into from

Conversation

0xSG
Copy link

@0xSG 0xSG commented Nov 7, 2018

Features added:

  • Automatically update links in a navigation list based on scroll position.
  • Favicon Added: Wikimedia Commons logo added as favicon.
  • Smooth scrolling Added: On clicking on the navigation links, it scrolls smoothly to the section.
  • Navigation bar:
    -- Styling and Coloring.
    -- Animation of active link.

Automatically update links in a navigation list based on scroll position
wikimedia commons logo added as favicon.
On clicking on the navigation links, it scrolls smoothly to the section.
NavBar:
* Styling and coloring.
* Animation of active link.
* Added comments to all added
 codes.
@0xSG 0xSG changed the title Just updated Navigation Bar Styling, animation, highlights active section and page smooth scrolling etc. Updated Navigation Bar Styling, animation, highlights active section and page smooth scrolling etc. Nov 7, 2018
@0xSG 0xSG changed the title Updated Navigation Bar Styling, animation, highlights active section and page smooth scrolling etc. Updated - Navigation Bar Styling, animation, highlights active section and page smooth scrolling etc. Nov 7, 2018
@0xSG
Copy link
Author

0xSG commented Nov 8, 2018

Hi. This is Surya here. The code which i added is tested and it's live at here.

@sivaraam
Copy link
Member

sivaraam commented Dec 2, 2018

Hi @0xpulsar,

Just saw the live website. The changes seems to be nice. But the favicon doesn't seem to have come into effect. I still see no icon in the tab for the website. Any idea why?

@0xSG
Copy link
Author

0xSG commented Dec 2, 2018

Let me check.

@0xSG
Copy link
Author

0xSG commented Dec 20, 2018

@sivaraam Can you fix it?
My exams are going on.

@sivaraam
Copy link
Member

Can you fix it?

Just to be sure, fix what? Fix the missing website icon?

The root of the website that goes live might not be the
current directory. So, expecting the 'images' folder to be
in the root is not valid.

Referencing the 'images' folder relatively fixes this issue.
@sivaraam
Copy link
Member

Anyways, I guess I've found the issue. I made a PR to your repo with the fix. 0xSG#1

You could see it live at https://sivaraam.github.io/commons-app.github.io/

@0xSG
Copy link
Author

0xSG commented Dec 23, 2018

@sivaraam, I have merged and solved merge conflict. can you review it?.

@sivaraam
Copy link
Member

@0xpulsar On a cursory view, I guess the merge isn't proper. You seem to be including styles from both bootstrap 4.0.0-alpha and 4.1.1 as a consequence of which the page looks odd at some places. Could you re-merge it properly?

@sivaraam sivaraam mentioned this pull request Feb 23, 2019
@sivaraam
Copy link
Member

Closing this as #34 has been merged. Also, the website icon has been added in #38.

Hope to see you again @0xpulsar with another PR :)

@sivaraam sivaraam closed this Mar 31, 2019
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.

2 participants