Including classes on searchBar to identify when is open and/or filled

[igorpoubel]

What problem is this solving?

It includes classes to check if the result list is opened and if the input is filled

How should this be manually tested?

Checking if the class "open" will be included when starts to fill the input. And check if the class "hasTerm" is included when there is anything on input.

Workspace -> search for file

Checklist/Reminders

• Updated README.md.
• Updated CHANGELOG.md.
• Linked this PR to a Clubhouse story (if applicable).
• Updated/created tests (important for bug fixes).
• Deleted the workspace after merging this PR (if applicable).

Screenshots or example usage

When just filled (without focus)

When opened (focus)

Type of changes

✔️	Type of Change
_	Bug fix
✔️	New feature
_	Breaking change
_	Technical improvements

Notes

[vtex-io-ci-cd]

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

• Patch

• Minor

• Major

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

• No thanks, I would rather do it manually 😞

[vtex-io-docs-bot]

Beep boop 🤖

I noticed you didn't make any changes at the docs/ folder

• There's nothing new to document 🤔
• I'll do it later 😞

In order to keep track, I'll create an issue if you decide now is not a good time

• I just updated 🎉🎉

[Klynger]

Hi @igorpoubel , thank you for the PR, just a couple of simples changes:

1. You forgot to add the CHANGELOG
2. I think in this case you don't have to create new handles, just use applyModifiers, here is an example
3. Instead of hasTerm I think it would be better filled and opened as a modifier

[igorpoubel]

@Klynger thanks for the notice

on CHANGELOG I'll have to put a new manifest version (even not released) ou I'll just insert on the current version?

For exemplo:
The current version is 3.117.2.

Already have this CHANGELOG:

option 1 - So I'll have to insert this way?

option 2 - Or this way? (assuming a new release)

I think is the option 1, am I right?

[Klynger]

@Klynger thanks for the notice

on CHANGELOG I'll have to put a new manifest version (even not released) ou I'll just insert on the current version?

For exemplo:
The current version is 3.117.2.

Already have this CHANGELOG:

option 1 - So I'll have to insert this way?

option 2 - Or this way? (assuming a new release)

I think is the option 1, am I right?

No, you just add on Unreleased and don't change anything in manifest.json or package.json, this is done automatically when we launch the version

[igorpoubel]

@Klynger so I just have to do this?

And the version and date will be added automatically. Right?
Do I keep add or fix?

[Klynger]

@Klynger so I just have to do this?

And the version and date will be added automatically. Right?
Do I keep add or fix?

You use Added and remove the empty space between Unreleased and Added. We follow the spec of CHANGELOG, the only difference is the empty line between the version and the sections that we remove

[igorpoubel]

@Klynger ohh, now I got it.

Ok! Thanks.

[Klynger]

Hi @igorpoubel , after you make the changes you can request my review again so I can get notified to review your PR again

[Klynger]

There are some lint problems, you can fix this by installing the dependencies from the root of the app and running yarn lint --fix

[vtex-io-docs-bot]

Beep boop 🤖 That's ok, I created an issue for this so we don't forget

[Klynger]

@all-contributors add @igorpoubel for code

[allcontributors]

@Klynger

I've put up a pull request to add @igorpoubel! 🎉