Redesign Settings Pages, More functionality#381

fragm3 commented 5 years ago

Fixes #381

PR: https://github.com/fossasia/susi_chromebot/pull/388 had undetectable merge conflicts, so creating a new PR.

Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[x] The unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[x] All the functions created/modified in this PR contain relevant docstrings.

Short description of what this resolves:

Screenshots:

Before:

Changes proposed in this pull request:

Added settings in tab
Modified settings page
Added custom theme picker
Add ignore DS_store file to gitignore

fragm3 commented 5 years ago

@zamhaq @Orbiter @gabru-md @ms10398 @rupav please review.

fragm3 commented 5 years ago

@gabru-md The color picker used is external dependency, the CSS files wont make it slower, since its just styling. And will be loaded in the settings page only, so maybe it's fine? Also, I can minify the css, should I do that? Thanks for your review.

fragm3 commented 5 years ago

@rupav You gave the wrong link.

The web app order:

How about we order like this:

Chat
Skills
Clear Chat
Export
Settings
About
Logout

rupav commented 5 years ago

Oh sorry @fragm3 . Correct link: https://github.com/fossasia/susi_chromebot/pull/425#issuecomment-427737133. Yes agreed, about as second last option makes sense.

fragm3 commented 5 years ago

Merge conflicts due to renaming of file. After renaming:

fossasia / susi_chromebot