Skip to content
This repository has been archived by the owner on Jan 26, 2024. It is now read-only.

Fixed Remove outline of input search bar #687 #689

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

priyal-hotwax
Copy link

@priyal-hotwax priyal-hotwax commented Dec 7, 2021

Related Issues

Closes #687

Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change

Contribution and Currently Important Rules Acceptance

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


priyal-hotwax seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@priyal-hotwax
Copy link
Author

image

Copy link
Collaborator

@dt2patel dt2patel left a comment

Choose a reason for hiding this comment

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

Please take a look at my comment. Mostly looks good though.

App.vue Outdated Show resolved Hide resolved
@@ -74,4 +74,9 @@ body {
}
}
}

.sf-input input, .sf-search-bar__input {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove outline of input
3 participants