Skip to content
This repository has been archived by the owner on Apr 20, 2021. It is now read-only.

Update website #25

Merged
merged 8 commits into from Jun 25, 2020
Merged

Update website #25

merged 8 commits into from Jun 25, 2020

Conversation

nissy-dev
Copy link
Member

@nissy-dev nissy-dev commented Jun 14, 2020

Fixed #24
Fixed #2
Fixed #7
Fixed #15

This PR is related to #24

What I did

  • Remove deepchemwebsite dir
  • Remove website/docs dir
  • Remove website/_sources dir
  • Remove website/_statics dir
  • Rename website/static to website/assets
  • Move all images to website/assets/img
  • Formet html and css
  • Add GitHub links in header
  • Fix broken links
  • Move js codes to website/assets/js/main.js -> I will make the anotehr PR
  • Fix mobile bugs -> I will make the anotehr PR

@nissy-dev nissy-dev changed the title [WiP] Update webiste [WIP] Update webiste Jun 14, 2020
@nissy-dev nissy-dev changed the title [WIP] Update webiste [WIP] Update website Jun 14, 2020
@rbharath
Copy link
Member

Thanks for taking on this cleanup! I think this will help simplify the website structure a good bit :)

style="padding:15%"></a>
</div>
<div class="col-sm-3 col-xs-6">
<a href="https://www.pandelab.org/"><img src="./assets/img/stanford.png"></a>
Copy link
Member Author

Choose a reason for hiding this comment

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

Pande lab link is broken. The link https://www.pandelab.org is OK?

@nissy-dev nissy-dev marked this pull request as ready for review June 25, 2020 08:50
@nissy-dev
Copy link
Member Author

nissy-dev commented Jun 25, 2020

@rbharath
I stopped refactoring js codes and fixing moible bugs.
I think this PR is ready to merge, so please check it!
https://github.com/nd-02110114/deepchem.io/tree/fix-link

I will make another PR about refactoring js codes and fixing moible bugs in the next week 🙇

@nissy-dev nissy-dev changed the title [WIP] Update website Update website Jun 25, 2020
Copy link
Member

@rbharath rbharath left a comment

Choose a reason for hiding this comment

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

Ok did a pass through and this looks good. Going to merge in.

@rbharath rbharath merged commit 446ec76 into deepchem:master Jun 25, 2020
@nissy-dev nissy-dev deleted the fix-link branch June 26, 2020 09:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants