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.

Sign up for GitHub

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

Jump to bottom

Use Vue to show/hide insecure/disconnected icons instead of CSS #3088

Merged
merged 2 commits into from
Feb 27, 2019
Merged

Use Vue to show/hide insecure/disconnected icons instead of CSS #3088

merged 2 commits into from
Feb 27, 2019

Conversation

astorije
Copy link
Member

Follow-up of #3082 (comment).

@xPaw, I believe this is what you meant, let me know if I messed up something here.

@astorije astorije added the Meta: Internal This is an internal codebase change (testing, linting, etc.). label Feb 27, 2019
@astorije astorije added this to the 3.1.0 milestone Feb 27, 2019
@astorije astorije requested a review from xPaw February 27, 2019 07:27
@astorije astorije merged commit 67effc5 into master Feb 27, 2019
@astorije astorije deleted the astorije/vue-network-status branch February 27, 2019 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xPaw xPaw xPaw approved these changes

Assignees
No one assigned
Labels
Meta: Internal This is an internal codebase change (testing, linting, etc.).
Projects
None yet
Milestone
3.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@astorije @xPaw