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

Update source TB logout instructions #1522

Merged
merged 1 commit into from
Jan 13, 2017
Merged

Conversation

redshiftzero
Copy link
Contributor

Noticed during #1521 that the flashed message is not that easy to read (see diff to see before and after pics) so I reformatted that and removed the "turn off JavaScript" note from the bar and bubble that appears (leaving just the "turn security slider to high" part since the JS part is unnecessary info for the source (they can click through to find out why JS is bad if they really care)). Also updated outdated screenshots of the source interface in the docs

Copy link
Contributor

@psivesely psivesely left a comment

Choose a reason for hiding this comment

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

Good improvement to source interface! I agree mention of disabling JavaScript is unecessary + redundant as that's encompassed w/in the action of setting the Security Slider to High.

@psivesely psivesely merged commit ff841d0 into develop Jan 13, 2017
@psivesely psivesely deleted the pretty-logout-instructions branch January 13, 2017 03:00
@psivesely psivesely added this to the 0.3.11 milestone Jan 17, 2017
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