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
Fixes #825 : Created components for the background selection #826
Conversation
Hi @yashLadha! It looks like one or more of your builds have failed.
|
c438022
to
0650500
Compare
Codecov Report
@@ Coverage Diff @@
## development #826 +/- ##
==========================================
Coverage 100% 100%
==========================================
Files 1 1
Lines 43 43
==========================================
Hits 43 43 Continue to review full report at Codecov.
|
0650500
to
7e2021b
Compare
Hi @yashLadha! It looks like one or more of your builds have failed.
|
7709845
to
5f790bc
Compare
Please review @djmgit @ParthS007 @vaibhavsingh97 @gabru-md |
5f790bc
to
daffb32
Compare
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.
LGTM ! Please check the comments inline 👍
<div class="field"> | ||
<label>Select from a custom color</label> | ||
{{col-pick-input value=defColor}} | ||
</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.
Add a Line EOF 👍
{{/each}} | ||
</div> | ||
{{/ui-dropdown}} | ||
</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.
Add a line EOF 👍
<div class="center aligned column"> | ||
{{ data-component/text-component sendText=(action 'mutateText') }} | ||
</div> | ||
<div class="ui horizontal divider"></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.
Add a second UI form for background-component 👍
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.
Add Or
between the choices available 👍
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.
Please add the changes suggested by @ParthS007
init() { | ||
// To be inflated from the backend data | ||
this.image = ''; | ||
// this.images = [{ 'id': 0, 'name': 'Test 0' }, { 'id': 1, 'name': 'Test 1' }]; |
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.
Please remove this comment
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.
This is for the reference at later stage, if someone wants to work on the file. Can be removed easily by setting flag in eslint and fixing through --fix
cheers 👍
daffb32
to
94450a1
Compare
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.
LGTM !
94450a1
to
b492b10
Compare
Please review it @djmgit @vaibhavsingh97 @gabru-md done suggested changes |
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.
lgtm
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.
lgtm
@yashLadha please rebase your branch. |
b492b10
to
31fc2b4
Compare
Done rebasing @djmgit good to merge 👍 |
Fixes #825
Checklist
development
branch.Preview Link
Changes proposed in this pull request: