-
Notifications
You must be signed in to change notification settings - Fork 771
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
Ui/page faqs #58
Ui/page faqs #58
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #58 +/- ##
==========================================
- Coverage 1.73% 1.68% -0.06%
==========================================
Files 295 311 +16
Lines 3972 4101 +129
Branches 489 497 +8
==========================================
Hits 69 69
- Misses 3569 3698 +129
Partials 334 334
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a couple of comments. Maybe we can refactor those.
@@ -0,0 +1,18 @@ | |||
<template> | |||
<div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whats this empty component for?
methods: { | ||
openFAQ: function (faqToOpen) { | ||
var currentState = this.showFAQs[faqToOpen] | ||
this.showFAQs.faq1 = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't you replace this with a string instead of having multiple booleans? use v-show to check whether the article should be shown or not.
data () { | ||
return { | ||
showFAQs: { | ||
faq1: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment above
# Conflicts: # src/components/FaqContents/CouldntEverybodyPutInRandomPrivateKeys.vue # src/components/FaqContents/DoesMyEtherWalletSupportBitcoinOrOtherCoins.vue # src/components/FaqContents/HowToBuyEthereumWithUSD.vue # src/components/FaqContents/WhyDontYouMakeMeMoveMyMouseToGenerateASeed.vue # src/components/FaqContents/WithdrawingDAOTokens.vue # src/components/FaqContents/index.js # src/containers/ByMnemonicContainer/ByMnemonicContainer.vue # src/containers/ByMnemonicContainer/components/VerificationModal/VerificationModal.vue # src/containers/HeaderContainer/HeaderContainer.vue # src/layouts/CreateWalletLayout/components/CreateWalletInput/CreateWalletInput.vue # src/layouts/HelpCenterLayout/HelpCenterLayout.vue # src/router/index.js
# Conflicts: # src/components/FaqContents/HowToBuyEthereumWithUSD.vue # src/components/FaqContents/WithdrawingDAOTokens.vue
included in Devop/fix develop #83 |
Devop