Aakansha99 / Github-Profile-Analyser

MIT License
16 stars 33 forks source link

Enhance site interface with CSS to more clearer and cleaner one, and minor site enhancements #58

Closed Git-Harshit closed 4 years ago

Git-Harshit commented 4 years ago

This commit adds improved looks to the light (non-dark) theme specifically, by lightening the background. Also, the theme toggler now indicates the current theme when toggled. Alongside, the commit now sets clickable links for user profiles followers, following, public repositories, and starred content. It also adds two headings to user profile details: users GitHub Pages hosted website, and Bio (About User). This PR is made in response to Issue #57.

Description

The updated code mainly aims at enhancing the site appearance, fixing issues like invisibility of darker text over darker section. Also, it involves minor code and performance improvisations.

Resolves #57

Type of Change:

Screenshots

Screenshots of the proposed changes compared with the current version.

Capture_Default-BG-Lightened

Capture_Light-BG-New-Looks

Capture_Dark_Theme

Capture_Theme_Button_OnHover

Capture_Buttons_OnHover

Capture_Profile_Clickables

Profile_Comparison

For older versions, of the displayed snippets, please check out the current version (before this PR is merged) of the project.

Additional Note:

Please add relevant labels to issue #57 like CH20, enhancement, and others (if not yet added, @Aakansha99) before merging this PR against the issue. It is all set, thanks to the repo owner!

Checklist:

Aakansha99 commented 4 years ago

Please add screenshots for the same

Git-Harshit commented 4 years ago

@Aakansha99, I have updated the PR description with added screenshots, and have also resolved a merge conflict with index.html file of base branch. Please have a look at it now.

Git-Harshit commented 4 years ago

Please let me know if the PR could be accepted and merged with the base branch, I'll fix the branch conflicts, just again. Else, feel free to close the PR.

Pawansinghla commented 4 years ago

please resolve these merge conflicts