fixes #417 : link to Settings.html in the drop-down menu added .

tathagat2006 commented 5 years ago

Fixes #417

Checklist

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

Short description of what this resolves:

link to Settings.html in the drop-down menu added .

Changes proposed in this pull request:

Pop up window contains settings button to navigate to seetings.html

-

championpaddler commented 5 years ago

Already present in #418

gabru-md commented 5 years ago

I suppose we can merge this stuff, logout/login should be the last button in the drop down, isn't it? @rupav @ms10398 @anshumanv ?

championpaddler commented 5 years ago

@gabru-md I have already created #418

championpaddler commented 5 years ago

@gabru-md The patch has been already submitted in #418

rupav commented 5 years ago

@tathagat2006 duplicate PR, so this can be closed. Refer #420 . @gabru-md commented the need to have settings option above Login/Logout option in the dropdown in that PR.

tathagat2006 commented 5 years ago

@gabru-md .idea folder removed. Thanks :)

championpaddler commented 5 years ago

@tathagat2006 This is copy of #418

akshatnitd commented 5 years ago

The fix for this issue has already been merged, You can try out some other issues.

fossasia / susi_chromebot